Moving libkfacebook to extragear

Albert Astals Cid aacid at kde.org
Wed Nov 7 19:28:19 GMT 2012


El Dimecres, 7 de novembre de 2012, a les 20:20:56, Martin Klapetek va 
escriure:
> On Wed, Nov 7, 2012 at 8:11 PM, Albert Astals Cid <aacid at kde.org> wrote:
> > El Dilluns, 5 de novembre de 2012, a les 22:59:43, Martin Klapetek va
> > 
> > escriure:
> > > On Mon, Nov 5, 2012 at 7:52 PM, Pino Toscano <pino at kde.org> wrote:
> > > > Some notes:
> > > > 
> > > > - as others have pointed out, public classes should be better use
> > > > d-pointers; for the jobs hierarchy, you could use a shared d-pointer,
> > > > see [1] for example
> > > 
> > > The data classes are finally done - all of them now use d-pointers and
> > 
> > none
> > 
> > > are qobject; I've added the qobject-parsers as Kevin suggested, please
> > 
> > give
> > 
> > > it a look.
> > 
> > You sure? I still see a lots of d-pointers missing (AllEventsListJob,
> > AllNotesListJob, etc).
> 
> Yes, I meant the data classes, those are the *Info classes. I'll see the
> rest over the weekend hopefully.

Ah, sorry

> 
> > Also you have some classes without the export thingie, like AttendeeInfo
> > or
> > the facebookTimeToKDateTime function
> 
> They are not to be exported, only for internal purposes.

Why are they in installed headers then?

Cheers,
  Albert

> 
> Cheers




More information about the kde-core-devel mailing list