refactor: move CreatedTransactionResult to an util file

bitcoin-core · Mar 22, 2024 · dec14e4 · dec14e4
1 parent 786e3c5
commit dec14e4
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 11 deletions.
diff --git a/src/Makefile.am b/src/Makefile.am
@@ -358,6 +358,7 @@ BITCOIN_CORE_H = \
   wallet/sqlite.h \
   wallet/transaction.h \
   wallet/types.h \
+  wallet/util_spend.h \
   wallet/wallet.h \
   wallet/walletdb.h \
   wallet/wallettool.h \

diff --git a/src/wallet/spend.h b/src/wallet/spend.h
@@ -10,6 +10,7 @@
 #include <util/result.h>
 #include <wallet/coinselection.h>
 #include <wallet/transaction.h>
+#include <wallet/util_spend.h>
 #include <wallet/wallet.h>
 
 #include <optional>
@@ -202,17 +203,6 @@ util::Result<SelectionResult> SelectCoins(const CWallet& wallet, CoinsResult& av
                                           const CAmount& nTargetValue, const CCoinControl& coin_control,
                                           const CoinSelectionParams& coin_selection_params) EXCLUSIVE_LOCKS_REQUIRED(wallet.cs_wallet);
 
-struct CreatedTransactionResult
-{
-    CTransactionRef tx;
-    CAmount fee;
-    FeeCalculation fee_calc;
-    std::optional<unsigned int> change_pos;
-
-    CreatedTransactionResult(CTransactionRef _tx, CAmount _fee, std::optional<unsigned int> _change_pos, const FeeCalculation& _fee_calc)
-        : tx(_tx), fee(_fee), fee_calc(_fee_calc), change_pos(_change_pos) {}
-};
-
 /**
  * Create a new transaction paying the recipients with a set of coins
  * selected by SelectCoins(); Also create the change output, when needed

diff --git a/src/wallet/util_spend.h b/src/wallet/util_spend.h
@@ -0,0 +1,24 @@
+// Copyright (c) 2024-present The Bitcoin Core developers
+// Distributed under the MIT software license, see the accompanying
+// file COPYING or https://www.opensource.org/licenses/mit-license.php.
+
+#ifndef BITCOIN_WALLET_UTIL_SPEND_H
+#define BITCOIN_WALLET_UTIL_SPEND_H
+
+#include <policy/fees.h>
+#include <wallet/transaction.h>
+
+#include <optional>
+
+struct CreatedTransactionResult
+{
+    CTransactionRef tx;
+    CAmount fee;
+    FeeCalculation fee_calc;
+    std::optional<unsigned int> change_pos;
+
+    CreatedTransactionResult(CTransactionRef _tx, CAmount _fee, std::optional<unsigned int> _change_pos, const FeeCalculation& _fee_calc)
+            : tx(_tx), fee(_fee), fee_calc(_fee_calc), change_pos(_change_pos) {}
+};
+
+#endif // BITCOIN_WALLET_UTIL_SPEND_H