commit ea8d06c39d9c94036e490b300d46f367c9eb78d9
author Damien George <damien.p.george@gmail.com> Thu Apr 17 23:19:36 2014 +0100
committer Damien George <damien.p.george@gmail.com> Thu Apr 17 23:19:36 2014 +0100
tree ce4999f60f6ca57f22e7d536593a6198ceee0695
parent 1e935d8689f3d15dc3bd06f08f2a0305b7e1c7f4

py: Add MP_OBJ_STOP_ITERATION and make good use of it.

Also make consistent use of MP_OBJ_NOT_SUPPORTED and MP_OBJ_NULL.
This helps a lot in debugging and understanding of function API.

py/objarray.c

diff --git a/py/objarray.c b/py/objarray.c
index 0011948..5c160bf 100644
--- a/py/objarray.c
+++ b/py/objarray.c
@@ -63,7 +63,7 @@
     mp_obj_t iterable = mp_getiter(initializer);
     mp_obj_t item;
     int i = 0;
-    while ((item = mp_iternext(iterable)) != MP_OBJ_NULL) {
+    while ((item = mp_iternext(iterable)) != MP_OBJ_STOP_ITERATION) {
         if (len == 0) {
             array_append(array, item);
         } else {
@@ -108,7 +108,7 @@
     switch (op) {
         case MP_UNARY_OP_BOOL: return MP_BOOL(o->len != 0);
         case MP_UNARY_OP_LEN: return MP_OBJ_NEW_SMALL_INT(o->len);
-        default: return MP_OBJ_NULL; // op not supported
+        default: return MP_OBJ_NOT_SUPPORTED;
     }
 }
 
@@ -227,7 +227,7 @@
     if (self->cur < self->array->len) {
         return mp_binary_get_val_array(self->array->typecode, self->array->items, self->cur++);
     } else {
-        return MP_OBJ_NULL;
+        return MP_OBJ_STOP_ITERATION;
     }
 }