Similarly to #29458, we're preallocating the result string based on the input string's length. The methods were already covered by tests.
refactor: Reserve memory for ToLower/ToUpper conversions #29606
pull l0rinc wants to merge 1 commits into bitcoin:master from l0rinc:paplorinc/reserve-case-conversion changing 1 files +2 −0-
l0rinc commented at 10:07 PM on March 8, 2024: contributor
-
Reserve memory for ToLower/ToUpper conversions 6f2f4a4d09
-
DrahtBot commented at 10:07 PM on March 8, 2024: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage
For detailed information about the code coverage, see the test coverage report.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK tdb3, Empact, maflcko, stickies-v, achow101 If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
- DrahtBot added the label Refactoring on Mar 8, 2024
-
tdb3 commented at 12:44 AM on March 9, 2024: contributor
ACK for 6f2f4a4d096a3b261258c8cdd96cca532988d1d3 Cloned, built, ran all unit and functional tests (all passed).
- jess2505 approved
- Empact approved
- Empact approved
-
Empact commented at 5:05 AM on March 9, 2024: contributor
-
maflcko commented at 9:55 AM on March 11, 2024: member
lgtm ACK 6f2f4a4d096a3b261258c8cdd96cca532988d1d3
- stickies-v approved
-
stickies-v commented at 5:12 PM on March 12, 2024: contributor
ACK 6f2f4a4d096a3b261258c8cdd96cca532988d1d3
-
achow101 commented at 11:29 AM on March 13, 2024: member
ACK 6f2f4a4d096a3b261258c8cdd96cca532988d1d3
- achow101 merged this on Mar 13, 2024
- achow101 closed this on Mar 13, 2024
- luke-jr referenced this in commit 2435c39a6c on Mar 14, 2024
- l0rinc deleted the branch on May 11, 2024
- PastaPastaPasta referenced this in commit 2421c19acc on Oct 24, 2024
- PastaPastaPasta referenced this in commit fb4cb6403f on Oct 24, 2024
- PastaPastaPasta referenced this in commit d0e15d59f8 on Oct 24, 2024
- PastaPastaPasta referenced this in commit aaccc9ea51 on Oct 24, 2024
- achow101 referenced this in commit 5a4bc5c036 on Nov 26, 2024
- bitcoin locked this on May 11, 2025