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

List:       kde-commits
Subject:    Re: KDE/kdebase/kicker/kicker/core
From:       André_Wöbbeking <Woebbeking () onlinehome ! de>
Date:       2005-08-21 17:29:10
Message-ID: 200508211929.10678.Woebbeking () onlinehome ! de
[Download RAW message or body]

On Saturday 20 August 2005 22:01, Matt Broadstone wrote:
> SVN commit 451474 by mbroadst:
>
> more cleanups, changed a lot of STL style iterators to use foreach.
> readability is next to godliness?
>
>  M  +2 -0      container_extension.cpp
>  M  +34 -44    extensionmanager.cpp
>  M  +1 -1      pluginmanager.cpp
>
>
> --- trunk/KDE/kdebase/kicker/kicker/core/extensionmanager.cpp
> #451473:451474 @@ -73,7 +73,7 @@
>  {
>      while (!_containers.isEmpty())
>      {
> -            delete _containers.takeFirst();
> +        delete _containers.takeFirst();
>      }

What do you thing about:

qDeleteAll(_containers);
_containers.clear();
[prev in list] [next in list] [prev in thread] [next in thread]