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 , 8 years ago
comment:2 by , 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 , 7 years ago
Milestone: | To Be Determined → Boost 1.59.0 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
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.