tests: simplify next_block() function in feature_block #18366

pull jnewbery wants to merge 1 commits into bitcoin:master from jnewbery:2020-03-feature-block-next-block changing 1 files +7 −7

jnewbery commented at 1:46 PM on March 17, 2020: member

The solve parameter is unnecessary. Remove it and add comments.
tests: simplify next_block() function in feature_block
The solve parameter is unnecessary. Remove it and add comments.
612a931d1a
jnewbery commented at 1:47 PM on March 17, 2020: member

follow-up from #18355 @MarcoFalke @TheQuantumPhysicist
MarcoFalke commented at 1:59 PM on March 17, 2020: member

ACK 612a931d1a3ac1678d02aed30c48fd25ccd113db

Fine with me
TheQuantumPhysicist commented at 2:07 PM on March 17, 2020: contributor

ACK 612a931

Looks good. Thanks for improving it :smile:
DrahtBot added the label Tests on Mar 17, 2020
practicalswift commented at 2:22 PM on March 17, 2020: contributor

ACK 612a931d1a3ac1678d02aed30c48fd25ccd113db -- simpler is better and patch looks correct :)
MarcoFalke merged this on Mar 17, 2020
MarcoFalke closed this on Mar 17, 2020
Fabcien referenced this in commit f68b71ad54 on Jan 8, 2021
DrahtBot locked this on Feb 15, 2022

Contributors

Labels