Another small step to get rid of legacy addresses in the RPC help texts -- for a RPCResult in fact we don't want to see any concrete address at all. All other RPCs containing the "addresses" array in the RPCResults (e.g. getrawtransaction, decodescript) show a generic "address" string instead of a concrete address, so we adapt this here as well.
rpc: generalize address in decoderawtransaction RPCResult #18164
pull theStack wants to merge 1 commits into bitcoin:master from theStack:20200216-rpc-remove-concrete-address-from-rpcresult changing 1 files +1 −1-
theStack commented at 10:28 PM on February 16, 2020: member
-
6e205eba9a
rpc: generalize address in decoderawtransaction RPCResult
All other RPCs containing the "addresses" array in the RPCResults (e.g. getrawtransaction, decodescript) show a generic "address" string instead of a concrete address, so we adapt this here as well.
- fanquake added the label RPC/REST/ZMQ on Feb 16, 2020
-
MarcoFalke commented at 12:39 AM on February 17, 2020: member
I'd prefer if this was done after #17809
-
DrahtBot commented at 3:51 AM on February 17, 2020: member
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #17809 (rpc: Auto-format RPCResult by MarcoFalke)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
-
emilengler commented at 3:00 PM on February 19, 2020: contributor
-
theStack commented at 12:55 PM on February 25, 2020: member
@MarcoFalke, @emilengler: #17809 already contains the change from this PR, hence it can be closed as soon as #17809 is merged.
- DrahtBot added the label Needs rebase on Mar 4, 2020
-
DrahtBot commented at 2:26 PM on March 4, 2020: member
<!--cf906140f33d8803c4a75a2196329ecb-->Needs rebase
-
MarcoFalke commented at 2:36 PM on March 4, 2020: member
@MarcoFalke, @emilengler: #17809 already contains the change from this PR, hence it can be closed as soon as #17809 is merged.
Ok, closing then.
- MarcoFalke closed this on Mar 4, 2020
- MarcoFalke locked this on Feb 15, 2022
Contributors
Labels