From f34d1e3545ebb5701c7642a369aa70aa457b72d5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=A9bastien=20Loriot?= Date: Fri, 8 Apr 2022 14:55:51 +0200 Subject: [PATCH] do not use default constructed kernel --- .../include/CGAL/Hilbert_sort_middle_2.h | 22 +++++++++---------- 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/Spatial_sorting/include/CGAL/Hilbert_sort_middle_2.h b/Spatial_sorting/include/CGAL/Hilbert_sort_middle_2.h index 1d0d6b7792f..6160a7bea58 100644 --- a/Spatial_sorting/include/CGAL/Hilbert_sort_middle_2.h +++ b/Spatial_sorting/include/CGAL/Hilbert_sort_middle_2.h @@ -123,22 +123,20 @@ public: void operator() (RandomAccessIterator begin, RandomAccessIterator end) const { //Bbox_2 box=bbox_2(begin, end); BUG: WE NEED TO FIX THIS - - K k; - double xmin=to_double(k.compute_x_2_object()(*begin)), - ymin=to_double(k.compute_y_2_object()(*begin)), + double xmin=to_double(_k.compute_x_2_object()(*begin)), + ymin=to_double(_k.compute_y_2_object()(*begin)), xmax=xmin, ymax=ymin; for(RandomAccessIterator it=begin+1; it xmax) - xmax = to_double(k.compute_x_2_object()(*it)); - if ( to_double(k.compute_y_2_object()(*it)) > ymax) - ymax = to_double(k.compute_y_2_object()(*it)); + if ( to_double(_k.compute_x_2_object()(*it)) < xmin) + xmin = to_double(_k.compute_x_2_object()(*it)); + if ( to_double(_k.compute_y_2_object()(*it)) < ymin) + ymin = to_double(_k.compute_y_2_object()(*it)); + if ( to_double(_k.compute_x_2_object()(*it)) > xmax) + xmax = to_double(_k.compute_x_2_object()(*it)); + if ( to_double(_k.compute_y_2_object()(*it)) > ymax) + ymax = to_double(_k.compute_y_2_object()(*it)); } sort <0, false, false> (begin, end, xmin, ymin, xmax, ymax);