Use const instead of constexpr in Ellipsoid constructor (#366)

adlarkin · web-flow · commit 3ade9b1d5ffd · 2022-01-12T19:34:43.000-08:00
Signed-off-by: Ashton Larkin &lt;42042756+adlarkin@users.noreply.github.com&gt;
diff --git a/include/ignition/math/detail/Ellipsoid.hh b/include/ignition/math/detail/Ellipsoid.hh
@@ -96,7 +96,7 @@ std::optional< MassMatrix3<T> > Ellipsoid<T>::MassMatrix() const
 template<typename T>
 T Ellipsoid<T>::Volume() const
 {
-  constexpr T kFourThirdsPi = 4. * IGN_PI / 3.;
+  const T kFourThirdsPi = 4. * IGN_PI / 3.;
   return kFourThirdsPi * this->radii.X() * this->radii.Y() * this->radii.Z();
 }
 

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,7 @@ std::optional< MassMatrix3<T> > Ellipsoid<T>::MassMatrix() const`
`96`	`96`	`template<typename T>`
`97`	`97`	`T Ellipsoid<T>::Volume() const`
`98`	`98`	`{`
`99`		`- constexpr T kFourThirdsPi = 4. * IGN_PI / 3.;`
	`99`	`+ const T kFourThirdsPi = 4. * IGN_PI / 3.;`
`100`	`100`	`return kFourThirdsPi * this->radii.X() * this->radii.Y() * this->radii.Z();`
`101`	`101`	`}`
`102`	`102`