changeset 617:a3d5ddbb6f16

Add some extra () protection in the th_is* macro wrapper arguments.
author Matti Hamalainen <ccr@tnsp.org>
date Fri, 17 Jan 2020 03:21:05 +0200
parents 594f197f7005
children 64fd686197d4
files th_string.h
diffstat 1 files changed, 18 insertions(+), 17 deletions(-) [+]
line wrap: on
line diff
--- a/th_string.h	Fri Jan 17 03:18:40 2020 +0200
+++ b/th_string.h	Fri Jan 17 03:21:05 2020 +0200
@@ -19,27 +19,28 @@
 extern "C" {
 #endif
 
+
 /** @def String utility wrapper macros
  */
-#define th_isalnum(c)   isalnum((int)(unsigned char) c)
-#define th_isalpha(c)   isalpha((int)(unsigned char) c)
-#define th_isascii(c)   isascii((int)(unsigned char) c)
-#define th_isblank(c)   isblank((int)(unsigned char) c)
-#define th_iscntrl(c)   iscntrl((int)(unsigned char) c)
-#define th_isdigit(c)   isdigit((int)(unsigned char) c)
-#define th_isgraph(c)   isgraph((int)(unsigned char) c)
-#define th_islower(c)   islower((int)(unsigned char) c)
-#define th_isprint(c)   isprint((int)(unsigned char) c)
-#define th_ispunct(c)   ispunct((int)(unsigned char) c)
-#define th_isspace(c)   isspace((int)(unsigned char) c)
-#define th_isupper(c)   isupper((int)(unsigned char) c)
-#define th_isxdigit(c)  isxdigit((int)(unsigned char) c)
-#define th_iscrlf(c)    ((c=='\r')||(c=='\n'))
+#define th_isalnum(c)   isalnum((int)(unsigned char) (c))
+#define th_isalpha(c)   isalpha((int)(unsigned char) (c))
+#define th_isascii(c)   isascii((int)(unsigned char) (c))
+#define th_isblank(c)   isblank((int)(unsigned char) (c))
+#define th_iscntrl(c)   iscntrl((int)(unsigned char) (c))
+#define th_isdigit(c)   isdigit((int)(unsigned char) (c))
+#define th_isgraph(c)   isgraph((int)(unsigned char) (c))
+#define th_islower(c)   islower((int)(unsigned char) (c))
+#define th_isprint(c)   isprint((int)(unsigned char) (c))
+#define th_ispunct(c)   ispunct((int)(unsigned char) (c))
+#define th_isspace(c)   isspace((int)(unsigned char) (c))
+#define th_isupper(c)   isupper((int)(unsigned char) (c))
+#define th_isxdigit(c)  isxdigit((int)(unsigned char) (c))
+#define th_iscrlf(c)    ((c) == '\r' || (c) == '\n')
 
-#define th_isspecial(q) (((q >= 0x5b) && (q <= 0x60)) || ((q >= 0x7b) && (q <= 0x7d)))
+#define th_isspecial(q) ((q) >= 0x5b && (q) <= 0x60 || (q) >= 0x7b && (q) <= 0x7d)
 
-#define th_tolower(c)   tolower((int)(unsigned char) c)
-#define th_toupper(c)   toupper((int)(unsigned char) c)
+#define th_tolower(c)   tolower((int)(unsigned char) (c))
+#define th_toupper(c)   toupper((int)(unsigned char) (c))
 
 
 /** @brief