On Thu, Nov 7, 2013 at 5:37 PM, Christopher Sean Morrison
<[email protected]> wrote:
> On Nov 7, 2013, at 6:29 PM, Tom Browder wrote:
>> Well, how about header only stuff?  E.g., flat_(map|set) which is what
...
> Sure, little harm in header-only components of Boost (which increasingly more 
> of Boost is not).

I have mentioned this before but with the latest Boost I am getting
lots of warnings due to -Wshadow--too many to file a bug for.

I pinged the Boost users list about the problem and stirred up a small
hornets' nest--one supporter and one not.  Here's part of the
naysayers comment (the first response to my e-mail):

<quote>
Different people have different conventions for things like member
functions. In this case, the author seems to be a fan of the "access any
members via this" approach, where it's perfectly fine to have shadowing.

If the whole reason for this hubbub is because you're one of those folks
that use -Werror, it's your responsibility to tune your set of warnings
to rule out the ones that cause excessive false positives.
</quote>

And here's my supporter's response:

<quote>
> Warnings are not errors for a reason, they exist as advice that may or
> may not apply.

Or they may, and any convention for managing shadowing applied to
headers would trivially enable a coder to enable shadowing warnings on
their own code. This potentially saves them mysterious bugs and
sniffing around in a debugger before finding that it was something the
compiler could have told them about if it weren't lost in the noise.

I agree with not being religious about it, but -Wshadow is a pretty
achievable goal.
</quote>

Maybe there is hope!

Best regards,

-Tom

------------------------------------------------------------------------------
November Webinars for C, C++, Fortran Developers
Accelerate application performance with scalable programming models. Explore
techniques for threading, error checking, porting, and tuning. Get the most 
from the latest Intel processors and coprocessors. See abstracts and register
http://pubads.g.doubleclick.net/gampad/clk?id=60136231&iu=/4140/ostg.clktrk
_______________________________________________
BRL-CAD Developer mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-devel

Reply via email to