glozow 381593c906 Merge bitcoin/bitcoin#24845: wallet: return error msg for "too-long-mempool-chain" ...

f3221d373a test: add wallet too-long-mempool-chain error coverage (furszy)
acf0119d24 wallet: return error msg for too-long-mempool-chain failure (furszy)

Pull request description:

  Fixes #23144.

  We currently return a general "Insufficient funds" from Coin
  Selection when we actually skipped unconfirmed UTXOs that
  surpassed the mempool ancestors limit.

  This PR make the error clearer by returning:
  "Unconfirmed UTXOs are available, but spending them creates
  a chain of transactions that will be rejected by the mempool"

  Also, added an early return from Coin Selection if the sum of
  the discarded coins decreases the available balance below the
  target amount.

ACKs for top commit:
  achow101:
    ACK f3221d373a
  S3RK:
    Code review ACK f3221d373a
  Xekyo:
    ACK f3221d373a

Tree-SHA512: 13e5824b75ac302280ff894560a4ebf32a74f32fe49ef8281f2bc99c0104b92cef33d3b143c6e131f3a07eafe64533af7fc60abff585142c134b9d6e531a6a66

2023-03-23 15:53:56 +00:00

..

rpc

Merge bitcoin/bitcoin#26194: rpc, wallet: use the same next_index key in listdescriptors and importdescriptors

2023-03-08 12:15:31 -05:00

test

wallet: single output groups filtering and grouping process

2023-03-06 09:45:40 -03:00

bdb.cpp

…

bdb.h

…

coincontrol.cpp

…

coincontrol.h

…

coinselection.cpp

wallet: remove unused methods

2023-03-08 10:32:30 -03:00

coinselection.h

wallet: remove coin control arg from AutomaticCoinSelection

2023-03-08 19:03:40 -03:00

context.cpp

…

context.h

…

crypter.cpp

…

crypter.h

…

db.cpp

…

db.h

…

dump.cpp

…

dump.h

…

external_signer_scriptpubkeyman.cpp

…

external_signer_scriptpubkeyman.h

…

feebumper.cpp

…

feebumper.h

…

fees.cpp

…

fees.h

…

init.cpp

…

interfaces.cpp

…

ismine.h

…

load.cpp

…

load.h

…

receive.cpp

…

receive.h

…

salvage.cpp

…

salvage.h

…

scriptpubkeyman.cpp

wallet: annotate bools in descriptor SPKM FillPSBT()

2023-02-23 11:46:29 +01:00

scriptpubkeyman.h

…

spend.cpp

Merge bitcoin/bitcoin#24845: wallet: return error msg for "too-long-mempool-chain"

2023-03-23 15:53:56 +00:00

spend.h

wallet: remove coin control arg from AutomaticCoinSelection

2023-03-08 19:03:40 -03:00

sqlite.cpp

Merge bitcoin/bitcoin#25491: wallet: use Mutex for g_sqlite_mutex instead of GlobalMutex

2023-03-06 10:50:10 -05:00

sqlite.h

Merge bitcoin/bitcoin#25491: wallet: use Mutex for g_sqlite_mutex instead of GlobalMutex

2023-03-06 10:50:10 -05:00

transaction.cpp

…

transaction.h

…

wallet.cpp

Merge bitcoin/bitcoin#25666: refactor: wallet, do not translate init options names

2023-03-19 12:24:21 +00:00

wallet.h

wallet: remove unused methods

2023-03-08 10:32:30 -03:00

walletdb.cpp

…

walletdb.h

…

wallettool.cpp

…

wallettool.h

…

walletutil.cpp

…

walletutil.h

…