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

List:       kmail-devel
Subject:    Re: [PATCH] UI abstraction proposal (partially only)
From:       Stephan Kulow <coolo () kde ! org>
Date:       2003-06-03 6:39:57
[Download RAW message or body]

On Tuesday 03 June 2003 00:00, Andreas Gungl wrote:

Something general. This code here:
> +  AddrBookGuiCallback *guiCallback = new AddrBookDefaultGuiCallback( this );
> +  if (KMAddrBookExternal::addVCard(mAddressee, guiCallback))
>      showButton(User1, false);
> +  delete guiCallback;

You better write it as
+ AddrBookGuiCallback guiCallback(this);
+ if (KMAddrBookExternal::addVCard(mAddressee, &guiCallback))
      showButton(User1, false);

Greetings, Stephan
_______________________________________________
KMail Developers mailing list
kmail@mail.kde.org
http://mail.kde.org/mailman/listinfo/kmail
[prev in list] [next in list] [prev in thread] [next in thread]