diff options
author | Philipp Oeser <info@graphics-engineer.com> | 2020-02-27 17:47:32 +0300 |
---|---|---|
committer | Philipp Oeser <info@graphics-engineer.com> | 2020-02-28 17:26:57 +0300 |
commit | 498397f7bd8158aec654d12ddf063868829b5903 (patch) | |
tree | b4b35a360c406ae65b0810622cf9e0cf28f5a85a /source/blender/editors/armature/armature_relations.c | |
parent | 4c83e6bac09fd9d7c973cbe056880d68f9944ae2 (diff) |
Fix (unreported) Separate bones creates empty armature
Fix for fix (rB8bf7ca9fe31c).
The fix was not actually working [neither for multi-armature-editing as
intended nor for single armatures] and it broke single-armature bone
separation completely.
More or less indentation issue, move relevant code out of bone loop, do
after all bones have been checked, guess this was intended anyways.
Differential Revision: https://developer.blender.org/D6949
Diffstat (limited to 'source/blender/editors/armature/armature_relations.c')
-rw-r--r-- | source/blender/editors/armature/armature_relations.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/source/blender/editors/armature/armature_relations.c b/source/blender/editors/armature/armature_relations.c index c2c7d7e5ee7..ae0501d14ef 100644 --- a/source/blender/editors/armature/armature_relations.c +++ b/source/blender/editors/armature/armature_relations.c @@ -639,14 +639,14 @@ static int separate_armature_exec(bContext *C, wmOperator *op) has_selected_any = true; } } - if (has_selected_bone == false) { - if (has_selected_any) { - /* Without this, we may leave head/tail selected - * which isn't expected after separating. */ - ED_armature_edit_deselect_all(ob_old); - } - continue; + } + if (has_selected_bone == false) { + if (has_selected_any) { + /* Without this, we may leave head/tail selected + * which isn't expected after separating. */ + ED_armature_edit_deselect_all(ob_old); } + continue; } } |