Page MenuHomePhabricator

Add src/rpc/abc.cpp to the files checked by check-rpc-mappings

Authored by Fabien on Jul 26 2019, 09:54.



Add the Bitcoin-ABC specific RPCs to the check script.

Test Plan

Should return no error:

./test/lint/ .

Add the following line to the vRPCConvertParams[] table in

{"setexcessiveblock", 1, "dummy"},


./test/lint/ .

Should trigger an error:

setexcessiveblock argument 1 (named dummy in vRPCConvertParams) is not
defined in dispatch table

Diff Detail

rABC Bitcoin ABC
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Fabien created this revision.Jul 26 2019, 09:54
Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 26 2019, 09:54

This doesn't look like it's going to be very useful if thins are working this way as we'll keep forgetting to add files in there.

deadalnix accepted this revision.Jul 29 2019, 00:32

You should consider changing the script to use a regex or something.

This revision is now accepted and ready to land.Jul 29 2019, 00:32
Fabien added a comment.Jul 29 2019, 12:46

Will address in a follow-up