Adds a return as no need to stay in this function any longer once it has been decided that direct-fetch will not occur, and improves code readability (IMHO).
[Net] Refactor: exit when no direct fetch #9418
pull rebroad wants to merge 2 commits into bitcoin:master from rebroad:ExitWhenNoDirectFetch changing 1 files +26 −26-
rebroad commented at 6:58 AM on December 24, 2016: contributor
-
Exit if deciding not to direct fetch e74f75e5ff
-
Reduce indentation for previous conditional section 21f6333b61
- rebroad force-pushed on Dec 24, 2016
- rebroad renamed this:
Exit when no direct fetch
Trivial refactor: exit when no direct fetch
on Dec 25, 2016 - fanquake added the label P2P on Dec 26, 2016
-
fanquake commented at 10:23 AM on December 26, 2016: member
Please squash these changes into a single commit.
- fanquake renamed this:
Trivial refactor: exit when no direct fetch
[Net] Refactor: exit when no direct fetch
on Jan 21, 2017 -
fanquake commented at 1:09 AM on January 21, 2017: member
Needs a rebase.
-
kallewoof commented at 2:44 AM on February 10, 2017: member
utACK + verified code with indentation changes was the same.
-
fanquake commented at 9:59 AM on April 2, 2017: member
Closing, nit needs addressing and still needs a rebase. Feel free to reopen when it's ready for re-review.
- fanquake closed this on Apr 2, 2017
- DrahtBot locked this on Sep 8, 2021