D9166: Remove double lookups

David Faure noreply at phabricator.kde.org
Mon Dec 4 12:36:17 UTC 2017


dfaure accepted this revision.
dfaure added a comment.
This revision is now accepted and ready to land.


  I expect this map to be very small, and creating a null KEmoticonsTheme allocates a d pointer, so I'm not 100% sure this is faster.
  
  But OK, I'm allergic to double lookups and people tend to reuse constructs they saw, so I still prefer the code after the change than before ;)

REPOSITORY
  R301 KEmoticons

BRANCH
  remove_double_loop

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

To: mlaurent, dfaure
Cc: #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171204/c0e0fac3/attachment.html>


More information about the Kde-frameworks-devel mailing list