D28538: Use more UDSEntry::reserve() calls to avoid reallocs on multiple inserts
Méven Car
noreply at phabricator.kde.org
Fri Apr 3 15:28:29 BST 2020
meven accepted this revision.
meven added a comment.
This revision is now accepted and ready to land.
Seems hard to miss cases like for `// +1 for UDS_DISPLAY_NAME` but I will trust you did enough review to be confident for all those cases.
A missing one will just cause an allocation anyway which is what this is fixing in the first place, so no arm done basically.
REPOSITORY
R320 KIO Extras
BRANCH
usemoreudsentryreserve
REVISION DETAIL
https://phabricator.kde.org/D28538
To: kossebau, #frameworks, sitter, meven
Cc: kde-frameworks-devel, kfm-devel, nikolaik, pberestov, iasensio, fprice, LeGast00n, cblack, fbampaloukas, alexde, GB_2, Codezela, feverfew, meven, michaelh, spoorun, navarromorales, firef, ngraham, andrebarros, bruns, emmanuelp, mikesomov
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20200403/e18cd901/attachment.htm>
More information about the kfm-devel
mailing list