bartvdbraak/blender
1
0
Fork 4
Code Issues 4 Pull Requests 1 Actions 1 Packages Projects Releases Wiki Activity

Tweak fill-region method to produce fewer 'gaps'

Browse Source
This commit is contained in:
Campbell Barton 2016-04-06 15:07:37 +10:00
parent 16597b691a
commit 78d39cad8b
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
source/blender/bmesh/tools/bmesh_path_region.c
View File

@ -315,12 +315,27 @@ static LinkNode *mesh_calc_path_region_elem(
BMLoop *l_first, *l_iter;
l_iter = l_first = BM_FACE_FIRST_LOOP(f);
bool ok = true;
#if 0
do {
if (!bm_vert_region_test_chain(l_iter->v, depths, pass)) {
ok = false;
break;
}
} while ((l_iter = l_iter->next) != l_first);
#else
/* Allowing a single failure on a face gives fewer 'gaps'.
* While correct, in practice they're often part of what a user would consider the 'region'. */
int ok_tests = f->len > 3 ? 1 : 0; /* how many times we may fail */
do {
if (!bm_vert_region_test_chain(l_iter->v, depths, pass)) {
if (ok_tests == 0) {
ok = false;
break;
}
ok_tests--;
}
} while ((l_iter = l_iter->next) != l_first);
#endif
if (ok) {
BLI_linklist_prepend(&path, f);