Topology checker: No double-delete of rubberbands (Fix #8095)

m-kuhn · m-kuhn · commit ce1a9a821db2 · 2013-07-02T13:40:52.000+02:00
diff --git a/src/plugins/topology/checkDock.cpp b/src/plugins/topology/checkDock.cpp
@@ -150,14 +150,8 @@ void checkDock::deleteErrors()
   mErrorList.clear();
   mErrorListModel->resetModel();
 
-  QList<QgsRubberBand*>::const_iterator rit;
-
-  for ( rit = mRbErrorMarkers.begin(); rit != mRbErrorMarkers.end(); ++rit )
-  {
-    QgsRubberBand* rb = *rit;
-    rb->reset();
-    delete rb;
-  }
+  qDeleteAll( mRbErrorMarkers );
+  mRbErrorMarkers.clear();
 }
 
 void checkDock::parseErrorListByLayer( QString layerId )
@@ -397,14 +391,7 @@ void checkDock::validate( ValidateType type )
 {
   mErrorList.clear();
 
-  QList<QgsRubberBand*>::const_iterator it;
-  for ( it = mRbErrorMarkers.begin(); it != mRbErrorMarkers.end(); ++it )
-  {
-    QgsRubberBand* rb = *it;
-    rb->reset();
-    delete rb;
-  }
-
+  qDeleteAll( mRbErrorMarkers );
   mRbErrorMarkers.clear();
 
   runTests( type );
