aboutsummaryrefslogtreecommitdiffstats
path: root/generator
diff options
context:
space:
mode:
authorPetteri Aimonen <jpa@git.mail.kapsi.fi>2012-06-30 19:28:49 +0300
committerPetteri Aimonen <jpa@git.mail.kapsi.fi>2012-06-30 19:28:49 +0300
commit78086cc27d746a425f3f1130e822275bdb623090 (patch)
treee22f1c33724651abe325cdad9c25fa9c1c815184 /generator
parent67add3259a6429cb4afc1bdb6345e8c057a55ca8 (diff)
Add PB_MANY_FIELDS option for supporting fields > 255.
Add generator warning if this is necessary. Fixes issue #14.
Diffstat (limited to 'generator')
-rw-r--r--generator/nanopb_generator.py38
1 files changed, 38 insertions, 0 deletions
diff --git a/generator/nanopb_generator.py b/generator/nanopb_generator.py
index 730c0aab..405fedaf 100644
--- a/generator/nanopb_generator.py
+++ b/generator/nanopb_generator.py
@@ -251,6 +251,20 @@ class Field:
result += '\n &%s_default}' % (self.struct_name + self.name)
return result
+
+ def needs_32bit_pb_field_t(self):
+ '''Determine if this field needs 32bit pb_field_t structure to compile properly.
+ Returns True, False or a C-expression for assert.'''
+ if self.tag > 255 or self.max_size > 255:
+ return True
+
+ if self.ltype == 'PB_LTYPE_SUBMESSAGE':
+ if self.htype == 'PB_HTYPE_ARRAY':
+ return 'pb_membersize(%s, %s[0]) > 255' % (self.struct_name, self.name)
+ else:
+ return 'pb_membersize(%s, %s) > 255' % (self.struct_name, self.name)
+
+ return False
class Message:
def __init__(self, names, desc):
@@ -412,6 +426,30 @@ def generate_header(dependencies, headername, enums, messages):
yield ' setting PB_MAX_REQUIRED_FIELDS to %d or more.\n' % largest_count
yield '#endif\n'
+ worst = False
+ worst_field = ''
+ for msg in messages:
+ for field in msg.fields:
+ status = field.needs_32bit_pb_field_t()
+ if status == True:
+ worst = True
+ worst_field = str(field.struct_name) + '.' + str(field.name)
+ elif status != False:
+ if worst == False:
+ worst = status
+ elif worst != True:
+ worst += ' || ' + status
+
+ if worst != False:
+ yield '\n/* Check that field information fits in pb_field_t */\n'
+ yield '#ifndef PB_MANY_FIELDS\n'
+ if worst == True:
+ yield '#error Field descriptor for %s is too large. Define PB_MANY_FIELDS to fix this.\n' % worst_field
+ else:
+ yield 'STATIC_ASSERT(!(%s), YOU_MUST_DEFINE_PB_MANY_FIELDS)\n' % worst
+ yield '#endif\n'
+
+ # End of header
yield '\n#endif\n'
def generate_source(headername, enums, messages):