rpc: clarify total_amount in gettxoutsetinfo doc #17196

pull benthecarman wants to merge 1 commits into bitcoin:master from benthecarman:gettxoutsetinfo-doc changing 1 files +1 −1

benthecarman commented at 3:02 AM on October 19, 2019: contributor

Small clarification
rpc: clarify total_amount in gettxoutsetinfo doc 33aebf672b
DrahtBot added the label RPC/REST/ZMQ on Oct 19, 2019

in src/rpc/blockchain.cpp:983 in 33aebf672b

 979 | @@ -980,7 +980,7 @@ static UniValue gettxoutsetinfo(const JSONRPCRequest& request)
 980 |              "  \"bogosize\": n,          (numeric) A meaningless metric for UTXO set size\n"
 981 |              "  \"hash_serialized_2\": \"hash\", (string) The serialized hash\n"
 982 |              "  \"disk_size\": n,         (numeric) The estimated size of the chainstate on disk\n"
 983 | -            "  \"total_amount\": x.xxx          (numeric) The total amount\n"
 984 | +            "  \"total_amount\": x.xxx          (numeric) The total amount of mined coins\n"

MarcoFalke commented at 12:47 PM on October 19, 2019:

I don't think this is true. It is the upper bound of coins that can be theoretically spent, which is lower than the total of mined coins.

sipa commented at 3:19 PM on October 19, 2019:

And "total of mined coins" is ambiguous too: https://www.reddit.com/r/Bitcoin/comments/djw5z9/z/f4962kr

laanwj commented at 11:35 AM on October 21, 2019: member

NACK it's just that, the total amount of coins in UTXOs, created through whatever means (most UTXOs will likely have been created as output of normal transactions); involving mining here is unnecessary
laanwj closed this on Oct 21, 2019
benthecarman deleted the branch on Oct 21, 2019
DrahtBot locked this on Dec 16, 2021

Contributors

Labels