commit b063b9b36d6d4bb8eed3140778c9969969db0ec6
author Damien George <damien.p.george@gmail.com> Sun Dec 21 16:24:09 2014 +0000
committer Damien George <damien.p.george@gmail.com> Sun Dec 21 16:24:09 2014 +0000
tree 26ee658baae1e8112602f9bb07ca0f03470d7138
parent 7b80d908bf5712de67fe37d33cf339781657a4a8

py: Fix iteration over map in 2 places.

py/runtime.c

diff --git a/py/runtime.c b/py/runtime.c
index 0690bd8..505c8e3 100644
--- a/py/runtime.c
+++ b/py/runtime.c
@@ -725,8 +725,8 @@
         // dictionary
         mp_map_t *map = mp_obj_dict_get_map(kw_dict);
         assert(args2_len + 2 * map->used <= args2_alloc); // should have enough, since kw_dict_len is in this case hinted correctly above
-        for (uint i = 0; i < map->alloc; i++) {
-            if (map->table[i].key != MP_OBJ_NULL) {
+        for (mp_uint_t i = 0; i < map->alloc; i++) {
+            if (MP_MAP_SLOT_IS_FILLED(map, i)) {
                 args2[args2_len++] = map->table[i].key;
                 args2[args2_len++] = map->table[i].value;
             }
@@ -1240,7 +1240,7 @@
 
     // TODO: Support __all__
     mp_map_t *map = mp_obj_dict_get_map(mp_obj_module_get_globals(module));
-    for (uint i = 0; i < map->alloc; i++) {
+    for (mp_uint_t i = 0; i < map->alloc; i++) {
         if (MP_MAP_SLOT_IS_FILLED(map, i)) {
             qstr name = MP_OBJ_QSTR_VALUE(map->table[i].key);
             if (*qstr_str(name) != '_') {