Skip to content

Conversation

@UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Dec 19, 2014

Select "regular" funds first when user unchecked "Darksend" checkbox on "Send" tab (function SelectCoinsMinConf is used only in that case right now). This should help to preserve already anonymized funds.
NOTE: It's straight forward and unoptimized. Not tested as testnet is dead right now.

@UdjinM6
Copy link
Author

UdjinM6 commented Dec 20, 2014

Ok, testnet is half-dead now ;) so I did few tests. Seems to work fine in terms of expected logic. However this brings darksend rounds calculations which can be quite expensive. Will rework.

@UdjinM6 UdjinM6 closed this Dec 20, 2014
@UdjinM6
Copy link
Author

UdjinM6 commented Dec 20, 2014

  • check against denoms array, no more rounds calculations
  • sort vCoins to move denoms down on the list

@UdjinM6 UdjinM6 reopened this Dec 20, 2014
@darkcoinproject
Copy link

It looks pretty good, can you make a merge request for the v0.11.0.x branch though? I'd rather not push out another version for 10. unless we have to

@UdjinM6 UdjinM6 deleted the select_nondenom_first branch January 16, 2015 01:55
barrystyle pushed a commit to PACGlobalOfficial/PAC that referenced this pull request Jul 17, 2019
* News URL updated

* QCalendarView styles modified 

* Replace version of the wallet
FornaxA pushed a commit to ioncoincore/ion that referenced this pull request Jul 6, 2020
SNAP: Trigger build of candidate (stable) branch
FornaxA pushed a commit to ioncoincore/ion that referenced this pull request Jul 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants