miner: indentation fixes, remove for (;;) #4169

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:format_miner changing 1 files +101 −98

Diapolo commented at 12:50 PM on May 10, 2014: none
- change a for (;;) into while (true), as we nowhere else use the first
- init nNonceFound to 0
- fix indentation in BitcoinMiner try/catch block
sipa commented at 12:53 PM on May 10, 2014: member

Do you mind waiting for #4100?
Diapolo commented at 12:59 PM on May 10, 2014: none

I have zero problems with that :).
Diapolo commented at 5:32 PM on June 10, 2014: none

@sipa Is your patch due to be merged soon or can we merge this before?

miner: indentation fixes, remove for (;;)

- change a for (;;) into while (true), as we nowhere else use the first
- init nNonceFound to 0
- fix indentation in BitcoinMiner try/catch block

0655fac0b1

Diapolo commented at 10:35 PM on June 21, 2014: none

@sipa Can you re-check and ACK now?
BitcoinPullTester commented at 7:10 PM on June 22, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4169_0655fac0b11c7fa1992f50537887728a0878a5eb/ 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 24, 2014
laanwj closed this on Jun 24, 2014
laanwj referenced this in commit 5dd0bfefcb on Jun 24, 2014
Diapolo deleted the branch on Jun 26, 2014
DrahtBot locked this on Sep 8, 2021

Contributors