[Boost-bugs] [Boost C++ Libraries] #3262: Boost::Accumulators::Statistics: Unused Globals Cause Warnings/Errors When Headers Are Not in System Path

Subject: [Boost-bugs] [Boost C++ Libraries] #3262: Boost::Accumulators::Statistics: Unused Globals Cause Warnings/Errors When Headers Are Not in System Path
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2009-07-14 02:24:10

#3262: Boost::Accumulators::Statistics: Unused Globals Cause Warnings/Errors When
Headers Are Not in System Path
--------------------------------------+-------------------------------------
 Reporter:  gerickson@…               |       Owner:  eric_niebler
     Type:  Bugs                      |      Status:  new         
Milestone:  To Be Determined          |   Component:  accumulator 
  Version:  Boost 1.39.0              |    Severity:  Problem     
 Keywords:                            |  
--------------------------------------+-------------------------------------
 I've recently added Boost 1.39.0 to an Xcode 3.1.3 project with the
 following warnings enabled in GCC/G++ 4.0.1:

 -Werror -Wreturn-type -Wunused-function -Wunused-label -Wunused-variable
 -Wshadow

 Unfortunately, when boost is not installed in a "system" path--as is the
 case here--unused and shadow variable warnings cause the build to fail
 when warnings are treated as errors. This issue last came up for me about
 a year ago in 1_35_0 with filesystem.hpp:

 http://article.gmane.org/gmane.comp.lib.boost.user/38297/match=shadow+variables+cause


 The included headers this time are:

 boost/accumulators/statistics/mean.hpp
 boost/accumulators/statistics/sum.hpp

 To test and fix, I isolated with:

 printf "#include <${BoostHeader}>\nint main(void) { return 0; }" |
 /Developer/usr/bin/gcc-4.0 -x c++ -Wall -Wshadow -Werror -o test
 -I${BoostRoot}/boost/include -

 The patches to address this are attached and impact the headers below:

 boost/accumulators/statistics/mean.hpp
 boost/accumulators/statistics/sum.hpp

 Because this problem seems to come up frequently, perhaps there is a way
 -Wshadow and -Wall can be forced for all unit tests?

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