wallet: move BDB specific classes to bdb.{cpp/h}#19290
Merged
meshcollider merged 5 commits intobitcoin:masterfrom Jun 17, 2020
Merged
wallet: move BDB specific classes to bdb.{cpp/h}#19290meshcollider merged 5 commits intobitcoin:masterfrom
meshcollider merged 5 commits intobitcoin:masterfrom
Conversation
Put the implementation in the cpp, not the h file.
Leave wallet/db.{cpp/h} for generic WalletDatabase stuff. The BDB
specific stuff goes into bdb.{cpp/h}
This was referenced Jun 16, 2020
Contributor
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
This was referenced Jun 16, 2020
Member
|
Code review ACK 61c1633 |
Contributor
5 tasks
maflcko
pushed a commit
that referenced
this pull request
Jun 18, 2020
…Exists functions into non-template functions a389ed5 walletdb: refactor Read, Write, Erase, and Exists into non-template func (Andrew Chow) Pull request description: In order to override these later, the specific details of how the Read, Write, Erase, and Exists functions interact with the actual database file need to go into functions that are not templated. The functions `ReadKey`, `WriteKey`, `EraseKey`, and `HasKey` are introduced to handle the actual interaction with the database. This is mostly a moveonly. Based on #19290 ACKs for top commit: ryanofsky: Code review ACK a389ed5. No changes since last review, just non-conflicting rebase Sjors: utACK a389ed5 MarcoFalke: ACK a389ed5 🔳 Tree-SHA512: 73bd2fe9ddc4a132d4db6b97e77f5d5f8aa68b8cb25192384f3bacd826365947763a9eee73672331d34578e3f5ade85ee6aa550ff4d89eb62e482250dd5973e4
Fabcien
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Dec 7, 2020
Summary: Partial backport (1/5) of core [[bitcoin/bitcoin#19290 | PR19290]]: bitcoin/bitcoin@c3538f4 Test Plan: ninja all check Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D8611
Fabcien
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Dec 7, 2020
Summary: Partial backport (2/5) of core [[bitcoin/bitcoin#19290 | PR19290]]: bitcoin/bitcoin@f6fc5f3 Depends on D8611. Test Plan: ninja all check Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D8612
Fabcien
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Dec 7, 2020
Summary: Partial backport (3/5) of core [[bitcoin/bitcoin#19290 | PR19290]]: bitcoin/bitcoin@25a6557 Depends on D8612. Test Plan: ninja all check Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D8613
Fabcien
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Dec 7, 2020
Summary: ``` Put the implementation in the cpp, not the h file. ``` Partial backport (4/5) of core [[bitcoin/bitcoin#19290 | PR19290]]: bitcoin/bitcoin@8f03364 Depends on D8613. Test Plan: ninja all check-all Reviewers: #bitcoin_abc, deadalnix Reviewed By: #bitcoin_abc, deadalnix Differential Revision: https://reviews.bitcoinabc.org/D8614
Fabcien
pushed a commit
to Bitcoin-ABC/bitcoin-abc
that referenced
this pull request
Dec 7, 2020
Summary:
```
Leave wallet/db.{cpp/h} for generic WalletDatabase stuff. The BDB
specific stuff goes into bdb.{cpp/h}
```
Completes backport (5/5) of core [[bitcoin/bitcoin#19290 | PR19290]]:
bitcoin/bitcoin@61c1633
Depends on D8614.
Test Plan:
ninja all check-all
Reviewers: #bitcoin_abc, majcosta
Reviewed By: #bitcoin_abc, majcosta
Subscribers: majcosta
Differential Revision: https://reviews.bitcoinabc.org/D8615
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moves the BDB specific classes from db.{cpp/h} to bdb.{cpp/h}.
To do this,
SplitWalletFilePathis first made non-static. ThenIsWalletLoadedfunctionality is moved toIsBDBWalletLoadedwhich is called byIsWalletLoaded. Then the bulk of db.{cpp/h} is moved to a new file bdb.{cpp/h}.While doing some moveonly stuff, an additional commit moves the
*CursorandTxn*implementations out of the header file and into the cpp file.Part of #18971