diff options
| author | Sébastien Crozet <developer@crozet.re> | 2023-01-21 18:59:34 +0100 |
|---|---|---|
| committer | Sébastien Crozet <developer@crozet.re> | 2023-01-21 19:00:07 +0100 |
| commit | de59c6b58c53535d4dd87a6de552d89d30984ea2 (patch) | |
| tree | e27c68a576c04d519ac5d2e447d043108ffbf83c /src | |
| parent | 155fb9b1c08a2c86fb36462d953790befe965164 (diff) | |
| download | rapier-de59c6b58c53535d4dd87a6de552d89d30984ea2.tar.gz rapier-de59c6b58c53535d4dd87a6de552d89d30984ea2.tar.bz2 rapier-de59c6b58c53535d4dd87a6de552d89d30984ea2.zip | |
Fix rigid-body mass-properties being incorrect after disable/enable
Diffstat (limited to 'src')
| -rw-r--r-- | src/pipeline/user_changes.rs | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/src/pipeline/user_changes.rs b/src/pipeline/user_changes.rs index eeda97a..a047256 100644 --- a/src/pipeline/user_changes.rs +++ b/src/pipeline/user_changes.rs @@ -161,16 +161,6 @@ pub(crate) fn handle_user_changes_to_rigid_bodies( } } - if changes - .intersects(RigidBodyChanges::LOCAL_MASS_PROPERTIES | RigidBodyChanges::COLLIDERS) - { - rb.mprops.recompute_mass_properties_from_colliders( - colliders, - &rb.colliders, - &rb.pos.position, - ); - } - if changes.contains(RigidBodyChanges::ENABLED_OR_DISABLED) { // Propagate the rigid-body’s enabled/disable status to its colliders. for handle in rb.colliders.0.iter() { @@ -208,6 +198,19 @@ pub(crate) fn handle_user_changes_to_rigid_bodies( } } + // NOTE: recompute the mass-properties AFTER dealing with the rigid-body changes + // that imply a collider change (in particular, after propagation of the + // enabled/disabled status). + if changes + .intersects(RigidBodyChanges::LOCAL_MASS_PROPERTIES | RigidBodyChanges::COLLIDERS) + { + rb.mprops.recompute_mass_properties_from_colliders( + colliders, + &rb.colliders, + &rb.pos.position, + ); + } + rb.ids = ids; rb.activation = activation; } |
