Page MenuHomePhabricator

[backport#17154 3/3][wallet] Remove `state` argument from CWallet::CommitTransaction
ClosedPublic

Authored by majcosta on Aug 1 2020, 15:15.

Details

Summary

The state return argument has not been set since commit 611291c198.
Remove it (and the one place that it's used in a calling function).

https://github.com/bitcoin/bitcoin/pull/17154/commits/9e95931865186d7a9a6dc54b64bd96507e9fea4b


Depends on D7113

Concludes backport of Core PR17154

Test Plan
ninja check check-functional

Diff Detail

Event Timeline

majcosta requested review of this revision.Aug 1 2020, 15:15

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

Fabien added inline comments.
src/interfaces/wallet.cpp
9

This is removed in the PR, what is it still used for ?

src/wallet/rpcwallet.cpp
10

Same question

src/interfaces/wallet.cpp
9

ContextualCheckTransaction still uses it

deadalnix added inline comments.
src/interfaces/wallet.cpp
9

It is defined in consensus/tx_verify.h

src/interfaces/wallet.cpp
71

I mean, TxValidationState is required for calls to ContextualCheckTransaction here

src/wallet/rpcwallet.cpp
692

here

766

here

1200

and here

This revision is now accepted and ready to land.Aug 4 2020, 22:49