Fixed clang-tidy warnings

src/class_diagram/model/class.cc

@@ -138,7 +138,7 @@ bool class_::template_specialization_found() const
 
 std::optional<std::string> class_::doxygen_link() const
 {
-    auto type = is_struct() ? "struct" : "class";
+    const auto *type = is_struct() ? "struct" : "class";
 
     auto name = name_and_ns();
     util::replace_all(name, "_", "__");

src/class_diagram/model/class_element.h

@@ -32,7 +32,7 @@ public:
     class_element(
         common::model::access_t scope, std::string name, std::string type);
 
-    virtual ~class_element() = default;
+    ~class_element() override = default;
 
     common::model::access_t access() const;
     std::string name() const;

src/class_diagram/model/concept.cc

@@ -70,9 +70,9 @@ std::string concept_::full_name(bool relative) const
     return res;
 }
 
-void concept_::add_parameter(method_parameter mp)
+void concept_::add_parameter(const method_parameter &mp)
 {
-    requires_parameters_.emplace_back(std::move(mp));
+    requires_parameters_.emplace_back(mp);
 }
 
 const std::vector<method_parameter> &concept_::requires_parameters() const

src/class_diagram/model/concept.h

@@ -53,7 +53,7 @@ public:
 
     std::string full_name_no_ns() const override;
 
-    void add_parameter(method_parameter mp);
+    void add_parameter(const method_parameter &mp);
 
     const std::vector<method_parameter> &requires_parameters() const;
 

src/class_diagram/model/method_parameter.h

@@ -31,7 +31,7 @@ public:
     method_parameter(
         std::string type, std::string name, std::string default_value = {});
 
-    virtual ~method_parameter() = default;
+    ~method_parameter() override = default;
 
     void set_type(const std::string &type);
     std::string type() const;

src/common/model/diagram_element.h

@@ -43,7 +43,7 @@ public:
 
     diagram_element();
 
-    virtual ~diagram_element() = default;
+    ~diagram_element() override = default;
 
     /**
      * @brief Returns diagram element id.

src/common/model/diagram_filter.cc

@@ -251,8 +251,8 @@ tvl::value_t namespace_filter::match(const diagram &d, const element &e) const
                     if (is_inclusive)
                         return e_ns.starts_with(nsit_ns) ||
                             nsit_ns.starts_with(e_ns) || e_ns == nsit_ns;
-                    else
-                        return e_ns.starts_with(nsit_ns) || e_ns == nsit_ns;
+
+                    return e_ns.starts_with(nsit_ns) || e_ns == nsit_ns;
                 }
 
                 return std::get<common::regex>(nsit.value()) %=

src/common/model/relationship.h

@@ -52,7 +52,7 @@ public:
         std::string multiplicity_source = "",
         std::string multiplicity_destination = "");
 
-    virtual ~relationship() = default;
+    ~relationship() override = default;
 
     /**
      * Set the type of relatinoship.