Fix fill ring tool used with advanced digitising crashes QGIS

(fix #13355) (cherry-picked from aa4d65d)
qgis · Nov 18, 2015 · f98fd33 · f98fd33
1 parent a1479a7
commit f98fd33
Showing 1 changed file with 6 additions and 17 deletions.
diff --git a/src/app/qgsmaptoolfillring.cpp b/src/app/qgsmaptoolfillring.cpp
@@ -122,25 +122,14 @@ void QgsMapToolFillRing::canvasMapReleaseEvent( QgsMapMouseEvent * e )
       yMin = std::numeric_limits<double>::max();
       yMax = -std::numeric_limits<double>::max();
 
-      for ( QList<QgsPoint>::const_iterator it = points().constBegin(); it != points().constEnd(); ++it )
+      Q_FOREACH ( const QgsPoint& point, points() )
       {
-        if ( it->x() < xMin )
+        xMin = qMin( xMin, point.x() );
-        {
+        xMax = qMax( xMax, point.x() );
-          xMin = it->x();
+        yMin = qMin( yMin, point.y() );
-        }
+        yMax = qMax( yMax, point.y() );
-        if ( it->x() > xMax )
-        {
-          xMax = it->x();
-        }
-        if ( it->y() < yMin )
-        {
-          yMin = it->y();
-        }
-        if ( it->y() > yMax )
-        {
-          yMax = it->y();
-        }
       }
+
       bBox.setXMinimum( xMin );
       bBox.setYMinimum( yMin );
       bBox.setXMaximum( xMax );