From 405570630aef19c77468f4a922f725d76fe58377 Mon Sep 17 00:00:00 2001
From: Siarhei Fedartsou <siarhei.fedartsou@gmail.com>
Date: Wed, 29 Jun 2022 21:51:48 +0200
Subject: [PATCH] Enable more clang-tidy checks

---
 .clang-tidy                                   | 11 ++++----
 CHANGELOG.md                                  |  1 +
 .../contractor/contract_excludable_graph.hpp  |  2 +-
 include/engine/datafacade_factory.hpp         |  2 +-
 include/extractor/raster_source.hpp           |  1 -
 include/extractor/restriction.hpp             |  2 +-
 include/guidance/turn_bearing.hpp             |  5 +---
 include/guidance/turn_lane_handler.hpp        | 12 ---------
 include/server/server.hpp                     |  2 +-
 include/storage/io_config.hpp                 |  7 ++---
 include/storage/shared_datatype.hpp           |  2 +-
 include/util/exception.hpp                    | 27 ++++++++++---------
 include/util/exception_utils.hpp              |  2 +-
 include/util/guidance/name_announcements.hpp  |  4 +--
 include/util/range_table.hpp                  |  6 +++++
 src/customize/customizer.cpp                  |  3 ++-
 src/engine/api/json_factory.cpp               |  3 +--
 src/extractor/intersection/intersection.cpp   |  2 --
 src/guidance/driveway_handler.cpp             |  3 ---
 src/guidance/turn_handler.cpp                 |  2 +-
 src/partitioner/dinic_max_flow.cpp            |  2 +-
 src/updater/updater.cpp                       |  2 +-
 src/util/conditional_restrictions.cpp         |  1 -
 23 files changed, 45 insertions(+), 59 deletions(-)

diff --git a/.clang-tidy b/.clang-tidy
index 9581f45b814..d1e92a31505 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -9,13 +9,10 @@ Checks: >
     -bugprone-implicit-widening-of-multiplication-result,
     -bugprone-integer-division,
     -bugprone-reserved-identifier,
-    -bugprone-macro-parentheses,
     -bugprone-unhandled-self-assignment,
-    -bugprone-suspicious-missing-comma,
     -bugprone-forward-declaration-namespace,
     -bugprone-sizeof-expression,
     -clang-analyzer-*,
-    -clang-diagnostic-unused-local-typedef,
     -clang-diagnostic-deprecated-declarations,
     google-*,
     -google-build-explicit-make-pair,
@@ -29,7 +26,6 @@ Checks: >
     -google-readability-todo,
     -google-runtime-int,
     -google-build-namespaces,
-    -google-global-names-in-headers,
     -google-runtime-references,
     -google-readability-function-size,
     llvm-*,
@@ -42,14 +38,17 @@ Checks: >
     misc-*,
     -misc-argument-comment,
     -misc-non-private-member-variables-in-classes,
-    -misc-unused-using-decls,
     -misc-unconventional-assign-operator,
     -misc-redundant-expression,
     -misc-no-recursion,
     -misc-misplaced-const,
     -misc-definitions-in-headers,
-    -misc-unused-alias-decls,
     -misc-unused-parameters,
