CGAL users discussion list

["Sebastien Loriot (GeometryFactory)" <>]

Hi Marius,

Thanks for the report. It should indeed be k<i. I'll fix it for the next 
release.

By curiosity how did you find the bug because the manifold edge test
coming after is failing on the tests I did.

Sebastien.

On 01/23/2014 06:12 AM, Marius Kintel wrote:
> Hi,
>
> Polyhedron_incremental_builder_3::test_facet_indices() tries to detect if a 
> list of indices has the same index specified twice.
> Due to a bug, this test doesn’t actually detect that in all cases:
> e.g., sending the vector [6,6,5,6] will pass this test while it should 
> obviously fail.
>
> The problem appears to be on line 752:
>          for ( std::size_t k = 0; k+1 < i; ++k) {
>
> ..where k is being iterated one step too short to detect this issue.
>
>   -Marius