From 57c240980f343e6e933b5525df184f0971f16d9f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Mael=20Rouxel-Labb=C3=A9?= Date: Wed, 10 Nov 2021 15:40:18 +0100 Subject: [PATCH] Use some const& to avoid copies --- .../include/CGAL/Cartesian_converter.h | 8 ++++---- .../include/CGAL/Homogeneous_converter.h | 17 +++++++++++++++-- 2 files changed, 19 insertions(+), 6 deletions(-) diff --git a/Cartesian_kernel/include/CGAL/Cartesian_converter.h b/Cartesian_kernel/include/CGAL/Cartesian_converter.h index 65a68c612fb..d1a22c57160 100644 --- a/Cartesian_kernel/include/CGAL/Cartesian_converter.h +++ b/Cartesian_kernel/include/CGAL/Cartesian_converter.h @@ -95,25 +95,25 @@ public: using Base::operator(); - Origin + const Origin& operator()(const Origin& o) const { return o; } - Null_vector + const Null_vector& operator()(const Null_vector& n) const { return n; } - Bbox_2 + const Bbox_2& operator()(const Bbox_2& b) const { return b; } - Bbox_3 + const Bbox_3& operator()(const Bbox_3& b) const { return b; diff --git a/Homogeneous_kernel/include/CGAL/Homogeneous_converter.h b/Homogeneous_kernel/include/CGAL/Homogeneous_converter.h index 601d9ac504f..be33758244f 100644 --- a/Homogeneous_kernel/include/CGAL/Homogeneous_converter.h +++ b/Homogeneous_kernel/include/CGAL/Homogeneous_converter.h @@ -28,6 +28,7 @@ #include #include #include +#include namespace CGAL { @@ -47,13 +48,25 @@ public: using Base::operator(); - Bbox_2 + const Origin& + operator()(const Origin& o) const + { + return o; + } + + const Null_vector& + operator()(const Null_vector& n) const + { + return n; + } + + const Bbox_2& operator()(const Bbox_2& b) { return b; } - Bbox_3 + const Bbox_3& operator()(const Bbox_3& b) { return b;