Moving libkfacebook to extragear

Martin Klapetek martin.klapetek at gmail.com
Thu Jan 3 22:12:40 GMT 2013


On Thu, Jan 3, 2013 at 9:52 PM, Albert Astals Cid <aacid at kde.org> wrote:

> El Dimecres, 2 de gener de 2013, a les 15:31:30, Martin Klapetek va
> escriure:
> > On Sun, Dec 30, 2012 at 6:34 PM, Albert Astals Cid <aacid at kde.org>
> wrote:
> > > AttendeeInfo misses a d_ptr?
> >
> > Are you deleting the d_ptr? Maybe i'm missing something but for example
> you
> >
> > > don't seem to delete the d_ptr of PagedListJob, no?
> >
> > I reworked the dpointers a bit, classes deriving directly from
> Facebook*Job
> > are now extending its dpointer, which is then deleted in the baseclass
> > (FacebookJob). All other dpointers are also properly deleted.
> > Partly-related - our d-pointer tutorials don't delete it either [1][2].
>
> The QScopedPointer will do the trick, no?
>

Right, that's the trick. Well it's quite confusing for newbies, it could
use better explaining. I'll try to extend the text a bit.

Anyways, anything else that should be fixed in the library?
-- 
Martin Klapetek | KDE Developer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130103/15ea4f41/attachment.htm>


More information about the kde-core-devel mailing list