Skip to content

Remove unnecessary wallet includes from rpcconsole.cpp - #613

Merged
hebasto merged 1 commit into
bitcoin-core:masterfrom
laanwj:2022-06-qtconsole-includes
Jun 2, 2022
Merged

Remove unnecessary wallet includes from rpcconsole.cpp#613
hebasto merged 1 commit into
bitcoin-core:masterfrom
laanwj:2022-06-qtconsole-includes

Conversation

@laanwj

Copy link
Copy Markdown
Member

@maflcko

Copy link
Copy Markdown
Contributor

cr ACK 0994273

Should be correct if it compiles?

@laanwj

Copy link
Copy Markdown
MemberAuthor

I think so! I also did a cursory check for direct use of wallet classes but couldn't find any. Everything goes through WalletModel etc.

@hebastohebasto left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 0994273.

It has become redundant since #118.

@hebastohebasto changed the title qt: Remove unnecessary wallet includes from rpcconsole.cppRemove unnecessary wallet includes from rpcconsole.cppJun 2, 2022
@hebasto
hebasto merged commit da6792b into bitcoin-core:masterJun 2, 2022
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jun 3, 2022
…pcconsole.cpp
0994273 qt: Remove unnecessary wallet includes from rpcconsole.cpp (laanwj)
Pull request description:
Fixesbitcoin#25266
ACKs for top commit:
MarcoFalke:
cr ACK 0994273
hebasto:
ACK 0994273.
Tree-SHA512: 02cef4a1f3522c4cd662853eb930dfdf0866a5bd959a00f4c42d8c741b0751df4cf2e14c304b93ca3ce699c0e9730caf3d6fa2053009af312c60e861b0f79179

@jarolrodjarolrod left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

post merge ACK 0994273

@bitcoin-corebitcoin-core locked and limited conversation to collaborators Jun 5, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

./wallet/bdb.h:27:10: fatal error: db_cxx.h: No such file or directory

4 participants

@laanwj@maflcko@jarolrod@hebasto