Subject: Re: [Boost-bugs] [Boost C++ Libraries] #4353: warning needs suppressed --> warning: "_MSC_VER" is not defined
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2010-06-20 20:09:48
#4353: warning needs suppressed --> warning: "_MSC_VER" is not defined
-----------------------------------------+----------------------------------
Reporter: Willem.deconinck@⦠| Owner: neilgroves
Type: Bugs | Status: new
Milestone: Boost 1.43.0 | Component: range
Version: Boost 1.43.0 | Severity: Cosmetic
Resolution: | Keywords:
-----------------------------------------+----------------------------------
Comment (by mdorey@â¦):
It's perhaps stating the obvious, but nonetheless worth doing so for the
benefit of search engines, that this warning occurs with gcc when passed
the -Wundef flag. (This little-known and rarely-used flag is useful
because it stops you from doing #if tests when the appropriate header file
hasn't been included.) Much of boost is clean in the presence of this
flag and I've had other similar patches accepted in the past. Another
technique that's sometimes used to fix this sort of issue is
BOOST_WORKAROUND, as demonstrated in eg
https://svn.boost.org/trac/boost/ticket/1291. I didn't choose that
approach because I'm less familiar with it and I was concerned for
consistency with the existing defined(_MSC_VER) test higher up this file.
-- Ticket URL: <https://svn.boost.org/trac/boost/ticket/4353#comment:2> 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:03 UTC