From 57b1cd5f6b06fbecf625bbd06d27cf0c606e3d0a Mon Sep 17 00:00:00 2001 From: Jilles Tjoelker Date: Thu, 15 Mar 2012 00:58:56 +0100 Subject: [PATCH] strip_colour: Prefer IsDigit to isdigit. Unlike isdigit, IsDigit can safely be applied to 'char's. Arguments to isdigit needs a character cast to 'unsigned char' or EOF. --- include/inline/stringops.h | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/include/inline/stringops.h b/include/inline/stringops.h index 61fe140c..9b74bfd3 100644 --- a/include/inline/stringops.h +++ b/include/inline/stringops.h @@ -39,15 +39,15 @@ strip_colour(char *string) switch (*c) { case 3: - if(isdigit(c[1])) + if(IsDigit(c[1])) { c++; - if(isdigit(c[1])) + if(IsDigit(c[1])) c++; - if(c[1] == ',' && isdigit(c[2])) + if(c[1] == ',' && IsDigit(c[2])) { c += 2; - if(isdigit(c[1])) + if(IsDigit(c[1])) c++; } } @@ -91,15 +91,15 @@ strip_unprintable(char *string) switch (*c) { case 3: - if(isdigit(c[1])) + if(IsDigit(c[1])) { c++; - if(isdigit(c[1])) + if(IsDigit(c[1])) c++; - if(c[1] == ',' && isdigit(c[2])) + if(c[1] == ',' && IsDigit(c[2])) { c += 2; - if(isdigit(c[1])) + if(IsDigit(c[1])) c++; } }