diff options
author | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 15:37:53 +0300 |
---|---|---|
committer | Sybren A. Stüvel <sybren@blender.org> | 2020-07-03 15:48:37 +0300 |
commit | 20869065b86263f72e469df17354bd6631be3712 (patch) | |
tree | 9f1aaaa7fecc62200ab48ef551df2510711a2cd7 /source/blender/bmesh/operators/bmo_connect_concave.c | |
parent | f3b8792b963b0b2a00fe7fe173f1beb85c209fbb (diff) |
Cleanup: BMesh, Clang-Tidy else-after-return fixes
This addresses warnings from Clang-Tidy's `readability-else-after-return`
rule in the `source/blender/bmesh` module.
No functional changes.
Diffstat (limited to 'source/blender/bmesh/operators/bmo_connect_concave.c')
-rw-r--r-- | source/blender/bmesh/operators/bmo_connect_concave.c | 25 |
1 files changed, 11 insertions, 14 deletions
diff --git a/source/blender/bmesh/operators/bmo_connect_concave.c b/source/blender/bmesh/operators/bmo_connect_concave.c index e7b86f102a2..bc1111676a3 100644 --- a/source/blender/bmesh/operators/bmo_connect_concave.c +++ b/source/blender/bmesh/operators/bmo_connect_concave.c @@ -61,23 +61,20 @@ static int bm_edge_length_cmp(const void *a_, const void *b_) if (e_a_concave < e_b_concave) { return -1; } - else if (e_a_concave > e_b_concave) { + if (e_a_concave > e_b_concave) { return 1; } - else { - /* otherwise shortest edges last */ - const float e_a_len = BM_edge_calc_length_squared(e_a); - const float e_b_len = BM_edge_calc_length_squared(e_b); - if (e_a_len < e_b_len) { - return 1; - } - else if (e_a_len > e_b_len) { - return -1; - } - else { - return 0; - } + + /* otherwise shortest edges last */ + const float e_a_len = BM_edge_calc_length_squared(e_a); + const float e_b_len = BM_edge_calc_length_squared(e_b); + if (e_a_len < e_b_len) { + return 1; + } + if (e_a_len > e_b_len) { + return -1; } + return 0; } static bool bm_face_split_by_concave(BMesh *bm, |