[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Re: playground-libs/libkvkontakte has moved to kdereview
From:       Albert Astals Cid <aacid () kde ! org>
Date:       2011-08-15 18:22:03
Message-ID: 10105629.cUhEgESNrJ () bluebox
[Download RAW message or body]

A Dilluns, 15 d'agost de 2011, Alexander Potashev vĂ reu escriure:
> 2011/8/15 Albert Astals Cid <aacid@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

[prev in list] [next in list] [prev in thread] [next in thread]