diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2017-10-01 16:42:39 +0200 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2017-10-01 16:46:56 +0200 |
| commit | fae2673d5d8dbb356423aa584a96d3ac367b5716 (patch) | |
| tree | 05e0c2944f7d6e9a8ad9c93b51c0e5816aafde8c /contrib | |
| parent | e542728cde676f218c552d841d0af29b92f9800b (diff) | |
qa: check-rpc-mapping must not run on empty lists
Diffstat (limited to 'contrib')
| -rwxr-xr-x | contrib/devtools/check-rpc-mappings.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/contrib/devtools/check-rpc-mappings.py b/contrib/devtools/check-rpc-mappings.py index d2698de041..7e96852c5c 100755 --- a/contrib/devtools/check-rpc-mappings.py +++ b/contrib/devtools/check-rpc-mappings.py @@ -63,7 +63,7 @@ def process_commands(fname): else: args = [] cmds.append(RPCCommand(name, args)) - assert not in_rpcs, "Something went wrong with parsing the C++ file: update the regexps" + assert not in_rpcs and cmds, "Something went wrong with parsing the C++ file: update the regexps" return cmds def process_mapping(fname): @@ -86,7 +86,7 @@ def process_mapping(fname): idx = int(m.group(2)) argname = parse_string(m.group(3)) cmds.append((name, idx, argname)) - assert not in_rpcs + assert not in_rpcs and cmds return cmds def main(): |