Refactor ComputeAndSetWaste()

murchandamus · May 24, 2024 · d186c64 · d186c64
1 parent 4c387cb
commit d186c64
Showing 1 changed file with 2 additions and 7 deletions.
diff --git a/src/wallet/coinselection.cpp b/src/wallet/coinselection.cpp
@@ -842,13 +842,8 @@ void SelectionResult::SetBumpFeeDiscount(const CAmount discount)
 
 void SelectionResult::ComputeAndSetWaste(const CAmount min_viable_change, const CAmount change_cost, const CAmount change_fee)
 {
-    const CAmount change = GetChange(min_viable_change, change_fee);
-
-    if (change > 0) {
-        m_waste = GetSelectionWaste(change_cost, m_target, m_use_effective);
-    } else {
-        m_waste = GetSelectionWaste(0, m_target, m_use_effective);
-    }
+    bool makes_change = {GetChange(min_viable_change, change_fee) != 0};
+    m_waste = GetSelectionWaste(makes_change ? change_cost : 0, m_target, m_use_effective);
 }
 
 void SelectionResult::SetAlgoCompleted(bool algo_completed)