Change BlockBuilderImpl’s m_excluded_clusters to unordered set since ordering is not used.
Change the set to a set of sequence numbers for a modest stability increase under fuzz testing.
Change BlockBuilderImpl’s m_excluded_clusters to unordered set since ordering is not used.
Change the set to a set of sequence numbers for a modest stability increase under fuzz testing.
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/33469.
See the guideline for information on the review process.
| Type | Reviewers |
|---|---|
| ACK | sipa, marcofleon, glozow |
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
No conflicts as of last run.
Change BlockBuilderImpl's m_excluded_clusters to unordered
set since ordering is not used.
Change the set to a set of sequence numbers for a modest
stability increase under fuzz testing.
ACK 9b43428c96872f0fbbbab4c066c6010fc18c6cc4
Ran 10M fuzz iterations. Must be good.
tACK 9b43428c96872f0fbbbab4c066c6010fc18c6cc4
AFL++ now showing 94.5% stability for the txgraph target with this change.
edit: For reference, it was 94% previously.