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
- 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

- example: if util.h includes stdint.h, remove it from util.cpp, as
  util.h is the first header included in util.cpp

379778bde6

Diapolo cross-referenced this on Nov 15, 2013 from issue core: remove includes in .cpp, if header is already in .h by Diapolo
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 12:58 AM on November 16, 2013: none

@laanwj Can you re-merge this, sorry for the confusion. Ping :).
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
bitcoin locked this on Sep 8, 2021

Contributors