diff options
| author | Dan Halbert <halbert@halwitz.org> | 2018-07-28 14:13:54 -0400 |
|---|---|---|
| committer | Dan Halbert <halbert@halwitz.org> | 2018-07-28 14:13:54 -0400 |
| commit | 2e260a0eab80dfd47fe319edb3173d112960a862 (patch) | |
| tree | 75a5c1b0da652c81832eaf30ce845463b51ed9b5 /py/gc_long_lived.c | |
| parent | f48b70050e3e4122eaae3eee47d35fc30bda2a4c (diff) | |
| parent | fbfc9c39957b389fd359ed8107d1e244e9749930 (diff) | |
Merge remote-tracking branch 'adafruit/master' into micropython-25ae98f-merge
Diffstat (limited to 'py/gc_long_lived.c')
| -rwxr-xr-x | py/gc_long_lived.c | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/py/gc_long_lived.c b/py/gc_long_lived.c index e30b389fa..3c54de7ed 100755 --- a/py/gc_long_lived.c +++ b/py/gc_long_lived.c @@ -91,6 +91,13 @@ mp_obj_dict_t *make_dict_long_lived(mp_obj_dict_t *dict, uint8_t max_depth) { if (dict == NULL || max_depth == 0) { return dict; } + // Don't recurse unnecessarily. Return immediately if we've already seen this dict. + if (dict->map.scanning) { + return dict; + } + // Mark that we're processing this dict. + dict->map.scanning = 1; + // Update all of the references first so that we reduce the chance of references to the old // copies. dict->map.table = gc_make_long_lived(dict->map.table); @@ -100,7 +107,10 @@ mp_obj_dict_t *make_dict_long_lived(mp_obj_dict_t *dict, uint8_t max_depth) { dict->map.table[i].value = make_obj_long_lived(value, max_depth - 1); } } - return gc_make_long_lived(dict); + dict = gc_make_long_lived(dict); + // Done recursing through this dict. + dict->map.scanning = 0; + return dict; } mp_obj_str_t *make_str_long_lived(mp_obj_str_t *str) { |
