util.c: atos_symbolize: Don't use locale-dependent fcns
authorKarl Williamson <khw@cpan.org>
Sat, 22 Jul 2017 21:09:28 +0000 (15:09 -0600)
committerKarl Williamson <khw@cpan.org>
Sun, 10 Sep 2017 05:05:11 +0000 (23:05 -0600)
It very likely isn't an issue here, but the core should not have
unintentional locale-dependent code .

util.c

diff --git a/util.c b/util.c
index 4584c73..0fdea7d 100644 (file)
--- a/util.c
+++ b/util.c
@@ -6096,17 +6096,17 @@ static const char* atos_parse(const char* p,
     UV uv;
 
     /* Skip trailing whitespace. */
-    while (p > start && isspace(*p)) p--;
+    while (p > start && isSPACE(*p)) p--;
     /* Now we should be at the close paren. */
     if (p == start || *p != ')')
         return NULL;
     close_paren = p;
     p--;
     /* Now we should be in the line number. */
-    if (p == start || !isdigit(*p))
+    if (p == start || !isDIGIT(*p))
         return NULL;
     /* Skip over the digits. */
-    while (p > start && isdigit(*p))
+    while (p > start && isDIGIT(*p))
         p--;
     /* Now we should be at the colon. */
     if (p == start || *p != ':')
@@ -6151,7 +6151,7 @@ static void atos_symbolize(atos_context* ctx,
      * the object name (used as "-o '%s'" ), leave since at least
      * partially the user controls it. */
     for (p = ctx->fname; *p; p++) {
-        if (*p == '\'' || iscntrl(*p)) {
+        if (*p == '\'' || isCNTRL(*p)) {
             ctx->unavail = TRUE;
             return;
         }