state.Invalid() is always false and alters state, check should be IsInvalid()
Broken since 942b33a
Fix semantic typo in state.CorruptionPossible check #4533
pull kazcw wants to merge 1 commits into bitcoin:master from kazcw:corruptedcheck changing 1 files +1 −1-
kazcw commented at 3:00 AM on July 15, 2014: contributor
-
43005cffa3
Fix semantic typo in state.CorruptionPossible check
state.Invalid() is always false, check should be IsInvalid() Broken since 942b33a
-
BitcoinPullTester commented at 3:58 AM on July 15, 2014: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4533_43005cffa394afde08a80b28ac52f7821c96d841/ 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 commented at 6:30 AM on July 15, 2014: member
Good catch!
-
sipa commented at 2:21 PM on July 15, 2014: member
ACK
-
ghost commented at 2:42 PM on July 15, 2014: none
ACK
-
gmaxwell commented at 1:56 AM on July 16, 2014: contributor
utACK
- laanwj merged this on Jul 17, 2014
- laanwj closed this on Jul 17, 2014
- laanwj referenced this in commit ea8c288a35 on Jul 17, 2014
- MarcoFalke locked this on Sep 8, 2021
