Some small refactors to remove unused and redundant fuzz code
fuzz: Misc refactor #21553
pull MarcoFalke wants to merge 3 commits into bitcoin:master from MarcoFalke:2103-fuzzCleanups changing 7 files +28 −26-
MarcoFalke commented at 7:45 AM on March 30, 2021: member
-
fuzz: [refactor] Use ConsumeScript in signature_checker fuzz target fa98f3f66e
- MarcoFalke added the label Refactoring on Mar 30, 2021
- MarcoFalke added the label Tests on Mar 30, 2021
- MarcoFalke force-pushed on Mar 30, 2021
-
practicalswift commented at 8:01 AM on March 30, 2021: contributor
cr ACK faa8b2de6b9749d33b4f11155a29d7094f23a144: patch looks correct and touches only
src/test/fuzz/(+ one addition tosrc/test/util/)Thanks for cleaning up!
-
fuzz: Removed unused try-catch in coins_view eeee8f5be1
- MarcoFalke force-pushed on Mar 30, 2021
-
practicalswift commented at 8:42 AM on March 30, 2021: contributor
cr re-ACK faaf133e4ada4a12783e7395a17f207f7d700e09
-
fa4926cca6
fuzz: [refactor] Use IsValidFlagCombination in signature_checker fuzz target
Can be reviewed with --color-moved=dimmed-zebra
- MarcoFalke force-pushed on Mar 30, 2021
-
MarcoFalke commented at 9:57 AM on March 30, 2021: member
Sorry for the force push. Should be done now. No change in coverage:
-
practicalswift commented at 5:56 PM on March 30, 2021: contributor
cr re-ACK fa4926cca68db2433e579d9b4a642248a9314673
- MarcoFalke merged this on Mar 30, 2021
- MarcoFalke closed this on Mar 30, 2021
- sidhujag referenced this in commit 2afeea14d9 on Mar 30, 2021
- MarcoFalke deleted the branch on Apr 1, 2021
- MarcoFalke referenced this in commit 7d19c85f4a on May 19, 2021
- DrahtBot locked this on Aug 16, 2022
Contributors
Labels