Moving libkfacebook to extragear
Albert Astals Cid
aacid at kde.org
Thu Jan 3 20:52:28 GMT 2013
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?
Cheers,
Albert
>
> > You probably also want to add Q_DISABLE_COPY to most of the classes with
> > d-pointers, otherwise the default constructor and operator= will do
> > something
> > that you don't really want.
>
> Fixed.
>
> [1] - http://techbase.kde.org/Policies/Library_Code_Policy#D-Pointers
> [2] -
> http://techbase.kde.org/Policies/Library_Code_Policy/Shared_D-Pointer_Exampl
> e
>
> Cheers
More information about the kde-core-devel
mailing list