Do not validate the geometry if it is known to be valid - save some time

wonder-sk · wonder-sk · commit 8422a9da4f54 · 2011-11-11T13:07:30.000-03:00
diff --git a/src/app/qgsmaptoolnodetool.cpp b/src/app/qgsmaptoolnodetool.cpp
@@ -954,11 +954,17 @@ void SelectionFeature::setSelectedFeature( QgsFeatureId featureId,
 
 void SelectionFeature::validateGeometry( QgsGeometry *g )
 {
-  QgsDebugMsg( "validating geometry" );
-
   if ( g == NULL )
     g = mFeature->geometry();
 
+  if ( g->isGeosValid() )
+  {
+    QgsDebugMsg( "geometry is valid - not validating." );
+    return;
+  }
+
+  QgsDebugMsg( "validating geometry" );
+
   g->validateGeometry( mGeomErrors );
 
   while ( !mGeomErrorMarkers.isEmpty() )
