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