Re: [Boost-bugs] [Boost C++ Libraries] #7022: Prevent tool Bug Fix.

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #7022: Prevent tool Bug Fix.
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2012-06-28 11:05:08

#7022: Prevent tool Bug Fix.
----------------------------------+-----------------------------------------
  Reporter:  g.gupta@…            |       Owner:  az_sw_dude
      Type:  Bugs                 |      Status:  new       
 Milestone:  To Be Determined     |   Component:  date_time 
   Version:  Boost 1.50.0         |    Severity:  Problem   
Resolution:                       |    Keywords:            
----------------------------------+-----------------------------------------
Description changed by viboes:

Old description:

> It is Prevent tool Bug Fix.In file
> boost/trunk/libs/date_time/src/gregorian/date_generators.cpp
>
> the IF condition:
>

> {{{
>    if(ele >= 1 || ele <= 5) {
>       return _nth_as_str[ele];
>       }
>     else {
>       return _nth_as_str[0];
>       }
>
> }}}
>
> is not used properly. Instead of "||" in this if condition it should be
> "&&". Only by use of "&&" this condition makes sense.

New description:

 It is Prevent tool Bug Fix.In file
 boost/trunk/libs/date_time/src/gregorian/date_generators.cpp

 the IF condition:


 {{{
    if(ele >= 1 || ele <= 5) {
       return _nth_as_str[ele];
       }
     else {
       return _nth_as_str[0];
       }

 }}}

 is not used properly. Instead of "{{{||}}}
 " in this if condition it should be "&&". Only by use of "&&" this
 condition makes sense.

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