[Boost-bugs] [Boost C++ Libraries] #6044: Assertion failed in relaxed_heap remove()

Subject: [Boost-bugs] [Boost C++ Libraries] #6044: Assertion failed in relaxed_heap remove()
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2011-10-20 20:27:02

#6044: Assertion failed in relaxed_heap remove()
--------------------------------------------------------+-------------------
 Reporter:  Ponomarev Andrew <ponomarev.a.v@…>          |       Owner:         
     Type:  Bugs                                        |      Status:  new    
Milestone:  To Be Determined                            |   Component:  None   
  Version:  Boost 1.47.0                                |    Severity:  Problem
 Keywords:                                              |  
--------------------------------------------------------+-------------------
 There is an assertion in boost/pending/relaxed_heap.hpp:166 when trying to
 remove() a zero-int from a relaxed_heap.
 Moreover, my investigation showed that that assert statement doesn't work
 in a proper way at all. As far as I understand it's purpose is to verify
 that the element being removed is present in heap. But attempt to remove()
 inexistent element doesn't cause an assertion, in the opposite, an attempt
 to remove() zero-int always cause an assertion.

 I propose to change this assertion to the way this check is performed in
 contains() method. Proposed patch is enclosed.

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