Apply suggestions from code review

Co-authored-by: Mael <mael.rouxel.labbe@geometryfactory.com>
This commit is contained in:
Sebastien Loriot 2021-12-17 10:17:13 +01:00 committed by GitHub
parent 0183aac332
commit bd6408a772
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 5 additions and 6 deletions

View File

@ -13,9 +13,9 @@
// $Id$
// SPDX-License-Identifier: BSL-1.0
//
// NOTE: this file have been taken from boost 1.77 for using
// with Modificable_priority_queue
// original file is <boost/pending/relaxed_heap.hpp>
// NOTE: this file has been taken from boost 1.77 to use
// with Modificable_priority_queue.
// The original file is <boost/pending/relaxed_heap.hpp>
//
#ifndef BOOST_RELAXED_HEAP_HEADER

View File

@ -93,8 +93,8 @@ the number of edges effectively removed.
\cgalParamType{Either `CGAL::Tag_true` or `CGAL::Tag_false`}
\cgalParamDefault{`CGAL::Tag_false()`}
\cgalParamExtra{Using a relaxed order will allow the algorithm to use a faster priority queue.
However, if for example a user expects that all elements shorter than a given value to be collapsed,
a relaxed order shall not be used.}
However, the ordering of the priority queue is no longer strict and there is a possibility
that some elements that ought to have been collapsed are not actually collapsed.}
\cgalParamNEnd
\cgalNamedParamsEnd

View File

@ -313,7 +313,6 @@ private:
CGAL_expensive_assertion(mPQ->contains(h));
mPQ->update(h);
data.set_is_in_PQ();
CGAL_assertion(data.is_in_PQ());
CGAL_expensive_assertion(mPQ->contains(h));