diff options
| author | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 18:30:31 +0300 |
|---|---|---|
| committer | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 18:42:46 +0300 |
| commit | 33a74941c5fd4efb6eefcaace3315d3e2b65681f (patch) | |
| tree | 2d3da68b73e4419750c7f767bbb78bf1a303ad18 /source/blender/editors/curve/editcurve.c | |
| parent | 367034f210137754ab4f07a0e7793066d2b69e59 (diff) | |
Cleanup: Editors, Clang-Tidy else-after-return fixes
This addresses warnings from Clang-Tidy's `readability-else-after-return`
rule in the `source/blender/editors` module.
No functional changes.
Diffstat (limited to 'source/blender/editors/curve/editcurve.c')
| -rw-r--r-- | source/blender/editors/curve/editcurve.c | 36 |
1 files changed, 15 insertions, 21 deletions
diff --git a/source/blender/editors/curve/editcurve.c b/source/blender/editors/curve/editcurve.c index a39c8261b32..38ab917ba72 100644 --- a/source/blender/editors/curve/editcurve.c +++ b/source/blender/editors/curve/editcurve.c @@ -2436,12 +2436,12 @@ static void adduplicateflagNurb( /* actvert in cyclicu selection */ break; } - else if (calc_duplicate_actvert(editnurb, - newnurb, - cu, - starta, - starta + newu, - cu->actvert - starta + b * newnu->pntsu)) { + if (calc_duplicate_actvert(editnurb, + newnurb, + cu, + starta, + starta + newu, + cu->actvert - starta + b * newnu->pntsu)) { /* actvert in 'current' iteration selection */ break; } @@ -4544,15 +4544,13 @@ static int make_segment_exec(bContext *C, wmOperator *op) if (nu_select_num > 1) { break; } - else { - /* only 1 selected, not first or last, a little complex, but intuitive */ - if (nu->pntsv == 1) { - if ((nu->bp->f1 & SELECT) || (nu->bp[nu->pntsu - 1].f1 & SELECT)) { - /* pass */ - } - else { - break; - } + /* only 1 selected, not first or last, a little complex, but intuitive */ + if (nu->pntsv == 1) { + if ((nu->bp->f1 & SELECT) || (nu->bp[nu->pntsu - 1].f1 & SELECT)) { + /* pass */ + } + else { + break; } } } @@ -5593,9 +5591,7 @@ static int add_vertex_exec(bContext *C, wmOperator *op) return OPERATOR_FINISHED; } - else { - return OPERATOR_CANCELLED; - } + return OPERATOR_CANCELLED; } static int add_vertex_invoke(bContext *C, wmOperator *op, const wmEvent *event) @@ -6517,9 +6513,7 @@ static int curve_delete_exec(bContext *C, wmOperator *op) if (changed_multi) { return OPERATOR_FINISHED; } - else { - return OPERATOR_CANCELLED; - } + return OPERATOR_CANCELLED; } static const EnumPropertyItem curve_delete_type_items[] = { |