diff options
author | Damien George <damien.p.george@gmail.com> | 2014-08-30 15:17:47 +0100 |
---|---|---|
committer | Damien George <damien.p.george@gmail.com> | 2014-08-30 15:17:47 +0100 |
commit | ca6d75f16d8fc465ddaa4dc90a6f87ab9667b12e (patch) | |
tree | 3392cc02838cecddcdc99d86fd808cb563f86d95 /py/obj.c | |
parent | 4abff7500fb05430a2ce952a2430d4d0ba16047e (diff) | |
download | micropython-ca6d75f16d8fc465ddaa4dc90a6f87ab9667b12e.tar.gz micropython-ca6d75f16d8fc465ddaa4dc90a6f87ab9667b12e.zip |
py: Small simplifications in tuple and list accessors.
Diffstat (limited to 'py/obj.c')
-rw-r--r-- | py/obj.c | 16 |
1 files changed, 4 insertions, 12 deletions
@@ -319,18 +319,10 @@ void mp_obj_get_array(mp_obj_t o, mp_uint_t *len, mp_obj_t **items) { } void mp_obj_get_array_fixed_n(mp_obj_t o, mp_uint_t len, mp_obj_t **items) { - if (MP_OBJ_IS_TYPE(o, &mp_type_tuple) || MP_OBJ_IS_TYPE(o, &mp_type_list)) { - mp_uint_t seq_len; - if (MP_OBJ_IS_TYPE(o, &mp_type_tuple)) { - mp_obj_tuple_get(o, &seq_len, items); - } else { - mp_obj_list_get(o, &seq_len, items); - } - if (seq_len != len) { - nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_ValueError, "requested length %d but object has length %d", len, seq_len)); - } - } else { - nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_TypeError, "object '%s' is not a tuple or list", mp_obj_get_type_str(o))); + mp_uint_t seq_len; + mp_obj_get_array(o, &seq_len, items); + if (seq_len != len) { + nlr_raise(mp_obj_new_exception_msg_varg(&mp_type_ValueError, "requested length %d but object has length %d", len, seq_len)); } } |