After discussing with BlueMatt, this appears to be harmless in its current state since it's always set before it's used. Initialize it anyway for readability and future safety.
Fix uninitialized variable added in b33b9a6fe #3316
pull theuni wants to merge 1 commits into bitcoin:master from theuni:fix-uninitialized changing 1 files +1 −1-
theuni commented at 12:51 AM on November 26, 2013: member
-
106f133de6
Fix uninitialized variable added in b33b9a6fe
After discussing with BlueMatt, this appears to be harmless in its current state since it's always set before it's used. Initialize it anyway for readability and future safety.
-
gavinandresen commented at 12:52 AM on November 26, 2013: contributor
ACK
-
BitcoinPullTester commented at 1:28 AM on November 26, 2013: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/106f133de6bdb577c4135847fd703d08f525ba46 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 referenced this in commit 03b6a1cee4 on Nov 26, 2013
- laanwj merged this on Nov 26, 2013
- laanwj closed this on Nov 26, 2013
- Bushstar referenced this in commit f2db4e7d66 on Apr 8, 2020
- DrahtBot locked this on Sep 8, 2021
Contributors