On Thursday 14 June 2012 22:13:28 Jignesh Kakadiya wrote:
> Updated patch.

This will call db.mimeTypeForName() every time, in the unlikely where the 
mimetype is missing, so my suggestion was
  defaultFolderIcon = "unknown";
rather than
  return DesktopIcon( "unknown", _size, _state );
(just under the kWarning)

Looks good otherwise, please adjust and commit.

-- 
David Faure, fa...@kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to