HomePhabricator

fix confusing parameter information

Description

fix confusing parameter information

Summary: reference to #10710

Test Plan:
// Add a bitcoin-abc node firstly
bitcoin-cli addnode 67.82.89.67 add

// Commands and returned results as following:

bitcoin-cli getaddednodeinfo true

   error code: -24
   error message:
   Error: Node has not been added.
bitcoin-cli getaddednodeinfo true 67.82.89.67

  error code: -1
  error message:
  getaddednodeinfo ( "node" )

// the correct usage:
bitcoin-cli getaddednodeinfo
bitcoin-cli getaddednodeinfo 67.82.89.67

So getaddednodeinfo only have one optional argument<node>.

Reviewers: O1 Bitcoin ABC, #bitcoin_abc, deadalnix, qshuai

Reviewed By: O1 Bitcoin ABC, #bitcoin_abc, deadalnix

Subscribers: jasonbcox, deadalnix, teamcity

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

Details

Provenance
qshuaiAuthored on Sep 19 2018, 16:39
jasonbcoxCommitted on Sep 19 2018, 19:49
jasonbcoxPushed on Sep 19 2018, 19:50
Reviewer
Restricted Owners Package
Differential Revision
D1447: fix confusing parameter information
Parents
rABC0d5427fcf95e: [ci] fix build parallelization
Branches
Unknown
Tags
Unknown