D14856: Search: Add workaround for missing icons in Gnome icon-theme

David Faure noreply at phabricator.kde.org
Wed Aug 15 20:03:26 BST 2018


dfaure added a comment.


  I don't think so, because QIcon::fromTheme just creates an icon engine. The actual lookup happens when the first call to isNull or paint (etc) is made. None of which are going to be done on the fallback icon if the main icon can be found.
  
  So this looks fine to me, unless a profiler or strace says otherwise.

REPOSITORY
  R39 KTextEditor

REVISION DETAIL
  https://phabricator.kde.org/D14856

To: sars, #kate, dhaumann, broulik
Cc: dfaure, bruns, broulik, dhaumann, kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, demsking, cullmann, sars
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20180815/5895f705/attachment.html>


More information about the KWrite-Devel mailing list