playground-libs/libkvkontakte has moved to kdereview

[Alexander Potashev]

2011/8/15 Albert Astals Cid <aacid at kde.org>:
> A Dilluns, 15 d'agost de 2011, Alexander Potashev vàreu escriure:
>> 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?

I thought about "QMap<QString, QVariant>" as of an easier solution
since you won't need to declare the NoteInfoPrivate class.

>> 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.

So, the NoteInfoPrivate class may not have any declaration (except for
the forward declaration) until it will be necessary, right?


-- 
Alexander Potashev