diff options
author | Damien George <damien.p.george@gmail.com> | 2016-01-03 15:55:55 +0000 |
---|---|---|
committer | Damien George <damien.p.george@gmail.com> | 2016-01-11 00:49:27 +0000 |
commit | 5b3f0b7f39bd67cc9182993c288f09f67a0890df (patch) | |
tree | d26730229745746b47e536a789a54f989d4ecffd /py/objdict.c | |
parent | 4b72b3a133ea87a1ef8b964508dc25c551ccf093 (diff) |
py: Change first arg of type.make_new from mp_obj_t to mp_obj_type_t*.
The first argument to the type.make_new method is naturally a uPy type,
and all uses of this argument cast it directly to a pointer to a type
structure. So it makes sense to just have it a pointer to a type from
the very beginning (and a const pointer at that). This patch makes
such a change, and removes all unnecessary casting to/from mp_obj_t.
Diffstat (limited to 'py/objdict.c')
-rw-r--r-- | py/objdict.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/py/objdict.c b/py/objdict.c index cb4c602b9..cc1f502d0 100644 --- a/py/objdict.c +++ b/py/objdict.c @@ -82,12 +82,12 @@ STATIC void dict_print(const mp_print_t *print, mp_obj_t self_in, mp_print_kind_ } } -STATIC mp_obj_t dict_make_new(mp_obj_t type_in, size_t n_args, size_t n_kw, const mp_obj_t *args) { +STATIC mp_obj_t dict_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *args) { mp_obj_t dict_out = mp_obj_new_dict(0); mp_obj_dict_t *dict = MP_OBJ_TO_PTR(dict_out); - dict->base.type = MP_OBJ_TO_PTR(type_in); + dict->base.type = type; #if MICROPY_PY_COLLECTIONS_ORDEREDDICT - if (MP_OBJ_TO_PTR(type_in) == &mp_type_ordereddict) { + if (type == &mp_type_ordereddict) { dict->map.is_ordered = 1; } #endif |