Move code to new walletdb module. Simplify header dependencies. #1108

pull jgarzik wants to merge 3 commits into bitcoin:master from jgarzik:headers changing 23 files +652 −598

jgarzik commented at 10:01 PM on April 15, 2012: contributor
Addresses issue #1105. Does not solve things completely, but one big code movement (walletdb) appeared to magically solve many include dependency problems.
1. Move defs and code to walletdb.{h,cpp} respectively. This solves a key circular dependency plaguing include file cleanups, namely that db.h inclusion no longer sucks in CWalletTx etc.
2. Reduce headers.h usage in most modules.

Move CWalletDB code to new walletdb module.

In addition to standard code separation, this change opens the door
to fixing several include inter-dependencies.

4af5d0addc

Greatly reduce use of headers.h. ea97c507a6
Move proto version to version.h. Reduce header deps a bit more. 0ff6ce4e2a
jgarzik commented at 11:15 PM on April 15, 2012: contributor

superceded by #1109
jgarzik closed this on Apr 15, 2012
jgarzik deleted the branch on Aug 24, 2014
lateminer referenced this in commit c7bc2e1288 on Dec 25, 2019
DrahtBot locked this on Sep 8, 2021

Contributors