From 6d698871f98c3945b338944d362a01249ab0e648 Mon Sep 17 00:00:00 2001 From: Andreas Fabri Date: Wed, 18 Sep 2024 16:18:13 +0100 Subject: [PATCH] cleanup --- .../include/CGAL/Frechet_distance/internal/curve.h | 9 +++++---- NewKernel_d/include/CGAL/NewKernel_d/Cartesian_base.h | 1 - 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/Frechet_distance/include/CGAL/Frechet_distance/internal/curve.h b/Frechet_distance/include/CGAL/Frechet_distance/internal/curve.h index f0127be78aa..3afdbbd9895 100644 --- a/Frechet_distance/include/CGAL/Frechet_distance/internal/curve.h +++ b/Frechet_distance/include/CGAL/Frechet_distance/internal/curve.h @@ -235,7 +235,8 @@ public: == Epeck_with_sqrt */ template - Curve(const PointRange& point_range) : prefix_length(point_range.size()) + Curve(const PointRange& point_range) + : prefix_length(point_range.size()) { if constexpr ( ! std::is_floating_point::type::value) { input.reserve(point_range.size()); @@ -376,11 +377,11 @@ std::ostream& operator<<(std::ostream& out, const Curve& curve); template void Curve::push_back(Point const& point) { - if (prefix_length.size()) { + if (prefix_length.empty()) { + prefix_length.push_back(0); + } else { auto segment_distance = distance(points.back(), point); prefix_length.push_back(prefix_length.back() + segment_distance); - } else { - prefix_length.push_back(0); } if (points.empty()){ extreme_points = point.bbox(); diff --git a/NewKernel_d/include/CGAL/NewKernel_d/Cartesian_base.h b/NewKernel_d/include/CGAL/NewKernel_d/Cartesian_base.h index 0b785dfe60b..e82ffbbf6f6 100644 --- a/NewKernel_d/include/CGAL/NewKernel_d/Cartesian_base.h +++ b/NewKernel_d/include/CGAL/NewKernel_d/Cartesian_base.h @@ -22,7 +22,6 @@ namespace CGAL { template < typename FT_, typename Dim_, typename Derived_=Default> struct Cartesian_base_d : public CGAL_BASE { - typedef FT_ FT; constexpr Cartesian_base_d(){} constexpr Cartesian_base_d(int d):CGAL_BASE(d){} };