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.
This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:04 UTC