BUG: clone convex shapes to avoid double transformations

Scaling, rotations and translations might be done twice to the same convex shapes if the same object is referenced in several convex shapes.
SimonRit · Feb 23, 2022 · 6d73ac0 · 6d73ac0
1 parent 9a386c5
commit 6d73ac0
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/rtkIntersectionOfConvexShapes.cxx b/src/rtkIntersectionOfConvexShapes.cxx
@@ -28,11 +28,10 @@ IntersectionOfConvexShapes ::IntersectionOfConvexShapes() = default;
 void
 IntersectionOfConvexShapes ::SetConvexShapes(const ConvexShapeVector & _arg)
 {
-  if (this->m_ConvexShapes != _arg)
-  {
-    this->m_ConvexShapes = _arg;
-    this->Modified();
-  }
+  m_ConvexShapes.clear();
+  for (auto & convexShape : _arg)
+    this->AddConvexShape(convexShape);
+  this->Modified();
 }
 
 bool