header include cleanup #4911

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:include_cleanup changing 36 files +67 −48

Diapolo commented at 10:53 AM on September 14, 2014: none
- ensures alphabetical ordering for includes etc. in source file headers
I know this will create (once more) some controversial feelings, but I'm going to keep it up-to-date for a while. No need to hurry, no need to rush... but let's please consider cleaning that up ONCE and use that style "forever", thanks.

header include cleanup

- ensures alphabetical ordering for includes etc. in source file headers

611116d4e3

BitcoinPullTester commented at 11:07 AM on September 14, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4911_611116d4e31634384e9757befbd5e5a1a80e8cd3/ 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.
sipa commented at 3:25 PM on September 14, 2014: member

ACK
jgarzik commented at 1:25 AM on September 15, 2014: contributor

sigh.

ut ACK
sipa merged this on Sep 16, 2014
sipa closed this on Sep 16, 2014
sipa referenced this in commit 7388b74cd2 on Sep 16, 2014
Diapolo deleted the branch on Sep 16, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors