diff options
author | Ryan Ofsky <ryan@ofsky.org> | 2024-02-06 19:44:59 -0500 |
---|---|---|
committer | Ryan Ofsky <ryan@ofsky.org> | 2024-02-06 19:45:04 -0500 |
commit | 5b8990a1f3c49b0b02b7383c69e95320acbda13e (patch) | |
tree | 771329ea4cc37ba42dc1c6577e39c4d50aed66e5 | |
parent | 592e01398e765bf3c3f4fc106f40c8be373b4bb0 (diff) | |
parent | b14298c5bca395e5ed6a27fe1758c0d1f4b824ac (diff) |
Merge bitcoin/bitcoin#29388: fuzz: remove unused `args` and `context` from `FuzzedWallet`
b14298c5bca395e5ed6a27fe1758c0d1f4b824ac fuzz: remove unused `args` and `context` from `FuzzedWallet` (brunoerg)
Pull request description:
`ArgsManager args` and `WalletContext context` were previously used to create the wallet into `FuzzedWallet`. After fa15861763df71e788849b587883b3c16bb12229, they are not used anymore. This PR removes them.
ACKs for top commit:
maflcko:
lgtm ACK b14298c5bca395e5ed6a27fe1758c0d1f4b824ac
epiccurious:
utACK b14298c5bca395e5ed6a27fe1758c0d1f4b824ac
ryanofsky:
Code review ACK b14298c5bca395e5ed6a27fe1758c0d1f4b824ac
Tree-SHA512: 164e6a66ba05e11176a0cf68db6257f0ac07459cf7aa01ec4302b303c156c205a68128373a0b8daba0a6dfbff990af7fa14465a6341a296312fb20ea778c7a8c
-rw-r--r-- | src/wallet/test/fuzz/notifications.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/wallet/test/fuzz/notifications.cpp b/src/wallet/test/fuzz/notifications.cpp index 376060421c..9a515828fe 100644 --- a/src/wallet/test/fuzz/notifications.cpp +++ b/src/wallet/test/fuzz/notifications.cpp @@ -3,7 +3,6 @@ // file COPYING or http://www.opensource.org/licenses/mit-license.php. #include <addresstype.h> -#include <common/args.h> #include <consensus/amount.h> #include <interfaces/chain.h> #include <kernel/chain.h> @@ -89,8 +88,6 @@ void ImportDescriptors(CWallet& wallet, const std::string& seed_insecure) * Wraps a descriptor wallet for fuzzing. */ struct FuzzedWallet { - ArgsManager args; - WalletContext context; std::shared_ptr<CWallet> wallet; FuzzedWallet(const std::string& name, const std::string& seed_insecure) { |