Clarify prevector::erase and avoid swap-to-clear #10534
pull sipa wants to merge 1 commits into bitcoin:master from sipa:clarify_erase changing 2 files +9 −2-
sipa commented at 7:18 pm on June 5, 2017: member
-
laanwj commented at 9:24 am on June 6, 2017: memberutACK https://github.com/bitcoin/bitcoin/pull/10534/commits/38a52358f9f145d16f1e4ebab0266740cee86868, thanks for adding the clarifying comment.
-
laanwj added the label Utils and libraries on Jun 6, 2017
-
in src/prevector.h:390 in 38a52358f9 outdated
386@@ -387,6 +387,12 @@ class prevector { 387 } 388 389 iterator erase(iterator first, iterator last) { 390+ // Erase is not allowed the change the object's capacity. That means
sdaftuar commented at 3:59 pm on June 7, 2017:nit: “to” change the object’s capacity.sdaftuar approvedsdaftuar commented at 4:01 pm on June 7, 2017: memberLooks good, thanks for adding. One typo in the comment.TheBlueMatt commented at 5:21 pm on June 7, 2017: memberutACK 38a52358f9f145d16f1e4ebab0266740cee86868 +/- grammarClarify prevector::erase and avoid swap-to-clear e241a63c23sipa force-pushed on Jun 9, 2017sipa commented at 8:23 pm on June 9, 2017: memberFixed.laanwj merged this on Jun 13, 2017laanwj closed this on Jun 13, 2017
laanwj referenced this in commit a514ac3dcb on Jun 13, 2017sipa deleted the branch on Jun 23, 2017PastaPastaPasta referenced this in commit c7c9ab5452 on Jul 5, 2019PastaPastaPasta referenced this in commit 7065f2fc8a on Jul 5, 2019PastaPastaPasta referenced this in commit 6bb54bd521 on Jul 6, 2019PastaPastaPasta referenced this in commit 98d63201c7 on Jul 8, 2019PastaPastaPasta referenced this in commit 26915a9173 on Jul 9, 2019PastaPastaPasta referenced this in commit c1c24205b7 on Jul 9, 2019barrystyle referenced this in commit c3126c1973 on Jan 22, 2020Fuzzbawls referenced this in commit 8dfc4806f7 on May 19, 2020DrahtBot locked this on Sep 8, 2021
sipa
laanwj
sdaftuar
TheBlueMatt
Labels
Utils/log/libs
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-06-17 19:13 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-06-17 19:13 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me