getting rid of unneccessary test for <default>

David Faure faure at kde.org
Fri Nov 23 00:03:12 GMT 2007


On Friday 23 November 2007, Thomas Braxton wrote:
> While trying to remove <default> as the default group name in KConfig,
> I found a few places that are testing for <default> in the list
> returned from KConfig::groupList(). groupList already removes
> <default> and any empty names from the list so any tests for this are
> superfluous. OK to commit?

Looks good to me.

Well, patches with unit tests always do look good anyway :-)

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).




More information about the kde-core-devel mailing list