Fix cached safety calculation in O2Tessellated

sawenzel · web-flow · commit 5afccc84cfd6 · 2026-03-06T16:56:23.000+01:00
diff --git a/Detectors/Base/src/O2Tessellated.cxx b/Detectors/Base/src/O2Tessellated.cxx
@@ -1359,8 +1359,8 @@ Double_t O2Tessellated::Safety(const Double_t* point, Bool_t in) const
 
     if (d2 < mCachedSafety * mCachedSafety) {
       // we moved less than known safety
-      mCachedSafety - std::sqrt(d2);
       cached_counter++;
+      return mCachedSafety - std::sqrt(d2);
     }
   }
 
@@ -1528,4 +1528,4 @@ void O2Tessellated::CalculateNormals()
   }
 }
 
-// NOLINTEND
+// NOLINTEND

Original file line number	Diff line number	Diff line change
`@@ -1359,8 +1359,8 @@ Double_t O2Tessellated::Safety(const Double_t* point, Bool_t in) const`
`1359`	`1359`
`1360`	`1360`	`if (d2 < mCachedSafety * mCachedSafety) {`
`1361`	`1361`	`// we moved less than known safety`
`1362`		`- mCachedSafety - std::sqrt(d2);`
`1363`	`1362`	`cached_counter++;`
	`1363`	`+ return mCachedSafety - std::sqrt(d2);`
`1364`	`1364`	`}`
`1365`	`1365`	`}`
`1366`	`1366`
`@@ -1528,4 +1528,4 @@ void O2Tessellated::CalculateNormals()`
`1528`	`1528`	`}`
`1529`	`1529`	`}`
`1530`	`1530`
`1531`		`-// NOLINTEND`
	`1531`	`+// NOLINTEND`