QMutableMapIterator uses map in destructor, use foreach.

Change-Id: Ie8737acaea614fdda66cb7642eaed21be41cae64
Reviewed-by: Peter Kümmel <syntheticpp@gmx.net>
Reviewed-by: Eike Ziller <eike.ziller@nokia.com>

src/plugins/coreplugin/sidebar.cpp

@@ -111,12 +111,9 @@ SideBar::SideBar(QList<SideBarItem*> itemList,
 
 SideBar::~SideBar()
 {
-    QMutableMapIterator<QString, QWeakPointer<SideBarItem> > iter(d->m_itemMap);
-    while(iter.hasNext()) {
-        iter.next();
-        if (!iter.value().isNull())
-            delete iter.value().data();
-    }
+    foreach (const QWeakPointer<SideBarItem> &i, d->m_itemMap)
+        if (!i.isNull())
+            delete i.data();
     delete d;
 }