From df4fe89f5c3640c32fe4c89efffe107106f602e8 Mon Sep 17 00:00:00 2001 From: Vicky Vergara Date: Fri, 24 Mar 2023 12:43:58 -0600 Subject: [PATCH] Fix clang warnings (#2495) * [clang] removing warnings * [doc] updating the page_history information --- doc/_static/page_history.js | 35 +++++++++++++++++----------- include/cpp_common/basePath_SSEC.hpp | 1 - include/cpp_common/identifier.h | 1 - include/cpp_common/identifiers.hpp | 1 - include/cpp_common/xy_vertex.h | 1 - include/vrp/order.h | 2 -- include/vrp/pd_orders.h | 1 - include/vrp/tw_node.h | 1 - include/vrp/vehicle.h | 1 - include/vrp/vehicle_node.h | 1 - include/vrp/vehicle_pickDeliver.h | 3 --- 11 files changed, 22 insertions(+), 26 deletions(-) diff --git a/doc/_static/page_history.js b/doc/_static/page_history.js index 12835860c7b..d33e22ab071 100644 --- a/doc/_static/page_history.js +++ b/doc/_static/page_history.js @@ -38,14 +38,18 @@ var newpages = [ 'pgr_bdAstarCost', 'pgr_articulationPoints', 'components-family', 'bdDijkstra-family', 'bdAstar-family', ] }, {v: '2.4', pages: [ 'pgr_maxFlow', 'pgr_aStarCostMatrix', 'pgr_aStarCost', 'KSP-category', - 'drivingDistance-category', 'costMatrix-category', 'cost-category', 'aStar-family' ] }, + 'cost-category', 'aStar-family' ] }, ]; var filesArr = [ + new createInfo('TSP-family', '2.3', [ { v: '2.3', n: 'tsp', d: 'tsp'}, ]), + new createInfo('costMatrix-category', '2.3', [ + { v: '2.3', n: 'CostMatrix', d: 'costMatrix'}, + ]), new createInfo('pgr_withPointsCostMatrix', '2.3', [ { v: '2.3', n: 'pgr_withPointsCostMatrix', d: 'costMatrix'}, ]), @@ -77,6 +81,9 @@ var filesArr = [ new createInfo('pgr_dijkstraCostMatrix', '2.3', [ { v: '2.3', n: 'pgr_dijkstraCostMatrix', d: 'costMatrix'}, ]), + new createInfo('contraction-family', '2.3', [ + { v: '2.3', n: 'contraction', d: 'contraction'}, + ]), new createInfo('pgr_contraction', '2.3', [ { v: '2.3', n: 'pgr_contractGraph', d: 'contraction'}, { v: '2.4', n: 'pgr_contractGraph', d: ''}, @@ -86,6 +93,12 @@ var filesArr = [ new createInfo('flow-family', '2.3', [ { v: '2.3', n: 'maxFlow', d: 'max_flow'}, ]), + + new createInfo('drivingDistance-category', '2.2', [ + { v: '2.2', n: 'drivingDistance', d: 'driving_distance'}, + { v: '2.3', n: 'drivingDistance', d: 'driving_distance'}, + ]), + new createInfo('withPoints-family', '2.2', [ { v: '2.2', n: 'withPoints', d: 'withPoints'}, { v: '2.3', n: 'withPoints', d: 'withPoints'}, @@ -102,8 +115,8 @@ var filesArr = [ { v: '2.6', n: 'proposed', d: ''}, ]), new createInfo('proposed', '2.2', [ - { v: '2.2', n: 'src/proposed', d: ''}, - { v: '2.3', n: 'src/proposed', d: ''}, + { v: '2.2', n: 'src/proposedNext', d: ''}, + { v: '2.3', n: 'src/proposedNext', d: ''}, ]), new createInfo('pgr_withPointsKSP', '2.2', [ { v: '2.2', n: 'pgr_withPointsKSP', d: 'withPoints'}, @@ -137,10 +150,6 @@ var filesArr = [ { v: '2.2', n: 'dijkstra', d: 'dijkstra'}, { v: '2.3', n: 'dijkstra', d: 'dijkstra'}, ]), - new createInfo('contraction-family', '2.2', [ - { v: '2.2', n: 'contraction', d: 'contraction'}, - { v: '2.3', n: 'contraction', d: 'contraction'}, - ]), new createInfo('allpairs-family', '2.2', [ { v: '2.2', n: 'allpairs', d: 'allpairs'}, { v: '2.3', n: 'allpairs', d: 'allpairs'}, @@ -190,13 +199,13 @@ var filesArr = [ ]), new createInfo('pgr_KSP', '2.0', [ { v: '2.0', n: 'index', d: 'ksp'}, - { v: '2.1', n: 'index', d: 'ksp'}, + { v: '2.1', n: 'ksp_v3', d: 'ksp'}, { v: '2.2', n: 'pgr_ksp', d: 'ksp'}, { v: '2.3', n: 'pgr_ksp', d: 'ksp'}, ]), new createInfo('pgr_dijkstra', '2.0', [ { v: '2.0', n: 'index', d: 'dijkstra'}, - { v: '2.1', n: 'index', d: 'dijkstra'}, + { v: '2.1', n: 'dijkstra_v3', d: 'dijkstra'}, { v: '2.2', n: 'pgr_dijkstra', d: 'dijkstra'}, { v: '2.3', n: 'pgr_dijkstra', d: 'dijkstra'} ]), @@ -267,10 +276,10 @@ var filesArr = [ { v: '2.3', n: 'pgr_analyzeGraph', d: 'topology'}, ]), new createInfo('pgRouting-concepts', '2.0', [ - { v: '2.0', n: 'doc/src/tutorial/tutorial', d: ''}, - { v: '2.1', n: 'doc/src/tutorial/tutorial', d: ''}, - { v: '2.2', n: 'doc/src/tutorial/tutorial', d: ''}, - { v: '2.3', n: 'doc/src/tutorial/tutorial', d: ''}, + { v: '2.0', n: 'doc/src/tutorial/index', d: ''}, + { v: '2.1', n: 'doc/src/tutorial/index', d: ''}, + { v: '2.2', n: 'doc/src/tutorial/index', d: ''}, + { v: '2.3', n: 'doc/src/tutorial/index', d: ''}, ]), new createInfo('support', '2.0', [ { v: '2.0', n: 'doc/src/introduction/support', d: ''}, diff --git a/include/cpp_common/basePath_SSEC.hpp b/include/cpp_common/basePath_SSEC.hpp index ce039d9e091..e45c17e4e43 100644 --- a/include/cpp_common/basePath_SSEC.hpp +++ b/include/cpp_common/basePath_SSEC.hpp @@ -59,7 +59,6 @@ class Path { Path(int64_t s_id, int64_t e_id) : m_start_id(s_id), m_end_id(e_id), m_tot_cost(0) {} - Path(const Path&) = default; int64_t start_id() const {return m_start_id;} void start_id(int64_t value) {m_start_id = value;} diff --git a/include/cpp_common/identifier.h b/include/cpp_common/identifier.h index 61f56d5e303..bb32f9dc84b 100644 --- a/include/cpp_common/identifier.h +++ b/include/cpp_common/identifier.h @@ -39,7 +39,6 @@ namespace pgrouting { class Identifier { public: Identifier() = default; - Identifier(const Identifier &v) = default; Identifier(const size_t _idx, const int64_t _id); int64_t id() const; diff --git a/include/cpp_common/identifiers.hpp b/include/cpp_common/identifiers.hpp index c0f7b23101a..43999dff6a4 100644 --- a/include/cpp_common/identifiers.hpp +++ b/include/cpp_common/identifiers.hpp @@ -55,7 +55,6 @@ class Identifiers { //! @name constructors //@{ Identifiers() = default; - Identifiers(const Identifiers&) = default; Identifiers(const std::set& data) { m_ids = data; } diff --git a/include/cpp_common/xy_vertex.h b/include/cpp_common/xy_vertex.h index 525a812b6fc..a934979283d 100644 --- a/include/cpp_common/xy_vertex.h +++ b/include/cpp_common/xy_vertex.h @@ -40,7 +40,6 @@ namespace pgrouting { class XY_vertex { public: XY_vertex() = default; - XY_vertex(const XY_vertex &) = default; XY_vertex(const int64_t _id, double _x, double _y) : id(_id), point(_x, _y) { } diff --git a/include/vrp/order.h b/include/vrp/order.h index 5944cb855c0..789bd651599 100644 --- a/include/vrp/order.h +++ b/include/vrp/order.h @@ -47,8 +47,6 @@ class Order : public Identifier { Order(size_t p_id, int64_t p_idx, const Vehicle_node &p_pickup, const Vehicle_node &p_deliver); - - Order(const Order &) = default; /*!@}*/ diff --git a/include/vrp/pd_orders.h b/include/vrp/pd_orders.h index e787c5f21a2..42978daafa6 100644 --- a/include/vrp/pd_orders.h +++ b/include/vrp/pd_orders.h @@ -60,7 +60,6 @@ class PD_Orders { * @{ */ PD_Orders() = default; - PD_Orders(const PD_Orders&) = default; explicit PD_Orders(const std::vector&); /*!@}*/ diff --git a/include/vrp/tw_node.h b/include/vrp/tw_node.h index 842b1aeb427..e2f1f28217e 100644 --- a/include/vrp/tw_node.h +++ b/include/vrp/tw_node.h @@ -210,7 +210,6 @@ class Tw_node : public Dnode { Tw_node() = default; - Tw_node(const Tw_node &) = default; Tw_node( size_t id, Orders_t data, diff --git a/include/vrp/vehicle.h b/include/vrp/vehicle.h index f18ae60f54c..7e761a5c0f7 100644 --- a/include/vrp/vehicle.h +++ b/include/vrp/vehicle.h @@ -91,7 +91,6 @@ class Vehicle : public Identifier { std::vector get_postgres_result(int vid) const; - Vehicle(const Vehicle &) = default; Vehicle( size_t idx, int64_t id, diff --git a/include/vrp/vehicle_node.h b/include/vrp/vehicle_node.h index f38a1494d6a..bb7a4c03eda 100644 --- a/include/vrp/vehicle_node.h +++ b/include/vrp/vehicle_node.h @@ -145,7 +145,6 @@ class Vehicle_node: public Tw_node { /*! \brief Construct from parameters */ Vehicle_node() = delete; - Vehicle_node(const Vehicle_node &) = default; explicit Vehicle_node(const Tw_node &node); private: diff --git a/include/vrp/vehicle_pickDeliver.h b/include/vrp/vehicle_pickDeliver.h index 25f162605ff..f9e918e91c9 100644 --- a/include/vrp/vehicle_pickDeliver.h +++ b/include/vrp/vehicle_pickDeliver.h @@ -67,9 +67,6 @@ class Vehicle_pickDeliver : public Vehicle { double p_speed, double factor); - Vehicle_pickDeliver(const Vehicle_pickDeliver &) = default; - - void set_compatibles(const PD_Orders &orders); bool is_order_feasable(const Order &order) const; Identifiers feasable_orders() const {return m_feasable_orders;}