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/objreversed.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/objreversed.c')
| -rw-r--r-- | py/objreversed.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/py/objreversed.c b/py/objreversed.c index b0da00a88..1e54bb522 100644 --- a/py/objreversed.c +++ b/py/objreversed.c @@ -38,7 +38,7 @@ typedef struct _mp_obj_reversed_t { mp_uint_t cur_index; // current index, plus 1; 0=no more, 1=last one (index 0) } mp_obj_reversed_t; -STATIC mp_obj_t reversed_make_new(mp_obj_t type_in, size_t n_args, size_t n_kw, const mp_obj_t *args) { +STATIC mp_obj_t reversed_make_new(const mp_obj_type_t *type, size_t n_args, size_t n_kw, const mp_obj_t *args) { mp_arg_check_num(n_args, n_kw, 1, 1, false); // check if __reversed__ exists, and if so delegate to it @@ -49,7 +49,7 @@ STATIC mp_obj_t reversed_make_new(mp_obj_t type_in, size_t n_args, size_t n_kw, } mp_obj_reversed_t *o = m_new_obj(mp_obj_reversed_t); - o->base.type = MP_OBJ_TO_PTR(type_in); + o->base.type = type; o->seq = args[0]; o->cur_index = mp_obj_get_int(mp_obj_len(args[0])); // start at the end of the sequence |