From 972baf6071e5cb4efbb327c1fd33dbd7c395e18c Mon Sep 17 00:00:00 2001 From: Dror Atariah Date: Fri, 19 Apr 2013 17:21:29 +0200 Subject: [PATCH] Added two Roadmap items - Correct the _locate() - Correct the way functors access the segment traits. --- .../include/CGAL/Arr_polyline_traits_2.h | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/Arrangement_on_surface_2/include/CGAL/Arr_polyline_traits_2.h b/Arrangement_on_surface_2/include/CGAL/Arr_polyline_traits_2.h index 50cfbcfc481..bca97ed05a0 100644 --- a/Arrangement_on_surface_2/include/CGAL/Arr_polyline_traits_2.h +++ b/Arrangement_on_surface_2/include/CGAL/Arr_polyline_traits_2.h @@ -28,6 +28,13 @@ * to the user. */ +/* + * Roadmap: In all functors, replace the member m_seg_traits with an instance + * of Arr_polyline_traits. Then use the member function + * segment_traits_2() to get whatever needed from the segment traits. + * See the implementation of Equal_2 + */ + #ifndef CGAL_ARR_POLYLINE_TRAITS_2_H #define CGAL_ARR_POLYLINE_TRAITS_2_H @@ -1555,6 +1562,12 @@ public: //@} private: + /* + * Roadmap: + * - _locate() should return an iterator to the located segment + * - Should not get m_seg_traits as an argument, instead it should be called + * as a member function of the Arr_polyline_traits_2 object. + */ /*! * Return the index of the segment in the polyline that contains the * point q in its x-range. The function performs a binary search, so if the