From 20869065b86263f72e469df17354bd6631be3712 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Fri, 3 Jul 2020 14:37:53 +0200 Subject: 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. --- source/blender/bmesh/tools/bmesh_intersect_edges.c | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) (limited to 'source/blender/bmesh/tools/bmesh_intersect_edges.c') diff --git a/source/blender/bmesh/tools/bmesh_intersect_edges.c b/source/blender/bmesh/tools/bmesh_intersect_edges.c index 3f0492aebb6..99a5a9edb57 100644 --- a/source/blender/bmesh/tools/bmesh_intersect_edges.c +++ b/source/blender/bmesh/tools/bmesh_intersect_edges.c @@ -127,12 +127,10 @@ static bool bm_vert_pair_share_splittable_face_cb(BMFace *UNUSED(f), if (IN_RANGE(lambda_b, range_min, range_max)) { return true; } - else { - copy_v3_v3(co, l_b->prev->v->co); - sub_v3_v3v3(dir, l_b->next->v->co, co); - if (isect_ray_ray_v3(v_a_co, v_a_b_dir, co, dir, NULL, &lambda_b)) { - return IN_RANGE(lambda_b, range_min, range_max); - } + copy_v3_v3(co, l_b->prev->v->co); + sub_v3_v3v3(dir, l_b->next->v->co, co); + if (isect_ray_ray_v3(v_a_co, v_a_b_dir, co, dir, NULL, &lambda_b)) { + return IN_RANGE(lambda_b, range_min, range_max); } } return false; @@ -476,9 +474,7 @@ static int sort_cmp_by_lambda_cb(const void *index1_v, const void *index2_v, voi if (pair_flat[index1].lambda > pair_flat[index2].lambda) { return 1; } - else { - return -1; - } + return -1; } /* -------------------------------------------------------------------- */ -- cgit v1.2.3