From fcf93adf1e766fbc69b05e598ee0aeb9b36b1c70 Mon Sep 17 00:00:00 2001 From: Emmanuel Gil Peyrot Date: Mon, 18 Jun 2018 21:48:07 +0200 Subject: =?UTF-8?q?Run=20clang-tidy=E2=80=99s=20modernize-use-emplace=20pa?= =?UTF-8?q?ss.?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reduces the boilerplate required to add a new element to a container. --- src/helper/geom.cpp | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) (limited to 'src/helper/geom.cpp') diff --git a/src/helper/geom.cpp b/src/helper/geom.cpp index e83fb42f2..253188f1d 100644 --- a/src/helper/geom.cpp +++ b/src/helper/geom.cpp @@ -707,7 +707,7 @@ recursive_bezier4(const double x1, const double y1, { if(d2 < m_distance_tolerance_square) { - m_points.push_back(Geom::Point(x2, y2)); + m_points.emplace_back(x2, y2); return; } } @@ -715,7 +715,7 @@ recursive_bezier4(const double x1, const double y1, { if(d3 < m_distance_tolerance_square) { - m_points.push_back(Geom::Point(x3, y3)); + m_points.emplace_back(x3, y3); return; } } @@ -728,7 +728,7 @@ recursive_bezier4(const double x1, const double y1, { if(m_angle_tolerance < curve_angle_tolerance_epsilon) { - m_points.push_back(Geom::Point(x23, y23)); + m_points.emplace_back(x23, y23); return; } @@ -739,8 +739,8 @@ recursive_bezier4(const double x1, const double y1, if(da1 < m_angle_tolerance) { - m_points.push_back(Geom::Point(x2, y2)); - m_points.push_back(Geom::Point(x3, y3)); + m_points.emplace_back(x2, y2); + m_points.emplace_back(x3, y3); return; } @@ -748,7 +748,7 @@ recursive_bezier4(const double x1, const double y1, { if(da1 > m_cusp_limit) { - m_points.push_back(Geom::Point(x3, y3)); + m_points.emplace_back(x3, y3); return; } } @@ -762,7 +762,7 @@ recursive_bezier4(const double x1, const double y1, { if(m_angle_tolerance < curve_angle_tolerance_epsilon) { - m_points.push_back(Geom::Point(x23, y23)); + m_points.emplace_back(x23, y23); return; } @@ -773,8 +773,8 @@ recursive_bezier4(const double x1, const double y1, if(da1 < m_angle_tolerance) { - m_points.push_back(Geom::Point(x2, y2)); - m_points.push_back(Geom::Point(x3, y3)); + m_points.emplace_back(x2, y2); + m_points.emplace_back(x3, y3); return; } @@ -782,7 +782,7 @@ recursive_bezier4(const double x1, const double y1, { if(da1 > m_cusp_limit) { - m_points.push_back(Geom::Point(x2, y2)); + m_points.emplace_back(x2, y2); return; } } @@ -799,7 +799,7 @@ recursive_bezier4(const double x1, const double y1, //---------------------- if(m_angle_tolerance < curve_angle_tolerance_epsilon) { - m_points.push_back(Geom::Point(x23, y23)); + m_points.emplace_back(x23, y23); return; } @@ -815,7 +815,7 @@ recursive_bezier4(const double x1, const double y1, { // Finally we can stop the recursion //---------------------- - m_points.push_back(Geom::Point(x23, y23)); + m_points.emplace_back(x23, y23); return; } @@ -823,13 +823,13 @@ recursive_bezier4(const double x1, const double y1, { if(da1 > m_cusp_limit) { - m_points.push_back(Geom::Point(x2, y2)); + m_points.emplace_back(x2, y2); return; } if(da2 > m_cusp_limit) { - m_points.push_back(Geom::Point(x3, y3)); + m_points.emplace_back(x3, y3); return; } } -- cgit v1.2.3