diff options
| author | Yukihiro "Matz" Matsumoto <[email protected]> | 2021-08-06 11:15:46 +0900 |
|---|---|---|
| committer | Yukihiro "Matz" Matsumoto <[email protected]> | 2021-08-06 11:15:46 +0900 |
| commit | ff2da0b66e38bc55a04149d8012e85984202f121 (patch) | |
| tree | 320fddf5d458d03606d5e18df4628dcf45a40a81 /src/numeric.c | |
| parent | 37dbb91b9614eb287729ecf0ef10e679f6b61aa1 (diff) | |
| download | mruby-ff2da0b66e38bc55a04149d8012e85984202f121.tar.gz mruby-ff2da0b66e38bc55a04149d8012e85984202f121.zip | |
numeric.c: fix a bug regarding `MRB_INT_MIN`.
Diffstat (limited to 'src/numeric.c')
| -rw-r--r-- | src/numeric.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/numeric.c b/src/numeric.c index ded2880ff..5fbf9c586 100644 --- a/src/numeric.c +++ b/src/numeric.c @@ -1117,8 +1117,10 @@ int_mod(mrb_state *mrb, mrb_value x) mrb_int a, b; a = mrb_integer(x); - if (mrb_integer_p(y) && a != MRB_INT_MIN && (b=mrb_integer(y)) != MRB_INT_MIN) { + if (mrb_integer_p(y)) { + b = mrb_integer(y); if (b == 0) int_zerodiv(mrb); + if (a == MRB_INT_MIN && b == -1) return mrb_fixnum_value(0); mrb_int mod = a % b; if ((a < 0) != (b < 0) && mod != 0) { mod += b; @@ -1128,12 +1130,10 @@ int_mod(mrb_state *mrb, mrb_value x) #ifdef MRB_NO_FLOAT mrb_raise(mrb, E_TYPE_ERROR, "non integer modulo"); #else - else { - mrb_float mod; + mrb_float mod; - flodivmod(mrb, (mrb_float)a, mrb_as_float(mrb, y), NULL, &mod); - return mrb_float_value(mrb, mod); - } + flodivmod(mrb, (mrb_float)a, mrb_as_float(mrb, y), NULL, &mod); + return mrb_float_value(mrb, mod); #endif } |