D5358: KMime: reduce memory allocations by using QLatin1String for appending to QString

David Faure noreply at phabricator.kde.org
Sun Apr 9 10:07:46 BST 2017


dfaure abandoned this revision.
dfaure added a comment.


  heaptrack shows that this actually makes things worse (same number of allocations, but on top of that it leads to more pre-allocated data in QString (note that the destination is empty, at least in parseAtom/parseToken - I just changed to '=' instead of "+=" to make that clearer).

REPOSITORY
  R180 PIM: KMime

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

To: dfaure
Cc: #kde_pim, dvasin, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20170409/740cefc4/attachment.html>


More information about the kde-pim mailing list