Re: [Boost-bugs] [Boost C++ Libraries] #7028: NDEBUG should disable asserts even if BOOST_ENABLE_ASSERT_HANDLER is defined

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #7028: NDEBUG should disable asserts even if BOOST_ENABLE_ASSERT_HANDLER is defined
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2013-03-14 21:50:51

#7028: NDEBUG should disable asserts even if BOOST_ENABLE_ASSERT_HANDLER is
defined
--------------------------------------+-------------------------------------
  Reporter:  mgaunard                 |       Owner:  viboes                                                
      Type:  Bugs                     |      Status:  assigned                                              
 Milestone:  To Be Determined         |   Component:  utility                                               
   Version:  Boost Development Trunk  |    Severity:  Problem                                               
Resolution:                           |    Keywords:  BOOST_ASSERT assert BOOST_ENABLE_ASSERT_HANDLER NDEBUG
--------------------------------------+-------------------------------------

Comment (by viboes):

 Sorry, I have missed the history of this library. IUUC now the
 introduction of BOOST_ASSERT_MSG in [68414] introduce the
 {{{defined(NDEBUG)}}}


 {{{
 #if defined(BOOST_DISABLE_ASSERTS) || defined(NDEBUG)

   #define BOOST_ASSERT_MSG(expr, msg) ((void)0)
 }}}

 which made BOOST_ASSERT_MSG to behave differently than BOOST_ASSERT.

 The introduction of BOOST_VERIFY using

 {{{
 #if defined(BOOST_DISABLE_ASSERTS) || (
 !defined(BOOST_ENABLE_ASSERT_HANDLER) && defined(NDEBUG) )
 }}}


 made also a difference.

 So should we remove the use of defined(NDEBUG) and update the
 documentation of BOOST_ASSERT_MSG?

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