[PATCH] KCoreConfigSkeleton

[Aaron J. Seigo]

On Saturday 24 November 2007, Oswald Buddenhagen wrote:
> while reviewing internal kconfig docu i found that the goodies of
> KConfigXT seem to be reserved to gui apps - no KCoreConfigSkeleton.
> after a short consultation with thiago to confirm that it can be done i
> just went ahead and factored out KCoreConfigSkeleton from
> KConfigSkeleton.

there's an additional problem now that we have nested groups as well: config 
objects may not start at the root of the file. right now KConfigSkeleton 
assumes it owns the entire file and there is no way to make it start from a 
given sub-group that i can find since it takes a KSharedConfig::Ptr in the 
constructors ...

then there are methods like writeConfig(KConfig * config) ... shouldn't those 
take a KConfigBase* isntead? that would at least make it possible to extend 
to allow applying a KConfigSkeleton to a config object starting at a non-root 
group.

as for the use case, i need this in plasma, for instance, where applets may 
provide a KConfigXT xml file for their settings but share a config file 
(using nested groups) with other applets.

i also noted that there are virtual methods that are inline in the header. 
shouldn't those go into the implementation?

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Trolltech
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071124/f0581f8e/attachment.sig>