HomePhabricator

Fix currency/fee-rate unit string in the help text

Description

Fix currency/fee-rate unit string in the help text

Summary:

1. The RPC help text should use the constant CURRENCY_UNIT defined in
  policy/feerate.cpp instead of the literal 'BTC'. In the following
  2 RPC commands, 'BTC' is written directly in the help text.
    1) estimatesmartfee
    2) estimaterawfee
  And also, for these help strings, the notation
  'fee-per-kilobyte (in BTC)' is somewhat ambiguous.

  To write more precisely, this commit changes to 'fee rate in BTC/kB'
  with using the constant CURRENCY_UNIT.

2. Some RPC command use 'satoshis' as the unit. It should be written
  as 'satoshis' instead of 'Satoshis' in the RPC help text.

  So, this commit fixes this typo in getblocktemplate.

3. The phrase that '... feerate (BTC per KB) ...'  is used to explain
  the fee rate in the help text of following 2 RPC commands.
    1) getmempoolinfo
    2) fundrawtransaction
  But they are different from other similar help text of the RPCs.
  And also, 'KB' implies Kibibyte (2^10 byte).

  To unify and to clarify, this commit changes these phrase to
  '... fee rate in BTC/kB ...'.
  (BTC references the constant 'CURRENCY_UNIT')

Backport of core PR11173
https://github.com/bitcoin/bitcoin/pull/11173/files

Depends on D3430

Test Plan:

bitcoin-cli help getmempoolinfo
bitcoin-cli help getblocktemplate
bitcoin-cli help fundrawtransaction

Reviewers: #bitcoin_abc, deadalnix, jasonbcox

Reviewed By: #bitcoin_abc, jasonbcox

Differential Revision: https://reviews.bitcoinabc.org/D3431

Details

Provenance
Akio Nakamura <nakamura@dgtechnologies.co.jp>Authored on Aug 28 2017, 03:44
FabienCommitted on Jun 27 2019, 20:11
jasonbcoxPushed on Jun 27 2019, 22:14
Reviewer
Restricted Project
Differential Revision
D3431: Fix currency/fee-rate unit string in the help text
Parents
rSTAGING04ca7facf463: getmempool mempoolminfee is a BCH/kB feerate
Branches
Unknown
Tags
Unknown