Hidden KDED desktop file crashing systemsettings - where to fix?

David Faure faure at kde.org
Wed Nov 3 15:22:50 GMT 2010


On Wednesday 03 November 2010, Jonathan Marten wrote:
> Possibly the "if ( KDesktopFile::isDesktopFile( ... ) )" is redundant,
> but I've tried not to change the structure of the original code too
> much.  If anyone thinks this could be cut out, then I'll happily do that.

Yes, it can be cut out, only .desktop files are known as services in ksycoca.

Patch looks good otherwise.

-- 
David Faure, faure at kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. Konqueror (http://www.konqueror.org).




More information about the kde-core-devel mailing list