diff options
author | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 15:52:32 +0300 |
---|---|---|
committer | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 17:15:00 +0300 |
commit | 7d0a0b8a6da0e1cd41a1b40971e7c803ed162963 (patch) | |
tree | 03ae0a0eab32e9d7101f78e4a2dc20f0c58af88b /source/blender/editors/armature/armature_edit.c | |
parent | f82e52ebc838b666c258090236f0323d0d4f6aee (diff) |
Cleanup: Editors/Armature, Clang-Tidy else-after-return fixes
This addresses warnings from Clang-Tidy's `readability-else-after-return`
rule in the `source/blender/editors/armature` module.
No functional changes.
Diffstat (limited to 'source/blender/editors/armature/armature_edit.c')
-rw-r--r-- | source/blender/editors/armature/armature_edit.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source/blender/editors/armature/armature_edit.c b/source/blender/editors/armature/armature_edit.c index a010fbd5e81..113fd54713e 100644 --- a/source/blender/editors/armature/armature_edit.c +++ b/source/blender/editors/armature/armature_edit.c @@ -728,7 +728,8 @@ static int armature_fill_bones_exec(bContext *C, wmOperator *op) BKE_report(op->reports, RPT_ERROR, "No joints selected"); return OPERATOR_CANCELLED; } - else if (mixed_object_error) { + + if (mixed_object_error) { BKE_report(op->reports, RPT_ERROR, "Bones for different objects selected"); BLI_freelistN(&points); return OPERATOR_CANCELLED; @@ -1093,7 +1094,8 @@ static int armature_align_bones_exec(bContext *C, wmOperator *op) BKE_report(op->reports, RPT_ERROR, "Operation requires an active bone"); return OPERATOR_CANCELLED; } - else if (arm->flag & ARM_MIRROR_EDIT) { + + if (arm->flag & ARM_MIRROR_EDIT) { /* For X-Axis Mirror Editing option, we may need a mirror copy of actbone * - if there's a mirrored copy of selbone, try to find a mirrored copy of actbone * (i.e. selbone="child.L" and actbone="parent.L", find "child.R" and "parent.R"). |