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

List:       kde-commits
Subject:    Re: KDE_3_3_BRANCH: kdepim/certmanager/lib/ui
From:       André_Wöbbeking <Woebbeking () onlinehome ! de>
Date:       2004-09-27 21:56:10
Message-ID: 200409272356.10655.Woebbeking () onlinehome ! de
[Download RAW message or body]

On Monday 27 September 2004 21:45, Marc Mutz wrote:
> CVS commit by mutz:
>
> Fix double-delete bug (aegypten2 issue #240). This is not a complete
> solution, that would involve overriding e.g. takeItem(), too. But
> I'll leave that to Qt4, whose Model/View abstraction will take care
> of that in a complete way.
>
>
>   M +18 -0     keylistview.cpp   1.24.2.1
>   M +3 -0      keylistview.h   1.20.2.1
>
>
> --- kdepim/certmanager/lib/ui/keylistview.cpp  #1.24:1.24.2.1
> @@ -229,4 +229,17 @@ void Kleo::KeyListView::clear() {
>  }
>
> +void Kleo::KeyListView::deregisterItem( const KeyListViewItem * item
> ) { +  if ( !item )
> +    return;
> +  std::map<QCString,KeyListViewItem*>::iterator it
> +    = d->itemMap.find( item->key().subkey(0).fingerprint() );
> +  if ( it == d->itemMap.end() )
> +    return;
> +  Q_ASSERT( it->second != item );

shouldn't this be == ?


André

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