[Boost-bugs] [Boost C++ Libraries] #7752: algorithm minmax_element triggers warning with Clang

Subject: [Boost-bugs] [Boost C++ Libraries] #7752: algorithm minmax_element triggers warning with Clang
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2012-11-30 12:07:15

#7752: algorithm minmax_element triggers warning with Clang
------------------------------------------+---------------------------------
 Reporter:  pbristow                      |       Owner:  marshall    
     Type:  Support Requests              |      Status:  new         
Milestone:  To Be Determined              |   Component:  algorithm   
  Version:  Boost Development Trunk       |    Severity:  Optimization
 Keywords:  minmax_element warning clang  |  
------------------------------------------+---------------------------------
 algorithm minmax_element triggers warning with Clang 3.1

 {{{
 if (comp(first, min_result))
   min_result = first, potential_min_result = last;
 }}}

 is flagged as having Cish comma abuse ;-)

 Unless I'm missing something subtle, this can easily be avoided with the
 more C++ish?

 {{{
 if (comp(first, min_result))
 {
   min_result = first;
   potential_min_result = last;
 }
 }}}

 path attached.

 (This does not show when compiling tests, but does show in user code.
 There are many duplicated warnings from Boost.Test that make it difficult
 to find other possibly helpful warnings.  And hex_test1 aborted...).

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