[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/KDE/4.5/kdelibs/kdeui/kernel
From:       Luboš Luňák <l.lunak () kde ! org>
Date:       2010-07-29 14:34:23
Message-ID: 20100729143423.74C90AC7A9 () svn ! kde ! org
[Download RAW message or body]

SVN commit 1156692 by lunakl:

Make sure the iterator goes out of scope before possibly deleting its list.
BUG: 243540



 M  +3 -2      kapplication.cpp  


--- branches/KDE/4.5/kdelibs/kdeui/kernel/kapplication.cpp #1156691:1156692
@@ -271,8 +271,9 @@
         return;
     // removeAll doesn't work, creating QWeakPointer to something that's about to be deleted aborts
     // x11Filter->removeAll( const_cast< QWidget* >( filter ));
-    QMutableListIterator< QWeakPointer< QWidget > > it( *x11Filter );
-    while( it.hasNext()) {
+    for( QMutableListIterator< QWeakPointer< QWidget > > it( *x11Filter );
+         it.hasNext();
+         ) {
         QWidget* w = it.next().data();
         if( w == filter || w == NULL )
             it.remove();
[prev in list] [next in list] [prev in thread] [next in thread]