cleanup after merge-rawtx #4616

pull Diapolo wants to merge 4 commits into bitcoin:master from Diapolo:bitcoin-tx-cleanup changing 5 files +30 −21

Diapolo commented at 6:44 AM on August 1, 2014: none

No description provided.

fix compilation error in core_io.h

- error: 'vector' in namespace 'std' does not name a type
- add <vector> include in core_io.h
- remove <vector> includes from core_read.cpp and core_write.cpp

1b73d36b2c

add license header to core_io.h, core_read/_write.cpp 2b600992e8
include file ordering cleanup for core_io.h, core_read/_write.cpp 8a36c327b7
include file ordering cleanup for bitcoin-tx and rpcmining cc39466575
Diapolo referenced this in commit a715a643bc on Aug 1, 2014
BitcoinPullTester commented at 7:00 AM on August 1, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4616_cc394665751ee6506d59f9982410fff6e6005b07/ 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 added the label Improvement on Aug 1, 2014
jgarzik commented at 10:23 AM on August 1, 2014: contributor

The first two commits were merged.

RE include file and namespace directive alphabetization, that is incredibly low priority and just wastes time to submit PRs for it.
jgarzik closed this on Aug 1, 2014
Diapolo deleted the branch on Aug 2, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels