D9236: Fill lists of default-constructed types directly, not append any by value
Friedrich W. H. Kossebau
noreply at phabricator.kde.org
Wed Dec 13 15:17:08 UTC 2017
kossebau added a comment.
Thanks for detailed reply @aaronpuchert Does not conflict with what I heard elsewhere, so big picture got more clear to me.
So my take-away is: default to one-pass for best practice, unless actual measurements are done.
So will discard this patch or, well, reduce it to the changes which are still one-pass afterwards (the ones where no further modification besides default constructor is done).
REPOSITORY
R32 KDevelop
REVISION DETAIL
https://phabricator.kde.org/D9236
To: kossebau, #kdevelop
Cc: aaronpuchert, brauch, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20171213/f7b24518/attachment.html>
More information about the KDevelop-devel
mailing list