z_mergetoaddress maximum_utxo_size is confusing #335

Open
opened 7 months ago by duke · 0 comments
duke commented 7 months ago
Owner

The 5th optional argument to z_mergetoaddress is maximum_utxo_size, the maximum size UTXO it will merge. It's not clear that this applies only to transparent UTXOs and not zUTXOs and a user may use it when operating on zUTXOs and merge things they do not intend.

It would be better if this argument applied to utxos+zutxos, which will require a new internals function to support that. Transparent utxos use AvailableCoins which supports a max size, but zutxos use GetFilteredNotes which does not.

The 5th optional argument to z_mergetoaddress is maximum_utxo_size, the maximum size UTXO it will merge. It's not clear that this applies only to transparent UTXOs and not zUTXOs and a user may use it when operating on zUTXOs and merge things they do not intend. It would be better if this argument applied to utxos+zutxos, which will require a new internals function to support that. Transparent utxos use `AvailableCoins` which supports a max size, but zutxos use `GetFilteredNotes` which does not.
duke added the
bug
label 7 months ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.