Opened 11 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 .. :(