Oops, don't let the coincident surface merging stuff try to merge

empty (no trims) surfaces. It will generate a screwy bounding box,
which will make things break numerically later.

[git-p4: depot-paths = "//depot/solvespace/": change = 1979]
solver
Jonathan Westhues 2009-06-10 00:26:09 -08:00
parent 2013f9f466
commit 19fbae5b66
1 changed files with 3 additions and 0 deletions

View File

@ -13,6 +13,9 @@ void SShell::MergeCoincidentSurfaces(void) {
for(i = 0; i < surface.n; i++) { for(i = 0; i < surface.n; i++) {
si = &(surface.elem[i]); si = &(surface.elem[i]);
if(si->tag) continue; if(si->tag) continue;
// Let someone else clean up the empty surfaces; we can certainly merge
// them, but we don't know how to calculate a reasonable bounding box.
if(si->trim.n == 0) continue;
SEdgeList sel; SEdgeList sel;
ZERO(&sel); ZERO(&sel);