From 1431278aba3347fdb55ec461008a07d9010aea7a Mon Sep 17 00:00:00 2001 From: hunt Date: Fri, 9 Sep 2005 09:12:37 +0000 Subject: 2005-09-09 Martin Hunt * stat-common.c (_stp_stat_print_valtype): Use _stp_div64(). * arith.c (_stp_div64): Check error before writing to it. Remove check against 32-bit LONG_MIN and -1. That only applies to 64-bit. (_stp_mod64): Ditto. --- runtime/arith.c | 26 ++++++++++++-------------- 1 file changed, 12 insertions(+), 14 deletions(-) (limited to 'runtime/arith.c') diff --git a/runtime/arith.c b/runtime/arith.c index ee75e4f8..abbac66c 100644 --- a/runtime/arith.c +++ b/runtime/arith.c @@ -32,24 +32,23 @@ int64_t _stp_div64 (const char **error, int64_t x, int64_t y) { #ifdef __LP64__ if (unlikely (y == 0 || (x == LONG_MIN && y == -1))) { - *error = "divisor out of range"; + if (error) *error = "divisor out of range"; return 0; } return x/y; #else - if (likely ((x >= LONG_MIN && x <= LONG_MAX) && (y >= LONG_MIN && y <= LONG_MAX))) { + if (likely ((x > LONG_MIN && x < LONG_MAX) && (y > LONG_MIN && y < LONG_MAX))) { long xx = (long) x; long yy = (long) y; - // check for division-by-zero and overflow - if (unlikely (yy == 0 || (xx == LONG_MIN && yy == -1))) { - *error = "divisor out of range"; + // check for division-by-zero + if (unlikely (yy == 0 )) { + if (error) *error = "division by 0"; return 0; } return xx / yy; - } else { + } else return _div64 (x, y); - } #endif } @@ -61,26 +60,25 @@ int64_t _stp_mod64 (const char **error, int64_t x, int64_t y) { #ifdef __LP64__ if (unlikely (y == 0 || (x == LONG_MIN && y == -1))) { - *error = "divisor out of range"; + if (error) *error = "divisor out of range"; return 0; } return x%y; #else - if (likely ((x >= LONG_MIN && x <= LONG_MAX) && (y >= LONG_MIN && y <= LONG_MAX))) { + if (likely ((x > LONG_MIN && x < LONG_MAX) && (y > LONG_MIN && y < LONG_MAX))) { long xx = (long) x; long yy = (long) y; - // check for division-by-zero and overflow - if (unlikely (yy == 0 || (xx == LONG_MIN && yy == -1))) { - *error = "divisor out of range"; + // check for division-by-zero + if (unlikely (yy == 0)) { + if (error) *error = "division by 0"; return 0; } return xx % yy; - } else { + } else return _mod64 (x,y); - } #endif } -- cgit