diff options
| author | Andrew Chow <github@achow101.com> | 2023-05-10 17:40:42 -0400 |
|---|---|---|
| committer | Andrew Chow <github@achow101.com> | 2023-05-10 17:50:42 -0400 |
| commit | 3ff67f77831474e342502e6a24be8a2ce5688e04 (patch) | |
| tree | bce3b70ba020fc71b0cddc3a974b78896364684e /src/test/validation_block_tests.cpp | |
| parent | e0a70c5b4f2c691e8d6b507c8ce879f0b0424254 (diff) | |
| parent | 72efc26439da9a1344a19569fb0cab01f82ae7d1 (diff) | |
Merge bitcoin/bitcoin#19690: util: improve FindByte() performance
72efc26439da9a1344a19569fb0cab01f82ae7d1 util: improve streams.h:FindByte() performance (Larry Ruane)
604df63f6c70b9692b067777ddb38d946ac0b2fc [bench] add streams findbyte (gzhao408)
Pull request description:
This PR is strictly a performance improvement; there is no functional change. The `CBufferedFile::FindByte()` method searches for the next occurrence of the given byte in the file. Currently, this is done by explicitly inspecting each byte in turn. This PR takes advantage of `std::find()` to do the same more efficiently, improving its CPU runtime by a factor of about 25 in typical use.
ACKs for top commit:
achow101:
re-ACK 72efc26439da9a1344a19569fb0cab01f82ae7d1
stickies-v:
re-ACK 72efc26439da9a1344a19569fb0cab01f82ae7d1
Tree-SHA512: ddf0bff335cc8aa34f911aa4e0558fa77ce35d963d602e4ab1c63090b4a386faf074548daf06ee829c7f2c760d06eed0125cf4c34e981c6129cea1804eb3b719
Diffstat (limited to 'src/test/validation_block_tests.cpp')
0 files changed, 0 insertions, 0 deletions