broulik added inline comments. INLINE COMMENTS
> kcmodulestateprobe.h:21 > + > +#ifndef KCMODULEDEFAULT_H > +#define KCMODULEDEFAULT_H `KCMODULESTATEPROBE_H` > kcmodulestateprobe.h:25 > +#include <QObject> > +#include <QVariantList> > +#include <kcmutils_export.h> Forward-declare > kcmodulestateprobe.h:39 > + virtual bool isDefaults() const = 0; > +}; > + Please add a `virtual_hook` so we can extend this class in the future without breaking ABI should we have the need to extract more data out of a settings module: virtual void virtual_hook(int id, void *data) REPOSITORY R295 KCMUtils REVISION DETAIL https://phabricator.kde.org/D28460 To: bport, #plasma, ervin Cc: broulik, kde-frameworks-devel, LeGast00n, cblack, GB_2, michaelh, ngraham, bruns
