incremental net cleanup 2 #4390

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:net_cleanup_incr_2 changing 1 files +2 −1

Diapolo commented at 6:20 PM on June 22, 2014: none

I'm going to split of some changes from #4348 to make review easier and hopefully merging faster :).
laanwj commented at 8:23 AM on June 23, 2014: member

Untested ACK
replace 3 separate calls to WSAGetLastError() with 1 6dc90ed8fa
Diapolo commented at 6:47 AM on June 24, 2014: none

@jgarzik Removed the commit you NACKed. Should be fine now.
BitcoinPullTester commented at 7:03 AM on June 24, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4390_6dc90ed8fa34db164fac956eed600b2ba66640a0/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
laanwj merged this on Jun 25, 2014
laanwj closed this on Jun 25, 2014
laanwj referenced this in commit 27383b9015 on Jun 25, 2014
Diapolo deleted the branch on Jun 26, 2014
reddink referenced this in commit 724ca4c8c5 on May 27, 2020
MarcoFalke locked this on Sep 8, 2021

Contributors