Page MenuHomePhabricator

rpc: Assert that RPCArg names are equal to CRPCCommand ones (mining)
ClosedPublic

Authored by PiRK on Sep 17 2021, 14:12.

Details

Summary

This is a backport of core#19717 [2/4]
https://github.com/bitcoin/bitcoin/pull/19717/commits/faaa46dc204d6d714f71dbc6f0bf02215dba0f0f

Note: the double \n in rpc_generate.py is required because of D7184

Depends on D10145 and D10149

Test Plan

ninja all check-all

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

PiRK requested review of this revision.Sep 17 2021, 14:12
PiRK planned changes to this revision.Sep 17 2021, 15:08

needs rebasing after D10149

PiRK edited the summary of this revision. (Show Details)

rebase onto D10149 and use const Config &

Fabien requested changes to this revision.Sep 19 2021, 05:16
Fabien added a subscriber: Fabien.
Fabien added inline comments.
test/functional/rpc_generate.py
21 ↗(On Diff #30033)

Duplicated \n, and please fix the layout while you're at it

This revision now requires changes to proceed.Sep 19 2021, 05:16

fix the layout of the generate help message

The extra \n is required because of D7184, which standardizes our help messages to always print a blank line between the command and the description.

Fabien requested changes to this revision.Sep 20 2021, 07:11
Fabien added inline comments.
test/functional/rpc_generate.py
23 ↗(On Diff #30048)

OK for the double \n, but please make the formatting consistent with the other parts of the code:

  • no need to put the \n on its own line
  • the space is at the end of the line, not the beginning
This revision now requires changes to proceed.Sep 20 2021, 07:11

put space at the end of the line, and second newline with the previous line

This revision is now accepted and ready to land.Sep 20 2021, 07:54