This has been unused since it was introduced in 872fee3fccc8b33b9af0a401b5f85ac5504b57eb.
net: Remove SetMaxOutboundTimeframe() #20416
pull jnewbery wants to merge 1 commits into bitcoin:master from jnewbery:2020-11-remove-set-outbound-timeframe changing 3 files +3 −20-
jnewbery commented at 11:16 AM on November 18, 2020: member
-
9b5ada5d58
[net] Remove SetMaxOutboundTimeframe()
This has been unused since it was introduced in 872fee3fccc8b33b9af0a401b5f85ac5504b57eb.
-
ajtowns commented at 11:22 AM on November 18, 2020: member
SetMaxOutboundTarget()is the same. This makes the fuzz test coverage worse I think, since these functions are currently the only way for the corresponding variables to get a value; and the benefit of not having code that's only used by tests is pretty minimal. So Concept NACK without corresponding improvements to the tests for me. -
-
- jnewbery closed this on Nov 18, 2020
- DrahtBot locked this on Feb 15, 2022
