diff options
author | Campbell Barton <ideasman42@gmail.com> | 2021-06-09 18:55:44 +0300 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2021-06-09 19:22:45 +0300 |
commit | 4443c4082ed094baf11b5c6cff280582776466b6 (patch) | |
tree | e2878edc62e22635e9feb589a5f9590c4fdf5d87 /source/blender/editors/uvedit/uvedit_select.c | |
parent | bda8887e0cdfd2fc035b6f1d97f16120f719c6a3 (diff) |
Cleanup: remove redundant checks which have already been tested
Note that these changes are limited simple cases as these kinds of
changes could allow for errors when refactoring code when the known
state is not so obvious.
Diffstat (limited to 'source/blender/editors/uvedit/uvedit_select.c')
-rw-r--r-- | source/blender/editors/uvedit/uvedit_select.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/source/blender/editors/uvedit/uvedit_select.c b/source/blender/editors/uvedit/uvedit_select.c index c10e132a4e2..39950f9d0c7 100644 --- a/source/blender/editors/uvedit/uvedit_select.c +++ b/source/blender/editors/uvedit/uvedit_select.c @@ -1381,7 +1381,7 @@ static void uv_select_linked_multi(Scene *scene, BM_ITER_ELEM (l, &liter, efa, BM_LOOPS_OF_FACE) { if (uvedit_uv_select_test(scene, l, cd_loop_uv_offset)) { bool add_to_stack = true; - if (uv_sync_select && !select_faces) { + if (uv_sync_select) { /* Special case, vertex/edge & sync select being enabled. * * Without this, a second linked select will 'grow' each time as each new @@ -1392,6 +1392,7 @@ static void uv_select_linked_multi(Scene *scene, * - The only other fully selected face is connected or, * - There are no connected fully selected faces UV-connected to this loop. */ + BLI_assert(!select_faces); if (uvedit_face_select_test(scene, l->f, cd_loop_uv_offset)) { /* pass */ } |