Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ap/accept tx to mempool call consistency #602

Merged
merged 2 commits into from
Sep 6, 2023

Conversation

a-petrini
Copy link
Contributor

No description provided.

Copy link
Contributor

@drgora drgora left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks

Copy link
Contributor

@JackPiri JackPiri left a comment

Choose a reason for hiding this comment

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

The PR is fine to me.
Just a question: is it ok that in test_asyncproofverifier.cpp and test_reindex.cpp we reset both mempool and connman, while in test_sidechain_blocks.cpp, test_wallet.cpp and test_wallet_cert.cpp we reset mempool only?
Thanks

This change updates AcceptTxToMemoryPool() enforcing the usage of the "CTxMemPool pool" input
parameter instead of relying on the global mempool instance. This change also inolve the
function GetMinRelayFee() that is updated in the same way.
This change updates several gtest that failed when executed singularly with the --gtest_filter
option, and not when executed inside the complete gtest suite.
The change added a few explicit initialization of the mempool and connection manager global
instances, to prevent segfault errors
@ptagl ptagl force-pushed the ap/acceptTxToMempool_call_consistency branch from 502e8a3 to d6469f0 Compare September 6, 2023 15:01
@ptagl ptagl merged commit 0a761ee into main Sep 6, 2023
@ptagl ptagl deleted the ap/acceptTxToMempool_call_consistency branch September 6, 2023 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants