Resolve cppcheck errors by adding explicit constructors and consts

Also add some suppressions.

Signed-off-by: Jeremy Nimmer <[email protected]>

Changelog.md

@@ -30,6 +30,9 @@
 1. Evict large function definitions from the Helpers.hh header file.
     * [Pull request 288](https://github.com/ignitionrobotics/ign-math/pull/288)
 
+1. Resolve cppcheck errors by adding explicit constructors and consts.
+    * [Pull request 291](https://github.com/ignitionrobotics/ign-math/pull/291)
+
 ## Ignition Math 6.x
 
 ### Ignition Math 6.x.x

include/ignition/math/OrientedBox.hh

@@ -131,15 +131,15 @@ namespace ignition
       /// \brief Set the box size.
       /// \param[in] _size Box size, in its own coordinate frame. Its absolute
       /// value will be taken, so the size is non-negative.
-      public: void Size(Vector3<T> &_size)
+      public: void Size(const Vector3<T> &_size)
       {
         // Enforce non-negative size
         this->size = _size.Abs();
       }
 
       /// \brief Set the box pose.
       /// \param[in] _pose Box pose.
-      public: void Pose(Pose3<T> &_pose)
+      public: void Pose(const Pose3<T> &_pose)
       {
         this->pose = _pose;
       }

include/ignition/math/Plane.hh

@@ -63,7 +63,7 @@ namespace ignition
       /// \brief Constructor from a normal and a distance
       /// \param[in] _normal The plane normal
       /// \param[in] _offset Offset along the normal
-      public: Plane(const Vector3<T> &_normal, T _offset = 0.0)
+      public: explicit Plane(const Vector3<T> &_normal, T _offset = 0.0)
       : normal(_normal), d(_offset)
       {
       }

include/ignition/math/SemanticVersion.hh

@@ -63,11 +63,11 @@ namespace ignition
       /// \param[in] _patch The patch number
       /// \param[in] _prerelease The prerelease string
       /// \param[in] _build The build metadata string
-      public: SemanticVersion(const unsigned int _major,
-                              const unsigned int _minor = 0,
-                              const unsigned int _patch = 0,
-                              const std::string &_prerelease = "",
-                              const std::string &_build = "");
+      public: explicit SemanticVersion(const unsigned int _major,
+                                       const unsigned int _minor = 0,
+                                       const unsigned int _patch = 0,
+                                       const std::string &_prerelease = "",
+                                       const std::string &_build = "");
 
       /// \brief Destructor
       public: ~SemanticVersion();

include/ignition/math/graph/Edge.hh

@@ -47,6 +47,7 @@ namespace graph
     /// \param[in] _vertices The vertices of the edge.
     /// \param[in] _data The data stored in the edge.
     /// \param[in] _weight The weight (cost) of the edge.
+    // cppcheck-suppress noExplicitConstructor
     EdgeInitializer(const VertexId_P &_vertices,
                     const E &_data = E(),
                     const double _weight = 1)

include/ignition/math/graph/GraphAlgorithms.hh

@@ -340,6 +340,7 @@ namespace graph
     // Add all vertices.
     for (auto const &vPair : _graph.Vertices())
     {
+      // cppcheck-suppress useStlAlgorithm
       vertices.push_back(vPair.second.get());
     }
 

include/ignition/math/graph/Vertex.hh

@@ -60,6 +60,7 @@ namespace graph
     /// \param[in] _name Non-unique vertex name.
     /// \param[in] _data User information.
     /// \param[in] _id Optional unique id.
+    // cppcheck-suppress noExplicitConstructor
     public: Vertex(const std::string &_name,
                    const V &_data = V(),
                    const VertexId _id = kNullId)

src/Frustum.cc

@@ -156,6 +156,7 @@ bool Frustum::Contains(const Vector3d &_p) const
   // visible.
   for (auto const &plane : this->dataPtr->planes)
   {
+    // cppcheck-suppress useStlAlgorithm
     if (plane.Side(_p) == Planed::NEGATIVE_SIDE)
       return false;
   }

src/Kmeans_TEST.cc

@@ -48,7 +48,10 @@ TEST(KmeansTest, Kmeans)
 
   // ::SetObservations()
   for (auto &elem : obsCopy)
+  {
+    // cppcheck-suppress useStlAlgorithm
     elem += math::Vector3d(0.1, 0.2, 0.0);
+  }
 
   EXPECT_TRUE(kmeans.Observations(obsCopy));