Opened 12 years ago
Closed 11 years ago
#608 closed defect (fixed)
IsValidOp should return eNestedShells not eNestedHoles at ln 530
Reported by: | geomworx | Owned by: | |
---|---|---|---|
Priority: | minor | Milestone: | 3.3.9 |
Component: | Default | Version: | 3.3.6 |
Severity: | Unassigned | Keywords: | history 3.3.8 |
Cc: |
Description
Stepping through code, I believe that inside 3rd_geos\IsValidOp.cpp::checkShellNotNested(), on line 530, the keyword:
eNestedHoles
should be replaced by:
eNestedShells
Since the error detected here is shells being nested, not holes being nested.
Change History (1)
comment:1 by , 11 years ago
Keywords: | history 3.3.8 added |
---|---|
Milestone: | 3.3.7 → 3.3.x |
Resolution: | → fixed |
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Good catch ! Fixed by r3755 in trunk and r3756 in 3.3 branch It surely reveals poor automated testing .. :(