Attached is a patch that cleans up some of the code in kdecore/config and adds a bunch of assertions to KConfigGroup. No feature changes. ok to commit?<div><br> </div><div>A couple of other things I would like to ask about:
</div><div>1) do we want to support an empty group name? We've never supported it before, and I really don't see a use for it, but it shouldn't be too hard to add support for.</div><div>1a) If we do, do we want to support empty sub-group names?
</div><div>2) this one depends on #1, should the name of the default group be exposed? It's really just an implementation detail that I think should never be returned to the user, unless we decide to support empty group names.
</div><div>3) should the translated entries from other locales be removed from the entryMap? IIRC the reason I left them in was because of an idea to allow changing of the locale without having to call reparseConfiguration, but I now think that wasn't a good idea and it was never fully implemented, so they shouldn't be in the entryMap. 
<div></div></div><div><br class="webkit-block-placeholder"></div><div>Thomas</div><div><br class="webkit-block-placeholder"></div>