playground-libs/libkvkontakte has moved to kdereview

Albert Astals Cid aacid at kde.org
Mon Aug 15 19:22:03 BST 2011


A Dilluns, 15 d'agost de 2011, Alexander Potashev vàreu escriure:
> 2011/8/15 Albert Astals Cid <aacid at kde.org>:
> > You do not use d pointers in any of your classes thus maintaining Binary
> > Compatibility is going to be almost impossible if you need to expand
> > them.
> How about adding a "QMap<QString, QVariant> m_ext;" to *Info classes,
> so that I can store additional variables there? Most (but not all)
> *Job classes are unlikely to be expanded later, because they perform
> very simple operations.

Why not simply use a d-pointer like it is explained in techbase?

> 
> If I'll add just a forward declaration like "class NoteInfoPrivate;"
> and a "NoteInfoPrivate *p;" into the NoteInfo class, will it be OK?

I guess you mean using a d-pointer, yes, that's the suggested way of dealing 
with this kind of issue.

> 
> > I'd also like if you used KCatalogLoader to load your translation
> > catalog.
> Done.

Great.

Albert

> 
> 
> --
> Alexander Potashev




More information about the kde-core-devel mailing list