diff options
| author | Damien George <damien.p.george@gmail.com> | 2017-08-29 13:04:01 +1000 |
|---|---|---|
| committer | Damien George <damien.p.george@gmail.com> | 2017-08-29 13:16:30 +1000 |
| commit | 58321dd9854d71a96e5db2d361e0efc05d9de8cf (patch) | |
| tree | 99868b98083b1b25715303ee6f2b62835492ffce /py/objset.c | |
| parent | be8e5744e64aec10a3499fd6ea034bbf4be0c871 (diff) | |
| download | micropython-58321dd9854d71a96e5db2d361e0efc05d9de8cf.tar.gz micropython-58321dd9854d71a96e5db2d361e0efc05d9de8cf.zip | |
all: Convert mp_uint_t to mp_unary_op_t/mp_binary_op_t where appropriate
The unary-op/binary-op enums are already defined, and there are no
arithmetic tricks used with these types, so it makes sense to use the
correct enum type for arguments that take these values. It also reduces
code size quite a bit for nan-boxing builds.
Diffstat (limited to 'py/objset.c')
| -rw-r--r-- | py/objset.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/py/objset.c b/py/objset.c index 376439b73e..d4a8a1a39b 100644 --- a/py/objset.c +++ b/py/objset.c @@ -455,7 +455,7 @@ STATIC mp_obj_t set_union(mp_obj_t self_in, mp_obj_t other_in) { } STATIC MP_DEFINE_CONST_FUN_OBJ_2(set_union_obj, set_union); -STATIC mp_obj_t set_unary_op(mp_uint_t op, mp_obj_t self_in) { +STATIC mp_obj_t set_unary_op(mp_unary_op_t op, mp_obj_t self_in) { mp_obj_set_t *self = MP_OBJ_TO_PTR(self_in); switch (op) { case MP_UNARY_OP_BOOL: return mp_obj_new_bool(self->set.used != 0); @@ -480,7 +480,7 @@ STATIC mp_obj_t set_unary_op(mp_uint_t op, mp_obj_t self_in) { } } -STATIC mp_obj_t set_binary_op(mp_uint_t op, mp_obj_t lhs, mp_obj_t rhs) { +STATIC mp_obj_t set_binary_op(mp_binary_op_t op, mp_obj_t lhs, mp_obj_t rhs) { mp_obj_t args[] = {lhs, rhs}; #if MICROPY_PY_BUILTINS_FROZENSET bool update = MP_OBJ_IS_TYPE(lhs, &mp_type_set); |