remove density warning, fix typo

Signed-off-by: Ian Chen <[email protected]>
gazebosim · Dec 11, 2024 · ca94a22 · ca94a22
1 parent 92da673
commit ca94a22
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 13 deletions.
diff --git a/src/Collision.cc b/src/Collision.cc
@@ -291,18 +291,8 @@ void Collision::CalculateInertial(
   // 3. DensityDefault value.
   else
   {
-    // If density was not explicitly set, send a warning
-    // about the default value being used
+    // If density was not explicitly set, default value is used
     density = DensityDefault();
-    Error densityMissingErr(
-      ErrorCode::ELEMENT_MISSING,
-      "Collision is missing a <density> child element. "
-      "Using a default density value of " +
-      std::to_string(DensityDefault()) + " kg/m^3. "
-    );
-    enforceConfigurablePolicyCondition(
-      _config.WarningsPolicy(), densityMissingErr, _errors
-    );
   }
 
   // If this Collision's auto inertia params have not been set, then use the

diff --git a/src/Link_TEST.cc b/src/Link_TEST.cc
@@ -777,7 +777,7 @@ TEST(DOMLink, ResolveAutoInertialsWithMass)
 {
   // A model with link inertial auto set to true.
   // The inertia matrix is specified but should be ignored.
-  // <mass> is speicifed - the auto computed inertial values should
+  // <mass> is specified - the auto computed inertial values should
   // be scaled based on the desired mass.
   std::string sdf = "<?xml version=\"1.0\"?>"
   "<sdf version=\"1.11\">"
@@ -826,7 +826,7 @@ TEST(DOMLink, ResolveAutoInertialsWithMassAndMultipleCollisions)
 {
   // A model with link inertial auto set to true.
   // The inertia matrix is specified but should be ignored.
-  // <mass> is speicifed - the auto computed inertial values should
+  // <mass> is specified - the auto computed inertial values should
   // be scaled based on the desired mass.
   std::string sdf = "<?xml version=\"1.0\"?>"
   "<sdf version=\"1.11\">"