Apply suggestions from code review

Co-authored-by: Alejandro Hernández Cordero <[email protected]> Signed-off-by: Benjamin Perseghetti <[email protected]>
rudislabs · May 16, 2024 · cc0f8cf · cc0f8cf
1 parent cbd801b
commit cc0f8cf
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 21 deletions.
diff --git a/include/gz/math/Cone.hh b/include/gz/math/Cone.hh
@@ -57,8 +57,8 @@ namespace gz
       /// \param[in] _radius Radius of the cone.
       /// \param[in] _rotOffset Rotational offset of the cone.
       public: Cone(const Precision _length, const Precision _radius,
-                  const Quaternion<Precision> &_rotOffset =
-                  Quaternion<Precision>::Identity);
+                   const Quaternion<Precision> &_rotOffset =
+                   Quaternion<Precision>::Identity);
 
       /// \brief Construct a cone with a length, radius, material and
       /// optionally a rotational offset.
@@ -67,9 +67,9 @@ namespace gz
       /// \param[in] _mat Material property for the cone.
       /// \param[in] _rotOffset Rotational offset of the cone.
       public: Cone(const Precision _length, const Precision _radius,
-                  const Material &_mat,
-                  const Quaternion<Precision> &_rotOffset =
-                  Quaternion<Precision>::Identity);
+                   const Material &_mat,
+                   const Quaternion<Precision> &_rotOffset =
+                   Quaternion<Precision>::Identity);
 
       /// \brief Get the radius in meters.
       /// \return The radius of the cone in meters.

diff --git a/include/gz/math/MassMatrix3.hh b/include/gz/math/MassMatrix3.hh
@@ -939,9 +939,9 @@ namespace gz
       /// \param[in] _rot Rotational offset of equivalent cone.
       /// \return True if inertial properties were set successfully.
       public: bool SetFromConeZ(const Material &_mat,
-                                    const T _length,
-                                    const T _radius,
-                            const Quaternion<T> &_rot = Quaternion<T>::Identity)
+                                const T _length,
+                                const T _radius,
+                                const Quaternion<T> &_rot = Quaternion<T>::Identity)
       {
         // Check that density, _radius and _length are strictly positive
         // and that quaternion is valid
@@ -963,9 +963,9 @@ namespace gz
       /// \param[in] _rot Rotational offset of equivalent cone.
       /// \return True if inertial properties were set successfully.
       public: bool SetFromConeZ(const T _mass,
-                                    const T _length,
-                                    const T _radius,
-                            const Quaternion<T> &_rot = Quaternion<T>::Identity)
+                                const T _length,
+                                const T _radius,
+                                const Quaternion<T> &_rot = Quaternion<T>::Identity)
       {
         // Check that _mass, _radius and _length are strictly positive
         // and that quaternion is valid
@@ -985,8 +985,8 @@ namespace gz
       /// \param[in] _rot Rotational offset of equivalent cone.
       /// \return True if inertial properties were set successfully.
       public: bool SetFromConeZ(const T _length,
-                                    const T _radius,
-                                    const Quaternion<T> &_rot)
+                                const T _radius,
+                                const Quaternion<T> &_rot)
       {
         // Check that _mass and _size are strictly positive
         // and that quaternion is valid

diff --git a/src/ruby/MassMatrix3.i b/src/ruby/MassMatrix3.i
@@ -104,16 +104,16 @@ namespace gz
                                     const T _radius,
                                     const Quaternion<T> &_rot);
       public: bool SetFromConeZ(const Material &_mat,
-                                    const T _length,
-                                    const T _radius,
-                                    const Quaternion<T> &_rot = Quaternion<T>::Identity);
+                                const T _length,
+                                const T _radius,
+                                const Quaternion<T> &_rot = Quaternion<T>::Identity);
       public: bool SetFromConeZ(const T _mass,
-                                    const T _length,
-                                    const T _radius,
-                                    const Quaternion<T> &_rot = Quaternion<T>::Identity);
+                                const T _length,
+                                const T _radius,
+                                const Quaternion<T> &_rot = Quaternion<T>::Identity);
       public: bool SetFromConeZ(const T _length,
-                                    const T _radius,
-                                    const Quaternion<T> &_rot);
+                                const T _radius,
+                                const Quaternion<T> &_rot);
       public: bool SetFromSphere(const Material &_mat, const T _radius);
       public: bool SetFromSphere(const T _mass, const T _radius);
       public: bool SetFromSphere(const T _radius);