diff options
author | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 15:38:26 +0300 |
---|---|---|
committer | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 15:48:38 +0300 |
commit | 9dce2c9d1432d2798854b909e6262fbfb94ce3c7 (patch) | |
tree | 863025e2198da67fab539b16c107b9569a1fb14c /source/blender/editors/gpencil/gpencil_fill.c | |
parent | 20869065b86263f72e469df17354bd6631be3712 (diff) |
Cleanup: Editors/GPencil, Clang-Tidy else-after-return fixes
This addresses warnings from Clang-Tidy's `readability-else-after-return`
rule in the `source/blender/editors/gpencil` module.
No functional changes.
Diffstat (limited to 'source/blender/editors/gpencil/gpencil_fill.c')
-rw-r--r-- | source/blender/editors/gpencil/gpencil_fill.c | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/source/blender/editors/gpencil/gpencil_fill.c b/source/blender/editors/gpencil/gpencil_fill.c index 35bd44b88f9..8c901da100d 100644 --- a/source/blender/editors/gpencil/gpencil_fill.c +++ b/source/blender/editors/gpencil/gpencil_fill.c @@ -1199,15 +1199,12 @@ static bool gpencil_fill_poll(bContext *C) return true; } - else { - CTX_wm_operator_poll_msg_set(C, "Active region not valid for filling operator"); - return false; - } - } - else { - CTX_wm_operator_poll_msg_set(C, "Active region not set"); + CTX_wm_operator_poll_msg_set(C, "Active region not valid for filling operator"); return false; } + + CTX_wm_operator_poll_msg_set(C, "Active region not set"); + return false; } /* Allocate memory and initialize values */ @@ -1388,9 +1385,8 @@ static int gpencil_fill_invoke(bContext *C, wmOperator *op, const wmEvent *UNUSE } return OPERATOR_CANCELLED; } - else { - tgpf = op->customdata; - } + + tgpf = op->customdata; /* Enable custom drawing handlers to show help lines */ if (tgpf->flag & GP_BRUSH_FILL_SHOW_HELPLINES) { |