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_hide.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_hide.c')
-rw-r--r-- | source/blender/editors/sculpt_paint/paint_hide.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/source/blender/editors/sculpt_paint/paint_hide.c b/source/blender/editors/sculpt_paint/paint_hide.c index e9dcc4a356a..339921fe601 100644 --- a/source/blender/editors/sculpt_paint/paint_hide.c +++ b/source/blender/editors/sculpt_paint/paint_hide.c @@ -70,13 +70,12 @@ static bool is_effected(PartialVisArea area, if (area == PARTIALVIS_ALL) { return true; } - else if (area == PARTIALVIS_MASKED) { + if (area == PARTIALVIS_MASKED) { return mask > 0.5f; } - else { - bool inside = isect_point_planes_v3(planes, 4, co); - return ((inside && area == PARTIALVIS_INSIDE) || (!inside && area == PARTIALVIS_OUTSIDE)); - } + + bool inside = isect_point_planes_v3(planes, 4, co); + return ((inside && area == PARTIALVIS_INSIDE) || (!inside && area == PARTIALVIS_OUTSIDE)); } static void partialvis_update_mesh(Object *ob, @@ -418,9 +417,7 @@ static int hide_show_invoke(bContext *C, wmOperator *op, const wmEvent *event) if (!ELEM(area, PARTIALVIS_ALL, PARTIALVIS_MASKED)) { return WM_gesture_box_invoke(C, op, event); } - else { - return op->type->exec(C, op); - } + return op->type->exec(C, op); } void PAINT_OT_hide_show(struct wmOperatorType *ot) |