diff options
author | Sebastian Koslowski <koslowski@kit.edu> | 2016-04-27 14:19:04 +0200 |
---|---|---|
committer | Sebastian Koslowski <koslowski@kit.edu> | 2016-04-27 16:47:08 +0200 |
commit | c85984f105106ff0a7e3b387d680e0f2f5884d55 (patch) | |
tree | c29638545c98d88bf59b09771ff39c4d8e557469 /grc/core/generator/Generator.py | |
parent | ec7ef7fe2942bf73e7ff9166530d25dd605a1fce (diff) |
grc: fix callback evaluation
Diffstat (limited to 'grc/core/generator/Generator.py')
-rw-r--r-- | grc/core/generator/Generator.py | 25 |
1 files changed, 11 insertions, 14 deletions
diff --git a/grc/core/generator/Generator.py b/grc/core/generator/Generator.py index 91671072d6..641bdea6e6 100644 --- a/grc/core/generator/Generator.py +++ b/grc/core/generator/Generator.py @@ -141,12 +141,12 @@ class TopBlockGenerator(object): pass return code - blocks = expr_utils.sort_objects( + blocks_all = expr_utils.sort_objects( filter(lambda b: b.get_enabled() and not b.get_bypassed(), fg.blocks), lambda b: b.get_id(), _get_block_sort_text ) # List of regular blocks (all blocks minus the special ones) - blocks = filter(lambda b: b not in (imports + parameters), blocks) + blocks = filter(lambda b: b not in (imports + parameters), blocks_all) for block in blocks: key = block.get_key() @@ -212,18 +212,15 @@ class TopBlockGenerator(object): msgs = filter(lambda c: c.is_msg(), fg.get_enabled_connections()) # List of variable names var_ids = [var.get_id() for var in parameters + variables] - # Prepend self. - replace_dict = dict([(var_id, 'self.%s' % var_id) for var_id in var_ids]) - # List of callbacks - callbacks = [ - expr_utils.expr_replace(cb, replace_dict) - for cb in sum([block.get_callbacks() for block in fg.get_enabled_blocks()], []) - ] + replace_dict = dict((var_id, 'self.' + var_id) for var_id in var_ids) + callbacks_all = [] + for block in blocks_all: + callbacks_all.extend(expr_utils.expr_replace(cb, replace_dict) for cb in block.get_callbacks()) # Map var id to callbacks - var_id2cbs = dict([ - (var_id, filter(lambda c: expr_utils.get_variable_dependencies(c, [var_id]), callbacks)) - for var_id in var_ids - ]) + callbacks = {} + for var_id in var_ids: + callbacks[var_id] = [callback for callback in callbacks_all + if expr_utils.get_variable_dependencies(callback, ['self.' + var_id])] # Load the namespace namespace = { 'title': title, @@ -237,7 +234,7 @@ class TopBlockGenerator(object): 'connection_templates': connection_templates, 'msgs': msgs, 'generate_options': self._generate_options, - 'var_id2cbs': var_id2cbs, + 'callbacks': callbacks, } # Build the template t = Template(open(FLOW_GRAPH_TEMPLATE, 'r').read(), namespace) |