Opened 8 years ago

Closed 7 years ago

#9960 closed Tasks (fixed)

Warnings on Clang

Reported by: Raffi Enficiaud Owned by: Raffi Enficiaud
Milestone: Boost 1.59.0 Component: test
Version: Boost Development Trunk Severity: Cosmetic
Keywords: Cc:

Description

Follow up of the merge request from Tim Blechmann. https://github.com/boostorg/test/pull/4

Requesting more info from Tim Blechman.

Change History (3)

comment:1 by timblechmann, 8 years ago

first: my family name contains two 'n'.

second: the warning fires on standard-compliant code. there is nothing to 'fix' only to 'work around'

second: iirc i was seeing the warning on ubuntu/clang, do not remember which version and currently i do not have time to check.

comment:2 by anonymous, 8 years ago

"first[...] second[...] second[...]" "first: my family name contains two 'n'. "

Please be less aggressive. From now on, I won't accept any merge to develop/master if there is no proper ticket filled.

"second: iirc i was seeing the warning on ubuntu/clang."

This is the kind of information I was asking. Currently I am not reproducing these warnings with the version of clang I am using (as I said OSX/XCode5.1). So I will give a try on Ubuntu.

"i do not have time to check"

This is not like I am on vacation neither.

You are willing to contribute, I am willing to maintain. Let's work properly together.

The warning described on SO https://stackoverflow.com/questions/16939888/why-does-clang-warn-within is a real one, in the sense that is warns for code that is prone to mistakes. I prefer fixing it, or at least selectively silence warnings coming from other libraries, which I guess should not be such a big deal.

Best,
Raffi

comment:3 by Raffi Enficiaud, 7 years ago

Milestone: To Be DeterminedBoost 1.59.0
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.