- example: if util.h includes stdint.h, remove it from util.cpp, as util.h is the first header included in util.cpp
core: remove includes in .cpp, if header is already in .h #3257
pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:headers changing 11 files +0 −20-
Diapolo commented at 11:21 AM on November 15, 2013: none
-
379778bde6
core: remove includes in .cpp, if header is already in .h
- example: if util.h includes stdint.h, remove it from util.cpp, as util.h is the first header included in util.cpp
-
BitcoinPullTester commented at 11:50 AM on November 15, 2013: none
Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/379778bde61f8589988cf113ec4c3961b8155b0a 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.
-
-
Diapolo commented at 3:03 PM on November 20, 2013: none
This shouldn't be open for so long, as it is uncontroversial IMHO...
- laanwj referenced this in commit d980f9b7d6 on Nov 20, 2013
- laanwj merged this on Nov 20, 2013
- laanwj closed this on Nov 20, 2013
- Diapolo deleted the branch on Nov 20, 2013
- DrahtBot locked this on Sep 8, 2021
Contributors