On Thu, Jan 3, 2013 at 9:52 PM, Albert Astals Cid <[email protected]> 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 <[email protected]> > 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
