diff options
| author | Thomas Klausner <wiz@NetBSD.org> | 2015-02-25 21:45:50 +0100 |
|---|---|---|
| committer | Adam Jackson <ajax@redhat.com> | 2015-10-20 12:52:08 -0400 |
| commit | d66f107d6e714a54515f39d94caf46aef9be7416 (patch) | |
| tree | ed0b265d971ce9c72c1471e5a227a3bb3c112207 /src/FreeType | |
| parent | 1a73d6828dfa03924f2d68644fb5b99afd9c78e2 (diff) | |
| download | xorg-lib-libXfont-d66f107d6e714a54515f39d94caf46aef9be7416.tar.gz | |
Fix is*() usage.
The argument must be an unsigned char or -1; in these cases
we know it's not -1 so cast it to unsigned char.
Fixes
warning: array subscript is of type 'char' [-Wchar-subscripts]
Signed-off-by: Thomas Klausner <wiz@NetBSD.org>
Diffstat (limited to 'src/FreeType')
| -rw-r--r-- | src/FreeType/ftfuncs.c | 6 | ||||
| -rw-r--r-- | src/FreeType/xttcap.c | 2 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/FreeType/ftfuncs.c b/src/FreeType/ftfuncs.c index a4969d1..c440fde 100644 --- a/src/FreeType/ftfuncs.c +++ b/src/FreeType/ftfuncs.c @@ -2061,7 +2061,7 @@ restrict_code_range_by_str(int count,unsigned short *refFirstCol, long val; /* skip comma and/or space */ - while (',' == *p || isspace(*p)) + while (',' == *p || isspace((unsigned char)*p)) p++; /* begin point */ @@ -2079,7 +2079,7 @@ restrict_code_range_by_str(int count,unsigned short *refFirstCol, } /* skip space */ - while (isspace(*p)) + while (isspace((unsigned char)*p)) p++; if (',' != *p && '\0' != *p) { @@ -2092,7 +2092,7 @@ restrict_code_range_by_str(int count,unsigned short *refFirstCol, break; /* skip space */ - while (isspace(*p)) + while (isspace((unsigned char)*p)) p++; val = strtol(p, (char **)&q, 0); diff --git a/src/FreeType/xttcap.c b/src/FreeType/xttcap.c index cee752e..e30e0f9 100644 --- a/src/FreeType/xttcap.c +++ b/src/FreeType/xttcap.c @@ -632,7 +632,7 @@ SPropRecValList_add_by_font_cap(SDynPropRecValList *pThisList, } break; } - if ( !isdigit(*p) ) + if ( !isdigit((unsigned char)*p) ) break; } } |