diff options
author | Campbell Barton <ideasman42@gmail.com> | 2018-03-22 09:50:58 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2018-03-22 09:50:58 +0300 |
commit | 865fbe343a9067f342c45fb2a1a63eca0f7cf2d7 (patch) | |
tree | 033f1552d92497da98e38d1fa95cd8d4cd162acb /source/blender/editors/armature/armature_utils.c | |
parent | ce51066e47175a38fd205987b6e6acc1a805dbe0 (diff) |
Fix T54348: Bone dissolve gives invalid hierarchy
Disconnected bones weren't handled correctly.
Diffstat (limited to 'source/blender/editors/armature/armature_utils.c')
-rw-r--r-- | source/blender/editors/armature/armature_utils.c | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/source/blender/editors/armature/armature_utils.c b/source/blender/editors/armature/armature_utils.c index 0efd9cec959..3928cb2d85d 100644 --- a/source/blender/editors/armature/armature_utils.c +++ b/source/blender/editors/armature/armature_utils.c @@ -133,7 +133,10 @@ void bone_free(bArmature *arm, EditBone *bone) BLI_freelinkN(arm->edbo, bone); } -void ED_armature_edit_bone_remove(bArmature *arm, EditBone *exBone) +/** + * \param clear_connected: When false caller is responsible for keeping the flag in a valid state. + */ +void ED_armature_edit_bone_remove_ex(bArmature *arm, EditBone *exBone, bool clear_connected) { EditBone *curBone; @@ -141,13 +144,20 @@ void ED_armature_edit_bone_remove(bArmature *arm, EditBone *exBone) for (curBone = arm->edbo->first; curBone; curBone = curBone->next) { if (curBone->parent == exBone) { curBone->parent = exBone->parent; - curBone->flag &= ~BONE_CONNECTED; + if (clear_connected) { + curBone->flag &= ~BONE_CONNECTED; + } } } bone_free(arm, exBone); } +void ED_armature_edit_bone_remove(bArmature *arm, EditBone *exBone) +{ + ED_armature_edit_bone_remove_ex(arm, exBone, true); +} + bool ED_armature_ebone_is_child_recursive(EditBone *ebone_parent, EditBone *ebone_child) { for (ebone_child = ebone_child->parent; ebone_child; ebone_child = ebone_child->parent) { |