Fix owner leak in destroy_shape() (#344)

Chubercik · web-flow · commit fbbb65a4de1d · 2025-02-21T22:08:47.000+01:00
Ensures that lingering owner references are cleared after a shape is destroyed by adding a call to `clear()` on the `owners` HashMap. Fixes #266.
diff --git a/src/shapes/rapier_shape_base.rs b/src/shapes/rapier_shape_base.rs
@@ -120,6 +120,7 @@ impl RapierShapeBase {
 
     pub fn destroy_shape(&mut self, physics_engine: &mut PhysicsEngine) {
         physics_engine.shape_destroy(self.get_id());
+        self.state.owners.clear();
     }
 
     #[cfg(feature = "serde-serialize")]

Original file line number	Diff line number	Diff line change
`@@ -120,6 +120,7 @@ impl RapierShapeBase {`
`120`	`120`
`121`	`121`	`pub fn destroy_shape(&mut self, physics_engine: &mut PhysicsEngine) {`
`122`	`122`	`physics_engine.shape_destroy(self.get_id());`
	`123`	`+ self.state.owners.clear();`
`123`	`124`	`}`
`124`	`125`
`125`	`126`	`#[cfg(feature = "serde-serialize")]`