diff options
author | Damien George <damien.p.george@gmail.com> | 2018-04-05 01:11:26 +1000 |
---|---|---|
committer | Damien George <damien.p.george@gmail.com> | 2018-04-05 01:11:26 +1000 |
commit | f1df86a0177c77769d73bc477570580b4f705acf (patch) | |
tree | a404d728c466239741dfca2550979a2ff0c1f896 /py/objint_mpz.c | |
parent | 5995a199a327b81553189d53035c586d7b0249dc (diff) |
py/objint: Simplify LHS arg type checking in int binary op functions.
The LHS passed to mp_obj_int_binary_op() will always be an integer, either
a small int or a big int, so the test for this type doesn't need to include
an "other, unsupported type" case.
Diffstat (limited to 'py/objint_mpz.c')
-rw-r--r-- | py/objint_mpz.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/py/objint_mpz.c b/py/objint_mpz.c index 17e3ee6d2..0f05c84f4 100644 --- a/py/objint_mpz.c +++ b/py/objint_mpz.c @@ -170,11 +170,9 @@ mp_obj_t mp_obj_int_binary_op(mp_binary_op_t op, mp_obj_t lhs_in, mp_obj_t rhs_i if (MP_OBJ_IS_SMALL_INT(lhs_in)) { mpz_init_fixed_from_int(&z_int, z_int_dig, MPZ_NUM_DIG_FOR_INT, MP_OBJ_SMALL_INT_VALUE(lhs_in)); zlhs = &z_int; - } else if (MP_OBJ_IS_TYPE(lhs_in, &mp_type_int)) { - zlhs = &((mp_obj_int_t*)MP_OBJ_TO_PTR(lhs_in))->mpz; } else { - // unsupported type - return MP_OBJ_NULL; + assert(MP_OBJ_IS_TYPE(lhs_in, &mp_type_int)); + zlhs = &((mp_obj_int_t*)MP_OBJ_TO_PTR(lhs_in))->mpz; } // if rhs is small int, then lhs was not (otherwise mp_binary_op handles it) |