Apply suggestions from code review

Co-authored-by: Alejandro Hernández Cordero <[email protected]> Signed-off-by: Benjamin Perseghetti <[email protected]>
rudislabs · May 20, 2024 · 341727d · 341727d
1 parent 7ab3d56
commit 341727d
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/src/Cone.cc b/src/Cone.cc
@@ -16,8 +16,9 @@
  * limitations under the License.
  *
 */
-#include <sstream>
 #include <optional>
+#include <sstream>
+#include <utility>
 
 #include <gz/math/Inertial.hh>
 #include <gz/math/Cone.hh>

diff --git a/src/Cone_TEST.cc b/src/Cone_TEST.cc
@@ -208,7 +208,7 @@ TEST(DOMCone, CalculateInertial)
   cone.SetRadius(r);
 
   double expectedMass = cone.Shape().Volume() * density;
-  double ixxIyy = (3/80.0) * expectedMass * (4*r*r + l*l);
+  double ixxIyy = (3 / 80.0) * expectedMass * (4 * r * r + l * l);
   double izz = 3.0 * expectedMass * r * r / 10.0;
 
   gz::math::MassMatrix3d expectedMassMat(

diff --git a/src/Geometry_TEST.cc b/src/Geometry_TEST.cc
@@ -426,7 +426,7 @@ TEST(DOMGeometry, CalculateInertial)
     cone.SetRadius(r);
 
     expectedMass = cone.Shape().Volume() * density;
-    double ixxIyy = (3/80.0) * expectedMass * (4*r*r + l*l);
+    double ixxIyy = (3 / 80.0) * expectedMass * (4 * r * r + l * l);
     double izz = 3.0 * expectedMass * r * r / 10.0;
 
     expectedMassMat.SetMass(expectedMass);