diff options
author | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 17:09:51 +0300 |
---|---|---|
committer | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 17:15:01 +0300 |
commit | 2a39b34a09702e4e64e27befc97dd6154b75ccc0 (patch) | |
tree | 1a4cfc24b941dee5895b2dbd388c9cc453a31ce3 /source/blender/editors/sculpt_paint/paint_curve.c | |
parent | fd5b093f84845ab7adbe7e6e4dec4bbadbbc16af (diff) |
Cleanup: Editors/Sculpt/Paint, Clang-Tidy else-after-return fixes
This addresses warnings from Clang-Tidy's `readability-else-after-return`
rule in the `source/blender/editors/sculpt_paint` module.
No functional changes.
Diffstat (limited to 'source/blender/editors/sculpt_paint/paint_curve.c')
-rw-r--r-- | source/blender/editors/sculpt_paint/paint_curve.c | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/source/blender/editors/sculpt_paint/paint_curve.c b/source/blender/editors/sculpt_paint/paint_curve.c index 68fd2f15877..74e022bf84f 100644 --- a/source/blender/editors/sculpt_paint/paint_curve.c +++ b/source/blender/editors/sculpt_paint/paint_curve.c @@ -142,19 +142,15 @@ static char paintcurve_point_side_index(const BezTriple *bezt, if ((bezt->f1 & SELECT) == (bezt->f3 & SELECT)) { return is_first ? SEL_F1 : SEL_F3; } - else if (bezt->f1 & SELECT) { + if (bezt->f1 & SELECT) { return SEL_F1; } - else if (bezt->f3 & SELECT) { + if (bezt->f3 & SELECT) { return SEL_F3; } - else { - return fallback; - } - } - else { - return 0; + return fallback; } + return 0; } /******************* Operators *********************************/ @@ -491,9 +487,7 @@ static int paintcurve_select_point_invoke(bContext *C, wmOperator *op, const wmE RNA_int_set_array(op->ptr, "location", loc); return OPERATOR_FINISHED; } - else { - return OPERATOR_CANCELLED; - } + return OPERATOR_CANCELLED; } static int paintcurve_select_point_exec(bContext *C, wmOperator *op) |