Skip to content

Commit

Permalink
refactor: revert m_next_resend to not be std::atomic
Browse files Browse the repository at this point in the history
Since m_next_resend is now only called from MaybeResendWalletTxs()
we don't have any potential race conditions anymore, so the usage
of std::atomic can be reverted.

Github-Pull: #26205
Rebased-From: b01682a
  • Loading branch information
stickies-v authored and fanquake committed Oct 13, 2022
1 parent 43ced0b commit 9b438f0
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/wallet/wallet.h
Original file line number Diff line number Diff line change
Expand Up @@ -250,7 +250,7 @@ class CWallet final : public WalletStorage, public interfaces::Chain::Notificati
int nWalletVersion GUARDED_BY(cs_wallet){FEATURE_BASE};

/** The next scheduled rebroadcast of wallet transactions. */
std::atomic<int64_t> m_next_resend{GetDefaultNextResend()};
int64_t m_next_resend{GetDefaultNextResend()};
/** Whether this wallet will submit newly created transactions to the node's mempool and
* prompt rebroadcasts (see ResendWalletTransactions()). */
bool fBroadcastTransactions = false;
Expand Down

0 comments on commit 9b438f0

Please sign in to comment.