Changed meshScale param to const ref, updated function docs

Signed-off-by: Jasmeet Singh <jasmeet0915@gmail.com>
gazebosim · Sep 26, 2023 · d892b53 · d892b53
1 parent ffd1c41
commit d892b53
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/MeshInertiaCalculator.cc b/src/MeshInertiaCalculator.cc
@@ -39,8 +39,8 @@ using namespace sim;
 
 //////////////////////////////////////////////////
 void MeshInertiaCalculator::GetMeshTriangles(
-  std::vector<Triangle>& _triangles,
-  gz::math::Vector3d _meshScale,
+  std::vector<Triangle> &_triangles,
+  const gz::math::Vector3d &_meshScale,
   const gz::common::Mesh* _mesh)
 {
   // Get the vertices & indices of the mesh

diff --git a/src/MeshInertiaCalculator.hh b/src/MeshInertiaCalculator.hh
@@ -82,10 +82,12 @@ namespace gz
         /// \param[out] _triangles A vector to hold all the Triangle
         /// instances obtained
         /// from the vertices & indices of the mesh
+        /// \param[in] _meshScale A vector with the scaling factor
+        /// of all the 3 axes
         /// \param[in] _mesh Mesh object
         public: void GetMeshTriangles(
-          std::vector<Triangle>& _triangles,
-          gz::math::Vector3d _meshScale,
+          std::vector<Triangle> &_triangles,
+          const gz::math::Vector3d &_meshScale,
           const gz::common::Mesh* _mesh);
 
         /// \brief Function to calculate the centroid of the mesh. Since