diff options
| author | Guy Harris <guy@alum.mit.edu> | 2014-10-16 23:41:29 -0700 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2014-10-17 06:41:58 +0000 |
| commit | 63ab36be9266f79d92ac37e632299c8f0e2244f1 (patch) | |
| tree | 37e3a280eb2a318c792ee01d39e6f69d438ba5df /ui/persfilepath_opt.c | |
| parent | 23b233d74185d5fb7721a4c4d97d150312c8e3e9 (diff) | |
| download | wireshark-63ab36be9266f79d92ac37e632299c8f0e2244f1.tar.gz | |
Don't use <ctype.h> macros, and eliminate an include of <ctype.h>.
This avoids locale-dependent tests, and fixes cases where we passed
signed char values to those macros (which is not safe with char being
signed, as it is on most, but not all, platforms).
Change-Id: Iea962cd7e4dfaf33c3c873ba43814cc6ed298d94
Reviewed-on: https://code.wireshark.org/review/4764
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'ui/persfilepath_opt.c')
| -rw-r--r-- | ui/persfilepath_opt.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/ui/persfilepath_opt.c b/ui/persfilepath_opt.c index cbd894e58f..d1569605a7 100644 --- a/ui/persfilepath_opt.c +++ b/ui/persfilepath_opt.c @@ -24,7 +24,6 @@ #include "config.h" #include <string.h> -#include <ctype.h> #include <errno.h> #include <glib.h> @@ -55,7 +54,7 @@ persfilepath_opt(int opt _U_, const char *optstr) * as we allow it in the preferences file, we might as well * allow it here). */ - while (isspace((guchar)*p)) + while (g_ascii_isspace(*p)) p++; if (*p == '\0') { /* |