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