aboutsummaryrefslogtreecommitdiffstats
path: root/scripts
diff options
context:
space:
mode:
authorKevin O'Connor <kevin@koconnor.net>2017-06-20 12:09:55 -0400
committerKevin O'Connor <kevin@koconnor.net>2017-06-29 13:33:59 -0400
commitc1bd628ce524f52851cf2bbcb4bc8bbc373dcd6b (patch)
tree4b3705dbf34c7f8ec0f52a7fadae17f43031d31b /scripts
parent849096d5f35185b082571665ca590fed9c35a073 (diff)
downloadkutter-c1bd628ce524f52851cf2bbcb4bc8bbc373dcd6b.tar.gz
kutter-c1bd628ce524f52851cf2bbcb4bc8bbc373dcd6b.tar.xz
kutter-c1bd628ce524f52851cf2bbcb4bc8bbc373dcd6b.zip
command: Directly call command_sendf() for ack/nak messages
Don't use the sendf() macro for ack and nak messages - directly call the command_sendf() code instead. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
Diffstat (limited to 'scripts')
-rw-r--r--scripts/buildcommands.py10
1 files changed, 1 insertions, 9 deletions
diff --git a/scripts/buildcommands.py b/scripts/buildcommands.py
index a26801ee..0c147191 100644
--- a/scripts/buildcommands.py
+++ b/scripts/buildcommands.py
@@ -29,9 +29,6 @@ def error(msg):
######################################################################
def build_parser(parser, iscmd, all_param_types):
- if parser.name == "#empty":
- return "\n // Empty message\n .max_size=%d," % (
- msgproto.MESSAGE_MIN,)
if parser.name == "#output":
comment = "Output: " + parser.msgformat
else:
@@ -71,8 +68,6 @@ def build_encoders(encoders, msg_to_id, all_param_types):
if msgid in did_output:
continue
s = msg
- if s == '#empty':
- s = ''
did_output[msgid] = True
code = (' if (__builtin_strcmp(str, "%s") == 0)\n'
' return &command_encoder_%s;\n' % (s, msgid))
@@ -308,10 +303,7 @@ def main():
error("Conflicting definition for command '%s'" % msgname)
messages_by_name[msgname] = msg
elif cmd == '_DECL_ENCODER':
- if len(parts) == 1:
- msgname = msg = "#empty"
- else:
- msgname = parts[1]
+ msgname = parts[1]
m = messages_by_name.get(msgname)
if m is not None and m != msg:
error("Conflicting definition for message '%s'" % msgname)