From a5a830bf605e74f8faa3e8f76eb45b94241d576e Mon Sep 17 00:00:00 2001 From: Manuel Caroli Date: Thu, 28 May 2009 15:19:17 +0000 Subject: [PATCH] removing obsolete todos --- Periodic_3_triangulation_3/include/CGAL/Periodic_3_offset_3.h | 1 - .../include/CGAL/Periodic_3_triangulation_3.h | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/Periodic_3_triangulation_3/include/CGAL/Periodic_3_offset_3.h b/Periodic_3_triangulation_3/include/CGAL/Periodic_3_offset_3.h index b782658eda7..f8f14114224 100644 --- a/Periodic_3_triangulation_3/include/CGAL/Periodic_3_offset_3.h +++ b/Periodic_3_triangulation_3/include/CGAL/Periodic_3_offset_3.h @@ -27,7 +27,6 @@ CGAL_BEGIN_NAMESPACE -// NGHK: make memory efficient later: class Periodic_3_offset_3 { template friend std::ostream & operator<<(std::ostream &os, diff --git a/Periodic_3_triangulation_3/include/CGAL/Periodic_3_triangulation_3.h b/Periodic_3_triangulation_3/include/CGAL/Periodic_3_triangulation_3.h index 1196768ce0e..e79ceddcc9b 100644 --- a/Periodic_3_triangulation_3/include/CGAL/Periodic_3_triangulation_3.h +++ b/Periodic_3_triangulation_3/include/CGAL/Periodic_3_triangulation_3.h @@ -701,7 +701,6 @@ public: lt = EMPTY; return ON_UNBOUNDED_SIDE; } - //TODO: make it work in case of offset != Off(0,0,0) ! return side_of_cell(p,Offset(),c,lt,i,j); } //@} @@ -1524,6 +1523,8 @@ try_next_cell: * * lt has a meaning only when ON_BOUNDED_SIDE or ON_BOUNDARY */ +// TODO: currently off is not used. It could probably be optimized +// using off. template < class GT, class TDS > inline Bounded_side Periodic_3_triangulation_3::side_of_cell( const Point & q, const Offset &off, Cell_handle c, @@ -2209,7 +2210,6 @@ template < class GT, class TDS > bool Periodic_3_triangulation_3:: is_valid(bool verbose, int level) const { - //todo: don't use assertions, add verbose output bool error = false; for (Cell_iterator cit = cells_begin(); cit != cells_end(); ++cit) {