Skip to content

Commit a59442f

Browse files
Remove restrictive validity check in caminfo (#5553)
* Removed preemptive validity check for polygon * Removed unused import * Updated changelog
1 parent 1a84260 commit a59442f

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@ release.
4141

4242

4343
### Fixed
44+
- Fixed a bug in PolygonTools in which the program exited before attempting to fix an invalid Polygon [#5520](https://github.com/DOI-USGS/ISIS3/issues/5520)
4445
- Fixed a bug in QVIEW's Stretch tool where the default min/max type was not an available option [#5289](https://github.com/DOI-USGS/ISIS3/issues/5289)
4546
- Fixed a bug in QVIEW where images would double load if loaded from the commandline [#5505](https://github.com/DOI-USGS/ISIS3/pull/5505)
4647

isis/src/base/objs/PolygonTools/PolygonTools.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ namespace Isis {
104104
geos::geom::Polygon *newPoly = globalFactory->createPolygon(
105105
globalFactory->createLinearRing(*xycoords), std::move(holes)).release();
106106

107-
if(newPoly->isValid() && !newPoly->isEmpty() && newPoly->getArea() > 1.0e-14) {
107+
if(!newPoly->isEmpty() && newPoly->getArea() > 1.0e-14) {
108108
xyPolys->push_back(newPoly);
109109
}
110110
else {
@@ -132,7 +132,6 @@ namespace Isis {
132132
throw IException(IException::Programmer, msg, _FILEINFO_);
133133
}
134134
}
135-
136135
} // end else
137136
}
138137

0 commit comments

Comments
 (0)