From 365aca40453995163bbd17231251512f9f9a103b Mon Sep 17 00:00:00 2001 From: yancy Date: Fri, 16 Sep 2022 14:22:42 +0200 Subject: refactor: Simplify feerate comparison statement --- src/wallet/coinselection.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/wallet/coinselection.cpp') diff --git a/src/wallet/coinselection.cpp b/src/wallet/coinselection.cpp index b568e90998..718bced07d 100644 --- a/src/wallet/coinselection.cpp +++ b/src/wallet/coinselection.cpp @@ -93,7 +93,7 @@ std::optional SelectCoinsBnB(std::vector& utxo_poo bool backtrack = false; if (curr_value + curr_available_value < selection_target || // Cannot possibly reach target with the amount remaining in the curr_available_value. curr_value > selection_target + cost_of_change || // Selected value is out of range, go back and try other branch - (curr_waste > best_waste && (utxo_pool.at(0).fee - utxo_pool.at(0).long_term_fee) > 0)) { // Don't select things which we know will be more wasteful if the waste is increasing + (curr_waste > best_waste && (utxo_pool.at(0).fee > utxo_pool.at(0).long_term_fee))) { // Don't select things which we know will be more wasteful if the waste is increasing backtrack = true; } else if (curr_value >= selection_target) { // Selected value is within range curr_waste += (curr_value - selection_target); // This is the excess value which is added to the waste for the below comparison -- cgit v1.2.3