Fixed a segfault with simplify tool (#2450)

git-svn-id: http://svn.osgeo.org/qgis/trunk/qgis@13176 c8812cc2-4d05-0410-92ff-de0c093fc19c
qgis · Mar 28, 2010 · 50298dd · 50298dd
1 parent b8c1bc8
commit 50298dd
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/app/qgsmaptoolsimplify.cpp b/src/app/qgsmaptoolsimplify.cpp
@@ -259,7 +259,7 @@ void QgsMapToolSimplify::canvasPressEvent( QMouseEvent * e )
   vlayer->select( QgsAttributeList(), selectRect, true );
 
   QgsGeometry* geometry = QgsGeometry::fromPoint( layerCoords );
-  double minDistance = 10000000;
+  double minDistance = DBL_MAX;
   double currentDistance;
   QgsFeature f;
 
@@ -274,16 +274,18 @@ void QgsMapToolSimplify::canvasPressEvent( QMouseEvent * e )
       mSelectedFeature = f;
     }
   }
-  if ( mSelectedFeature.geometry()->isMultipart() )
-  {
-    QMessageBox::critical( 0, tr( "Unsupported operation" ), tr( "Multipart features are not supported for simplification." ) );
-    return;
-  }
+
   // delete previous rubberband (if any)
   removeRubberBand();
 
   if ( mSelectedFeature.isValid() )
   {
+    if ( mSelectedFeature.geometry()->isMultipart() )
+    {
+      QMessageBox::critical( 0, tr( "Unsupported operation" ), tr( "Multipart features are not supported for simplification." ) );
+      return;
+    }
+
     mRubberBand = new QgsRubberBand( mCanvas );
     mRubberBand->setToGeometry( mSelectedFeature.geometry(), false );
     mRubberBand->setColor( Qt::red );