+    performance-*,
+    -performance-noexcept-move-constructor,
+    -performance-move-const-arg,
+    -performance-no-int-to-ptr,
+    -performance-unnecessary-value-param,
     readability-*,
     -readability-avoid-const-params-in-decls,
     -readability-braces-around-statements,
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 5ce4f393dd6..dbfdb844c17 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -3,6 +3,7 @@
     - API:
       - FIXED: Fix inefficient osrm-routed connection handling [#6113](https://github.com/Project-OSRM/osrm-backend/pull/6113)
     - Build:
+      - CHANGED: Enable more clang-tidy checks. [#6262](https://github.com/Project-OSRM/osrm-backend/pull/6270)
       - CHANGED: Configure clang-tidy job on CI. [#6261](https://github.com/Project-OSRM/osrm-backend/pull/6261)
       - CHANGED: Use Github Actions for building container images [#6138](https://github.com/Project-OSRM/osrm-backend/pull/6138)
       - CHANGED: Upgrade Boost dependency to 1.70 [#6113](https://github.com/Project-OSRM/osrm-backend/pull/6113)
diff --git a/include/contractor/contract_excludable_graph.hpp b/include/contractor/contract_excludable_graph.hpp
index 74ea8373b71..9d0cc697924 100644
--- a/include/contractor/contract_excludable_graph.hpp
+++ b/include/contractor/contract_excludable_graph.hpp
@@ -18,7 +18,7 @@ inline auto contractFullGraph(ContractorGraph contractor_graph,
                               std::vector<EdgeWeight> node_weights)
 {
     auto num_nodes = contractor_graph.GetNumberOfNodes();
-    contractGraph(contractor_graph, node_weights);
+    contractGraph(contractor_graph, std::move(node_weights));
 
     auto edges = toEdges<QueryEdge>(std::move(contractor_graph));
     std::vector<bool> edge_filter(edges.size(), true);
diff --git a/include/engine/datafacade_factory.hpp b/include/engine/datafacade_factory.hpp
index 853716be950..e01c70f7275 100644
--- a/include/engine/datafacade_factory.hpp
+++ b/include/engine/datafacade_factory.hpp
@@ -65,7 +65,7 @@ template <template <typename A> class FacadeT, typename AlgorithmT> class DataFa
 
         for (const auto &exclude_prefix : exclude_prefixes)
         {
-            auto index_begin = exclude_prefix.find_last_of("/");
+            auto index_begin = exclude_prefix.find_last_of('/');
             BOOST_ASSERT_MSG(index_begin != std::string::npos,
                              "The exclude prefix needs to be a valid data path.");
             std::size_t index =
diff --git a/include/extractor/raster_source.hpp b/include/extractor/raster_source.hpp
index 45e495f221a..e910892520c 100644
--- a/include/extractor/raster_source.hpp
+++ b/include/extractor/raster_source.hpp
@@ -18,7 +18,6 @@
 #include <iterator>
 #include <string>
 #include <unordered_map>
-using namespace std;
 
 namespace osrm
 {
diff --git a/include/extractor/restriction.hpp b/include/extractor/restriction.hpp
index 8d7f64657ff..7f17eaeaee6 100644
--- a/include/extractor/restriction.hpp
+++ b/include/extractor/restriction.hpp
@@ -186,7 +186,7 @@ struct TurnRestriction
     }
 
     // construction for WayRestrictions
-    explicit TurnRestriction(WayRestriction way_restriction, bool is_only = false)
+    explicit TurnRestriction(const WayRestriction &way_restriction, bool is_only = false)
         : node_or_way(way_restriction), is_only(is_only)
     {
     }
diff --git a/include/guidance/turn_bearing.hpp b/include/guidance/turn_bearing.hpp
index ad6e3ac1afb..12ba8567c7b 100644
--- a/include/guidance/turn_bearing.hpp
+++ b/include/guidance/turn_bearing.hpp
@@ -9,15 +9,12 @@ namespace osrm
 {
 namespace guidance
 {
-namespace
-{
-const double bearing_scale = 360.0 / 256.0;
-}
 
 #pragma pack(push, 1)
 class TurnBearing
 {
   public:
+    static constexpr double bearing_scale = 360.0 / 256.0;
     // discretizes a bearing into distinct units of 1.4 degrees
     TurnBearing(const double value = 0) : bearing(value / bearing_scale)
     {
diff --git a/include/guidance/turn_lane_handler.hpp b/include/guidance/turn_lane_handler.hpp
index d0947d98026..d0344ad5c1a 100644
--- a/include/guidance/turn_lane_handler.hpp
+++ b/include/guidance/turn_lane_handler.hpp
@@ -52,15 +52,6 @@ typedef enum TurnLaneScenario
     NUM_SCENARIOS
 } TurnLaneScenario;
 
-const constexpr char *scenario_names[] = {"Simple",
-                                          "Partition Local",
-                                          "Simple Previous",
-                                          "Partition Previous",
-                                          "Sliproad",
-                                          "Merge",
-                                          "None",
-                                          "Invalid",
-                                          "Unknown"};
 } // namespace
 
 class TurnLaneHandler
@@ -150,9 +141,6 @@ class TurnLaneHandler
                          LaneDataVector &lane_data) const;
 };
 
-static_assert(sizeof(scenario_names) / sizeof(*scenario_names) == TurnLaneScenario::NUM_SCENARIOS,
-              "Number of scenarios needs to match the number of scenario names.");
-
 } // namespace lanes
 } // namespace guidance
 } // namespace osrm
diff --git a/include/server/server.hpp b/include/server/server.hpp
index 1cc3f11d24f..280b486683d 100644
--- a/include/server/server.hpp
+++ b/include/server/server.hpp
@@ -77,7 +77,7 @@ class Server
                 boost::bind(&boost::asio::io_context::run, &io_context));
             threads.push_back(thread);
         }
-        for (auto thread : threads)
+        for (const auto &thread : threads)
         {
             thread->join();
         }
diff --git a/include/storage/io_config.hpp b/include/storage/io_config.hpp
index a76ad956266..577291ee110 100644
--- a/include/storage/io_config.hpp
+++ b/include/storage/io_config.hpp
@@ -18,8 +18,9 @@ struct IOConfig
     IOConfig(std::vector<boost::filesystem::path> required_input_files_,
              std::vector<boost::filesystem::path> optional_input_files_,
              std::vector<boost::filesystem::path> output_files_)
-        : required_input_files(required_input_files_), optional_input_files(optional_input_files_),
-          output_files(output_files_)
+        : required_input_files(std::move(required_input_files_)),
+          optional_input_files(std::move(optional_input_files_)),
+          output_files(std::move(output_files_))
     {
     }
 
@@ -47,7 +48,7 @@ struct IOConfig
 
         std::array<std::string, 6> known_extensions{
             {".osm.bz2", ".osm.pbf", ".osm.xml", ".pbf", ".osm", ".osrm"}};
-        for (auto ext : known_extensions)
+        for (const auto &ext : known_extensions)
         {
             const auto pos = path.find(ext);
             if (pos != std::string::npos)
diff --git a/include/storage/shared_datatype.hpp b/include/storage/shared_datatype.hpp
index 5677c8af0a6..bc4bf4efc68 100644
--- a/include/storage/shared_datatype.hpp
+++ b/include/storage/shared_datatype.hpp
@@ -36,7 +36,7 @@ inline std::string trimName(const std::string &name_prefix, const std::string &n
     // list directory and
     if (!name_prefix.empty() && name_prefix.back() == '/')
     {
-        auto directory_position = name.find_first_of("/", name_prefix.length());
+        auto directory_position = name.find_first_of('/', name_prefix.length());
         // this is a "file" in the directory of name_prefix
         if (directory_position == std::string::npos)
         {
diff --git a/include/util/exception.hpp b/include/util/exception.hpp
index 871c4ba27cd..7a4b89ca4cf 100644
--- a/include/util/exception.hpp
+++ b/include/util/exception.hpp
@@ -47,7 +47,7 @@ class exception : public std::exception
   public:
     explicit exception(const char *message) : message(message) {}
     explicit exception(std::string message) : message(std::move(message)) {}
-    explicit exception(boost::format message) : message(message.str()) {}
+    explicit exception(const boost::format &message) : message(message.str()) {}
     const char *what() const noexcept override { return message.c_str(); }
 
   private:
@@ -65,18 +65,19 @@ class exception : public std::exception
  */
 
 constexpr const std::array<const char *, 11> ErrorDescriptions = {{
-    "",                                                      // Dummy - ErrorCode values start at 2
-    "",                                                      // Dummy - ErrorCode values start at 2
-    "Fingerprint did not match the expected value",          // InvalidFingerprint
-    "File is incompatible with this version of OSRM",        // IncompatibleFileVersion
-    "Problem opening file",                                  // FileOpenError
-    "Problem reading from file",                             // FileReadError
-    "Problem writing to file",                               // FileWriteError
-    "I/O error occurred",                                    // FileIOError
-    "Unexpected end of file",                                // UnexpectedEndOfFile
-    "The dataset you are trying to load is not "             // IncompatibleDataset
-    "compatible with the routing algorithm you want to use." // ...continued...
-    "Incompatible algorithm"                                 // IncompatibleAlgorithm
+    "",                                               // Dummy - ErrorCode values start at 2
+    "",                                               // Dummy - ErrorCode values start at 2
+    "Fingerprint did not match the expected value",   // InvalidFingerprint
+    "File is incompatible with this version of OSRM", // IncompatibleFileVersion
+    "Problem opening file",                           // FileOpenError
+    "Problem reading from file",                      // FileReadError
+    "Problem writing to file",                        // FileWriteError
+    "I/O error occurred",                             // FileIOError
+    "Unexpected end of file",                         // UnexpectedEndOfFile
+    // NOLINTNEXTLINE(bugprone-suspicious-missing-comma)
+    "The dataset you are trying to load is not "              // IncompatibleDataset
+    "compatible with the routing algorithm you want to use.", // ...continued...
+    "Incompatible algorithm"                                  // IncompatibleAlgorithm
 }};
 
 #ifndef NDEBUG
diff --git a/include/util/exception_utils.hpp b/include/util/exception_utils.hpp
index e933fcb4e93..58c128af0d5 100644
--- a/include/util/exception_utils.hpp
+++ b/include/util/exception_utils.hpp
@@ -10,6 +10,6 @@
 #define OSRM_SOURCE_FILE_ PROJECT_RELATIVE_PATH_(__FILE__)
 
 // This is the macro to use
-#define SOURCE_REF OSRM_SOURCE_FILE_ + ":" + std::to_string(__LINE__)
+#define SOURCE_REF (OSRM_SOURCE_FILE_ + ":" + std::to_string(__LINE__))
 
 #endif // SOURCE_MACROS_HPP
diff --git a/include/util/guidance/name_announcements.hpp b/include/util/guidance/name_announcements.hpp
index 2a3953eadaa..fa8f9db430f 100644
--- a/include/util/guidance/name_announcements.hpp
+++ b/include/util/guidance/name_announcements.hpp
@@ -71,12 +71,12 @@ template <typename StringView> inline auto decompose(const StringView &lhs, cons
         // we compare suffixes based on this value, it might break UTF chars, but as long as we are
         // consistent in handling, we do not create bad results
         std::string str = boost::to_lower_copy(view.to_string());
-        auto front = str.find_first_not_of(" ");
+        auto front = str.find_first_not_of(' ');
 
         if (front == std::string::npos)
             return str;
 
-        auto back = str.find_last_not_of(" ");
+        auto back = str.find_last_not_of(' ');
         return str.substr(front, back - front + 1);
     };
 
diff --git a/include/util/range_table.hpp b/include/util/range_table.hpp
index ef374589428..aa1b89f1c24 100644
--- a/include/util/range_table.hpp
+++ b/include/util/range_table.hpp
@@ -83,19 +83,25 @@ template <unsigned BLOCK_SIZE, storage::Ownership Ownership> class RangeTable
         unsigned block_idx = 0;
         unsigned block_counter = 0;
         BlockT block;
+#ifndef BOOST_ASSERT_IS_VOID
         unsigned block_sum = 0;
+#endif
         for (const unsigned l : lengths)
         {
             // first entry of a block: encode absolute offset
             if (block_idx == 0)
             {
                 block_offsets.push_back(lengths_prefix_sum);
+#ifndef BOOST_ASSERT_IS_VOID
                 block_sum = 0;
+#endif
             }
             else
             {
                 block[block_idx - 1] = last_length;
+#ifndef BOOST_ASSERT_IS_VOID
                 block_sum += last_length;
+#endif
             }
 
             BOOST_ASSERT((block_idx == 0 && block_offsets[block_counter] == lengths_prefix_sum) ||
diff --git a/src/customize/customizer.cpp b/src/customize/customizer.cpp
index 27b37673376..da95f996797 100644
--- a/src/customize/customizer.cpp
+++ b/src/customize/customizer.cpp
@@ -108,8 +108,9 @@ std::vector<CellMetric> customizeFilteredMetrics(const partitioner::MultiLevelEd
                                                  const std::vector<std::vector<bool>> &node_filters)
 {
     std::vector<CellMetric> metrics;
+    metrics.reserve(node_filters.size());
 
-    for (auto filter : node_filters)
+    for (const auto &filter : node_filters)
     {
         auto metric = storage.MakeMetric();
         customizer.Customize(graph, storage, filter, metric);
diff --git a/src/engine/api/json_factory.cpp b/src/engine/api/json_factory.cpp
index 8df45c94c85..9ab7e4734f4 100644
--- a/src/engine/api/json_factory.cpp
+++ b/src/engine/api/json_factory.cpp
@@ -19,7 +19,6 @@
 #include <utility>
 #include <vector>
 
-namespace TurnType = osrm::guidance::TurnType;
 using TurnInstruction = osrm::guidance::TurnInstruction;
 
 namespace osrm
@@ -242,7 +241,7 @@ util::json::Object makeWaypoint(const util::Coordinate &location,
                                 std::string name,
                                 const Hint &hint)
 {
-    auto waypoint = makeWaypoint(location, distance, name);
+    auto waypoint = makeWaypoint(location, distance, std::move(name));
     waypoint.values["hint"] = hint.ToBase64();
     return waypoint;
 }
diff --git a/src/extractor/intersection/intersection.cpp b/src/extractor/intersection/intersection.cpp
index 8ab635fbda8..530252a4c61 100644
--- a/src/extractor/intersection/intersection.cpp
+++ b/src/extractor/intersection/intersection.cpp
@@ -5,8 +5,6 @@
 
 #include <boost/range/adaptors.hpp>
 
-using osrm::util::angularDeviation;
-
 namespace osrm
 {
 namespace extractor
diff --git a/src/guidance/driveway_handler.cpp b/src/guidance/driveway_handler.cpp
index ae779a5df4f..4c7eee14aab 100644
--- a/src/guidance/driveway_handler.cpp
+++ b/src/guidance/driveway_handler.cpp
@@ -2,9 +2,6 @@
 
 #include "util/assert.hpp"
 
-using osrm::guidance::getTurnDirection;
-using osrm::util::angularDeviation;
-
 namespace osrm
 {
 namespace guidance
diff --git a/src/guidance/turn_handler.cpp b/src/guidance/turn_handler.cpp
index b187073ef51..35ee3b13904 100644
--- a/src/guidance/turn_handler.cpp
+++ b/src/guidance/turn_handler.cpp
@@ -684,7 +684,7 @@ bool TurnHandler::isCompatibleByRoadClass(const Intersection &intersection, cons
 boost::optional<TurnHandler::Fork> TurnHandler::findFork(const EdgeID via_edge,
                                                          Intersection &intersection) const
 {
-    const auto fork = findForkCandidatesByGeometry(intersection);
+    auto fork = findForkCandidatesByGeometry(intersection);
     if (fork)
     {
         // makes sure that the fork is isolated from other neighbouring streets on the left and
diff --git a/src/partitioner/dinic_max_flow.cpp b/src/partitioner/dinic_max_flow.cpp
index fc8efc4d2ca..a5def6a67c4 100644
--- a/src/partitioner/dinic_max_flow.cpp
+++ b/src/partitioner/dinic_max_flow.cpp
@@ -88,7 +88,7 @@ DinicMaxFlow::MinCut DinicMaxFlow::operator()(const BisectionGraphView &view,
             // heuristic)
             for (auto s : source_nodes)
                 levels[s] = 0;
-            const auto cut = MakeCut(view, levels, flow_value);
+            auto cut = MakeCut(view, levels, flow_value);
             return cut;
         }
     } while (true);
diff --git a/src/updater/updater.cpp b/src/updater/updater.cpp
index 78a0c19c46c..3e6688b17e6 100644
--- a/src/updater/updater.cpp
+++ b/src/updater/updater.cpp
@@ -504,7 +504,7 @@ bool IsRestrictionValid(const Timezoner &tz_handler, const extractor::Conditiona
 std::vector<std::uint64_t>
 updateConditionalTurns(std::vector<TurnPenalty> &turn_weight_penalties,
                        const std::vector<extractor::ConditionalTurnPenalty> &conditional_turns,
-                       Timezoner time_zone_handler)
+                       const Timezoner &time_zone_handler)
 {
     std::vector<std::uint64_t> updated_turns;
     if (conditional_turns.size() == 0)
diff --git a/src/util/conditional_restrictions.cpp b/src/util/conditional_restrictions.cpp
index 14aa2ea8af0..b748f1162c9 100644
--- a/src/util/conditional_restrictions.cpp
+++ b/src/util/conditional_restrictions.cpp
@@ -31,7 +31,6 @@ namespace detail
 
 namespace
 {
-namespace ph = boost::phoenix;
 namespace qi = boost::spirit::qi;
 } // namespace