From 76f07979bbed5d4b05bb4ba22733c4a676a19d72 Mon Sep 17 00:00:00 2001 From: Kevin O'Connor Date: Sun, 18 Nov 2018 10:54:47 -0500 Subject: [PATCH] buildcommands: Make sure to not use 96 or more message ids The mcu code assumes the encoder and parser id is always one byte. Signed-off-by: Kevin O'Connor --- scripts/buildcommands.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/scripts/buildcommands.py b/scripts/buildcommands.py index 09ecad70..97f1513a 100644 --- a/scripts/buildcommands.py +++ b/scripts/buildcommands.py @@ -168,6 +168,9 @@ class HandleCommandGeneration: if msg not in self.msg_to_id: msgid += 1 self.msg_to_id[msg] = msgid + if msgid >= 96: + # The mcu currently assumes all message ids encode to one byte + error("Too many message ids") def update_data_dictionary(self, data): messages = { msgid: msg for msg, msgid in self.msg_to_id.items() } data['messages'] = messages