Re: [Boost-bugs] [Boost C++ Libraries] #9378: g++ 4.7 -Wshadow warnings need attention

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #9378: g++ 4.7 -Wshadow warnings need attention
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2014-03-18 22:53:23

#9378: g++ 4.7 -Wshadow warnings need attention
------------------------------------------+--------------------------
  Reporter:  Tom Browder <tom.browder@…>  |      Owner:  eric_niebler
      Type:  Bugs                         |     Status:  new
 Milestone:  To Be Determined             |  Component:  xpressive
   Version:  Boost 1.55.0                 |   Severity:  Problem
Resolution:                               |   Keywords:
------------------------------------------+--------------------------

Comment (by anonymous):

 Not sure how you can think that way. This is not a question of "working
 around the quirks" of compilers, it's about writing good code. As I said,
 we had an error due to shadowing. Compilers almost universally support
 warning you about this, because it leads to mistakes. But code that
 prevents us from taking advantage of the compiler makes it harder for us
 to write good code.

 Boost itself can benefit from not shadowing variables, so I don't
 understand why you wouldn't be all for it. It's simple: shadowing
 variables opens you up to potential errors, and not shadowing is trivial.

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/9378#comment:8>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.