Re: [Boost-bugs] [Boost C++ Libraries] #4527: warning in Boost.format and Boost.test C4224

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #4527: warning in Boost.format and Boost.test C4224
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2010-08-13 17:33:34

#4527: warning in Boost.format and Boost.test C4224
--------------------------------------+-------------------------------------
  Reporter:  pbristow                 |       Owner:  samuel_krempp
      Type:  Bugs                     |      Status:  new          
 Milestone:  Boost 1.44.0             |   Component:  format       
   Version:  Boost Development Trunk  |    Severity:  Cosmetic     
Resolution:                           |    Keywords:  format C4224 
--------------------------------------+-------------------------------------

Comment (by steven_watanabe):

 The code is perfectly fine.  To see just how much code would have to be
 changed to avoid this warning entirely try running:

 {{{
 grep -R -I --exclude-dir=.svn --exclude-dir=bind --exclude-dir=mpl
 '\barg\b' boost
 }}}

 Reusing a name in this context is not a stylistic problem at all.  The use
 of arg in Boost.Bind is totally irrelevant to Boost.Format.  When I'm
 writing an argument name in my own member function in my own class in my
 own namespace, I don't want to have to worry about any other part of
 Boost.  In order to prevent it we can't use any class name in any parent
 namespace as an argument name.  This means that classes added in the
 global namespace (which we have no control over whatsoever) can trigger
 this warning in random parts of Boost.

 Also, /Za is not turned on by default anyway.

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