From fa931b46eb7737be619c9c6e02be1111c17f0f89 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?S=C3=A9bastien=20Helleu?= Date: Mon, 23 Mar 2020 07:23:08 +0100 Subject: [PATCH] core: fix compiler warnings on Cygwin --- src/core/wee-calc.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/core/wee-calc.c b/src/core/wee-calc.c index cba170264..175cb9913 100644 --- a/src/core/wee-calc.c +++ b/src/core/wee-calc.c @@ -202,7 +202,7 @@ calc_format_result (double value, char *result, int max_size) i = strlen (result) - 1; while (i >= 0) { - if (!isdigit (result[i]) && (result[i] != '-')) + if (!isdigit ((unsigned char)result[i]) && (result[i] != '-')) { result[i] = '\0'; break; @@ -282,7 +282,7 @@ calc_expression (const char *expr) arraylist_add (list_ops, ptr_operator); last_symbol = CALC_SYMBOL_PARENTHESIS_OPEN; } - else if (isdigit (expr[i]) || (expr[i] == '.') + else if (isdigit ((unsigned char)expr[i]) || (expr[i] == '.') || ((expr[i] == '-') && ((last_symbol == CALC_SYMBOL_NONE) || (last_symbol == CALC_SYMBOL_PARENTHESIS_OPEN) @@ -296,7 +296,8 @@ calc_expression (const char *expr) factor = -1; i++; } - while (expr[i] && (isdigit (expr[i]) || (expr[i] == '.'))) + while (expr[i] + && (isdigit ((unsigned char)expr[i]) || (expr[i] == '.'))) { if (expr[i] == '.') { @@ -347,7 +348,7 @@ calc_expression (const char *expr) i2 = i + 1; while (expr[i2] && (expr[i2] != ' ') && (expr[i2] != '(') && (expr[i2] != ')') && (expr[i2] != '.') - && (expr[i2] != '-') && !isdigit (expr[i2])) + && (expr[i2] != '-') && !isdigit ((unsigned char)expr[i2])) { i2++; }