[Boost-bugs] [Boost C++ Libraries] #7338: pending/indirect_cmp has wrong return type

Subject: [Boost-bugs] [Boost C++ Libraries] #7338: pending/indirect_cmp has wrong return type
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2012-09-06 08:34:43

#7338: pending/indirect_cmp has wrong return type
-------------------------------------------------+--------------------------
 Reporter:  Alex Hagen-Zanker <ahh34@…>          |       Owner:  jewillco
     Type:  Bugs                                 |      Status:  new     
Milestone:  To Be Determined                     |   Component:  graph   
  Version:  Boost 1.52.0                         |    Severity:  Problem 
 Keywords:  pending                              |  
-------------------------------------------------+--------------------------
 The return_type for indirect_cmp now is PropertyMap::value_type it should
 be Compare::result_type (or simply bool).

 The return type for the operator(a,b) should then be return_type instead
 of bool.

 Also, the PropertyMap may be an LValuePropertyMap and then it might be
 avoided to copy the values before comparison.

 Patch attached

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