Lazily build index in IndexedPointInAreaLocator

libgeos · Mar 10, 2020 · 6b12bcc · 6b12bcc
1 parent 17110ec
commit 6b12bcc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/NEWS b/NEWS
@@ -12,6 +12,7 @@ Changes in 3.8.1
     (#1013, Dan Baston)
   - Fix missing vtable for LineString and CoordinateArraySequenceFactory
     (#299 and #1016, Evgen Bodunov)
+  - Reduce performance regression in GEOSBuffer (#1020)
 
 Changes in 3.8.0
 2019-10-10

diff --git a/src/algorithm/locate/IndexedPointInAreaLocator.cpp b/src/algorithm/locate/IndexedPointInAreaLocator.cpp
@@ -96,13 +96,15 @@ IndexedPointInAreaLocator::IndexedPointInAreaLocator(const geom::Geometry& g)
             &&	areaGeomId != typeid(geom::LinearRing)) {
         throw util::IllegalArgumentException("Argument must be Polygonal or LinearRing");
     }
-
-    buildIndex(areaGeom);
 }
 
 geom::Location
 IndexedPointInAreaLocator::locate(const geom::Coordinate* /*const*/ p)
 {
+    if (index == nullptr) {
+        buildIndex(areaGeom);
+    }
+
     algorithm::RayCrossingCounter rcc(*p);
 
     IndexedPointInAreaLocator::SegmentVisitor visitor(&rcc);