Merge pull request InsightSoftwareConsortium#5128 from seanm/non-bool…

…-return BUG: -1 was being returned in a function that returns bool
Kitware · Jan 9, 2025 · 43bdc37 · 43bdc37
2 parents dbb5005 + db0990b
commit 43bdc37
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Modules/Core/Common/include/itkHexahedronCell.hxx b/Modules/Core/Common/include/itkHexahedronCell.hxx
@@ -452,7 +452,7 @@ HexahedronCell<TCellInterface>::EvaluatePosition(CoordinateType *          x,
     else if ((itk::Math::abs(pcoords[0]) > ITK_DIVERGED) || (itk::Math::abs(pcoords[1]) > ITK_DIVERGED) ||
              (itk::Math::abs(pcoords[2]) > ITK_DIVERGED))
     {
-      return -1;
+      return false;
     }
 
     //  if not converged, repeat

diff --git a/Modules/Core/Common/include/itkQuadrilateralCell.hxx b/Modules/Core/Common/include/itkQuadrilateralCell.hxx
@@ -373,7 +373,7 @@ QuadrilateralCell<TCellInterface>::EvaluatePosition(CoordinateType *          x,
     // Test for bad divergence (S.Hirschberg 11.12.2001)
     else if ((itk::Math::abs(pcoords[0]) > ITK_DIVERGED) || (itk::Math::abs(pcoords[1]) > ITK_DIVERGED))
     {
-      return -1;
+      return false;
     }
 
     //  if not converged, repeat