[Boost-bugs] [Boost C++ Libraries] #11765: boost_1_59_0/boost/intrusive/sgtree.hpp:830: bad if test ?

Subject: [Boost-bugs] [Boost C++ Libraries] #11765: boost_1_59_0/boost/intrusive/sgtree.hpp:830: bad if test ?
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2015-10-27 17:22:16

#11765: boost_1_59_0/boost/intrusive/sgtree.hpp:830: bad if test ?
----------------------------------------+------------------------------
 Reporter:  David Binderman <dcb314@…>  |       Type:  Bugs
   Status:  new                         |  Milestone:  To Be Determined
Component:  None                        |    Version:  Boost 1.59.0
 Severity:  Problem                     |   Keywords:
----------------------------------------+------------------------------
 [boost_1_59_0/boost/intrusive/sgtree.hpp:830]: (warning) Logical
 conjunction always evaluates to false: new_alpha < 0.5f && new_alpha >=
 1.0f.

 Source code is

      if(new_alpha < 0.5f && new_alpha >= 1.0f)  return;

 Maybe better code

      if(new_alpha < 0.5f || new_alpha >= 1.0f)  return;

 Compiling with gcc flag -Wlogical-op might help find further
 examples of this problem.

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