KToolBar api cleanup

David Faure faure at kde.org
Mon Oct 15 20:33:37 BST 2007


KConfigGroup usage in KToolBar is weird, it has kde3 logic like "if group name is empty
then set it to some group name made up by ktoolbar".

I checked KDE+extragear+playground with lxr and with gcc:
1) the few calls to saveSettings+applySettings do pass a group name, which is of course the only
sensible thing to do with KConfigGroup;
2) protected settingsGroup() was only used internally (and only for the fallback group thing) -> removed
3) protected applyAppearanceSettings() is only used internally -> made private

Patch attached (normally I would have just committed, but well, let's play by the rules ;)

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ktoolbar.diff
Type: text/x-diff
Size: 3559 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20071015/d2e6f7b1/attachment.diff>


More information about the kde-core-devel mailing list