Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CWallet: EstimateAnonFeeIncluded (WIP) #72

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

CWallet: EstimateAnonFeeIncluded (WIP) #72

wants to merge 8 commits into from

Commits on Nov 12, 2016

  1. Merge pull request #13 from shadowproject/master 

    merge
    @kewde
    kewde committed Nov 12, 2016
    Configuration menu
    Copy the full SHA
    bb8d18b View commit details
    Browse the repository at this point in the history

Commits on Nov 15, 2016

  1. Modified EstimateAnonFee 

    Created EstimateAnonFeeIncluded

EstimateAnonFeeIncluded calculates how much you can spend such that nValue + nFees < nMaxAmount.
Searches for nValue (while loop) until value found that equation makes sense.
while loop reduces nMaxAmount with nTransactionFee (lowest steps I could imagine) until an acceptable value is found.

TODO:
spend dust as fee so wallet shows a zero balance.
    @kewde
    kewde committed Nov 15, 2016
    Configuration menu
    Copy the full SHA
    c0c5f33 View commit details
    Browse the repository at this point in the history

  2. Fix small typos

    @kewde
    kewde committed Nov 15, 2016
    Configuration menu
    Copy the full SHA
    44f72fb View commit details
    Browse the repository at this point in the history

  3. Add JSON RPC command options (ability for testing options)

    @kewde
    kewde committed Nov 15, 2016
    Configuration menu
    Copy the full SHA
    03d112c View commit details
    Browse the repository at this point in the history

Commits on Nov 16, 2016

  1. Changed step size to MIN_TX_FEE_ANON 

    Added and cleaned up debugging
Improved JSON RPC estimateanonfee output
Found segmentation fault on narration
    @kewde
    kewde committed Nov 16, 2016
    Configuration menu
    Copy the full SHA
    2387072 View commit details
    Browse the repository at this point in the history

  2. Remove PRId64 (fails on windows builds), now casts int64_t to int 

    Improved RPC logic
    @kewde
    kewde committed Nov 16, 2016
    Configuration menu
    Copy the full SHA
    48e1ed5 View commit details
    Browse the repository at this point in the history

  3. Trigger TravisCI rebuild

    @kewde
    kewde committed Nov 16, 2016
    Configuration menu
    Copy the full SHA
    ba208a5 View commit details
    Browse the repository at this point in the history

  4. Use %d and uncasted values instead

    @kewde
    kewde committed Nov 16, 2016
    Configuration menu
    Copy the full SHA
    8fe4ee4 View commit details
    Browse the repository at this point in the history