Subject: CGAL users discussion list
List archive
- From: Efraim Fogel <>
- To:
- Subject: Re: [cgal-discuss] Arr_vertical_decomposition_visitor.h
- Date: Tue, 30 Dec 2008 17:49:26 +0200
Jannis Warnat wrote:
Hi,It looks like you are right.
following code segment is from Arr_vertical_decomposition_visitor.h (CGAL-3.3.1):
bool after_handle_event(Event* event, SL_iterator above, bool /* above_on_event */)
{
if (! event->is_finite())
{
// We only need to consider events with a finite x-cooridinates.
if (event->infinity_at_x() != NO_BOUNDARY)
return (true);
Boundary_type y_inf = event->infinity_at_y();
if (y_inf == PLUS_INFINITY)
{
// Update the fictitious top halfedge.
m_top_he = m_top_he->twin()->next()->twin();
}
else
{
CGAL_assertion (y_inf == PLUS_INFINITY);
// Update the fictitious bottom halfedge.
m_bottom_he = m_bottom_he->prev();
}
return (true);
}
The CGAL_assertion should check for y_inf == MINUS_INFINITY, shouldn´t it?
Kind regards,
Jannis Warnat
The entire arrangement package was revamped. The coming release will contain the new stuff including new code for vertical decomposition.
--
____ _ ____ _
/_____/_) o /__________ __ //
(____ ( ( ( (_/ (_/-(-'_(/
_/
- [cgal-discuss] Arr_vertical_decomposition_visitor.h, Jannis Warnat, 12/30/2008
- Re: [cgal-discuss] Arr_vertical_decomposition_visitor.h, Efraim Fogel, 12/30/2008
Archive powered by MHonArc 2.6.16.