Matias Furszyfer
Matias Furszyfer
Found it while was digging over a `getnewaddress` timeout on the functional test suite. ### Context: We are calling `TopUp()` twice in the following flows for descriptor wallets: A) `CWallet::GetNewDestination`:...
Finishing the encapsulation work started in #25337. #### Context We are currently using the address book for different purposes and features in the wallet such us: 1) Store and retrieve...
#### # Context (Current Flow on Master) In the transaction creation process, in order to select which coins the new transaction will spend, we first obtain all the available coins...
The idea originates from https://github.com/bitcoin/bitcoin/pull/24845#issuecomment-1130310321. Note: For clarity, it's recommended to start reviewing from the end result to understand the structure of the flow. #### GroupOutputs function rationale: If "Avoid...
Focused on the following changes: 1) Removed the entire locked coins lookup inside `ListCoins` by including them directly on the `AvailableCoins` result (where we were skipping them before). 2) Grouped...
Simple PR; adds a "include_immature_coinbase" flag to `listunspent` to include the immature coinbase UTXOs on the response. Requested by #25728.
Negative PR with extended test coverage :). 1) Cleaned duplicated code and added coverage for the 'AvailableCoins' incremental result. 2) The class `AvailableCoinsTestingSetup` inside `availablecoins_tests.cpp` is a plain copy of...
### First, Improvements for SQLite and BDB #### Context Each process, has a set of activities that are performed within the context of certain event. In many of the inner...
Based on #11240 context, adding test coverage for the behavior introduced in #11225 and to the current wallet limitations. This is the first step towards adding the ability to store...
Simple, and not interesting, refactor that someone has to do sooner or later. We are translating some init arguments names when those shouldn't be translated.