diff options
| author | fanquake <fanquake@gmail.com> | 2021-12-28 20:21:23 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2021-12-28 20:21:38 +0800 |
| commit | 45249e70d993f5162c9ae47d6f0755974228425c (patch) | |
| tree | 29869e9bf33e7f7fd51f09ab8a4ada285ffa5b33 /src | |
| parent | d3582f2d3bfebb32316aa3974f6f27db20a610f5 (diff) | |
| parent | fa5d0b6ecda384b591fff42b4072fa7d02525975 (diff) | |
| download | bitcoin-45249e70d993f5162c9ae47d6f0755974228425c.tar.xz | |
Merge bitcoin/bitcoin#23881: doc: Remove TODO from block fuzz test
fa5d0b6ecda384b591fff42b4072fa7d02525975 doc: Remove TODO from block fuzz test (MarcoFalke)
Pull request description:
I don't see a reason to fix the TODO, as all call sites assume at least one tx in `vtx`. So remove it.
ACKs for top commit:
fanquake:
ACK fa5d0b6ecda384b591fff42b4072fa7d02525975
Tree-SHA512: 8fe03a5bdebf4d6dad00f410a1dc6449c485a4d61afb79ebcaa221489cde2a465fea981742fde130238ebed6d2f1f032aa4976726ee07b23fe7fb745b2728d46
Diffstat (limited to 'src')
| -rw-r--r-- | src/test/fuzz/block.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/test/fuzz/block.cpp b/src/test/fuzz/block.cpp index 65a33de4b4..c750f7ee3a 100644 --- a/src/test/fuzz/block.cpp +++ b/src/test/fuzz/block.cpp @@ -58,8 +58,6 @@ FUZZ_TARGET_INIT(block, initialize_block) (void)block.ToString(); (void)BlockMerkleRoot(block); if (!block.vtx.empty()) { - // TODO: Avoid array index out of bounds error in BlockWitnessMerkleRoot - // when block.vtx.empty(). (void)BlockWitnessMerkleRoot(block); } (void)GetBlockWeight(block); |