Re: [Boost-bugs] [Boost C++ Libraries] #12362: Default case handling in exception_ptr missing clue for coverity to treat switch fall-through as intentional

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #12362: Default case handling in exception_ptr missing clue for coverity to treat switch fall-through as intentional
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2016-07-30 00:57:56

#12362: Default case handling in exception_ptr missing clue for coverity to treat
switch fall-through as intentional
----------------------------------------------+----------------------------
  Reporter:  James E. King, III <jim.king@…>  |      Owner:  emildotchevski
      Type:  Bugs                             |     Status:  closed
 Milestone:  To Be Determined                 |  Component:  exception
   Version:  Boost 1.54.0                     |   Severity:  Cosmetic
Resolution:  invalid                          |   Keywords:
----------------------------------------------+----------------------------

Comment (by James E. King, III <jim.king@…>):

 Is BOOST_ASSERT in every build or it is only in debug builds?

 If the next line is never reached, there's probably a coverity comment
 hint to indicate that too like the old /* NOTREACHED */ for lint.

 If it does not assert in a release build then it falls through and perhaps
 an exception would be a better choice so that it does not behave
 differently in debug vs. release?

 Thanks.

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