Don't use GEOS to calculate length of points/linestrings

These calculations are so trivial that the expense of conversion to a geos representation is unwanted overhead
qgis · Feb 24, 2021 · ff17205 · ff17205
1 parent 4f29af5
commit ff17205
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/src/core/geometry/qgsgeometry.cpp b/src/core/geometry/qgsgeometry.cpp
@@ -1786,6 +1786,13 @@ double QgsGeometry::length() const
   {
     return -1.0;
   }
+
+  // avoid calling geos for trivial geometry calculations
+  if ( QgsWkbTypes::geometryType( d->geometry->wkbType() ) == QgsWkbTypes::PointGeometry || QgsWkbTypes::geometryType( d->geometry->wkbType() ) == QgsWkbTypes::LineGeometry )
+  {
+    return d->geometry->length();
+  }
+
   QgsGeos g( d->geometry.get() );
   mLastError.clear();
   return g.length( &mLastError );