Page MenuHomePhabricator

[backport#16451 2/3][wallet] Flatten CWalletTx class hierarchy
ClosedPublic

Authored by majcosta on Sat, Jul 25, 07:15.

Details

Summary

Removes CMerkleTx as a base class for CWalletTx. Serialization logic is
moved from CMerkleTx to CWalletTx.

https://github.com/bitcoin/bitcoin/pull/16451/commits/783a76f23ba4f33b6e6f609eaf3bf41afd9bcd6f


Depends on D7038

Partial backport of Core PR16451

Test Plan
ninja check check-functional

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

majcosta created this revision.Sat, Jul 25, 07:15
Herald added a reviewer: Restricted Project. · View Herald TranscriptSat, Jul 25, 07:15
deadalnix accepted this revision.Sat, Jul 25, 13:05
This revision is now accepted and ready to land.Sat, Jul 25, 13:05
teamcity edited the summary of this revision. (Show Details)Mon, Jul 27, 22:03

[Bot Message]
One or more PR numbers were detected in the summary.
Links to those PRs have been inserted into the summary for reference.