diff options
author | Alan Coopersmith <alan.coopersmith@oracle.com> | 2018-03-09 20:31:22 -0800 |
---|---|---|
committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2018-03-09 20:34:37 -0800 |
commit | bffe5030b03af5afb67dbeb053016ffc7569f48a (patch) | |
tree | 9037065ad3e9c19cc6e6b5a42a4820e5711d4a7e | |
parent | 385e447e3aecd40db2b0f2110308663b3bc478f6 (diff) |
Fix -Wsign-compare warnings
xstdcmap.c: In function ‘parse’:
xstdcmap.c:152:14: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (i=0; i < NPROPERTIES; i++) {
^
xstdcmap.c: In function ‘getDeepestVisual’:
xstdcmap.c:220:51: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
if (vinfo->class == visual_class && vinfo->depth > maxdepth)
^
xstdcmap.c: In function ‘doIndividualColormaps’:
xstdcmap.c:288:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
for (i=0; i < NPROPERTIES; i++) {
^
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
-rw-r--r-- | xstdcmap.c | 7 |
1 files changed, 4 insertions, 3 deletions
@@ -148,7 +148,7 @@ parse(int argc, char **argv) snprintf(option, sizeof(option), "%s%s", program_name, ".delete"); if (XrmGetResource(database, option, (char *) NULL, &type, &value)) { - register int i; + unsigned int i; for (i=0; i < NPROPERTIES; i++) { if (strcmp((char *) value.addr, propertyTable[i].nickname) == 0) { propertyTable[i].delete++; @@ -213,7 +213,7 @@ getDeepestVisual(int visual_class, /* specifies the desired visual class */ int nvisuals) /* specifies number of visuals in the list */ { register int i; - unsigned int maxdepth = 0; + int maxdepth = 0; XVisualInfo *v = NULL; for (i=0; i < nvisuals; i++, vinfo++) @@ -276,7 +276,8 @@ visualStringFromClass(int class) static int doIndividualColormaps(void) { - int i, screen, nvisuals; + unsigned int i; + int screen, nvisuals; Status status = -1; XVisualInfo *vinfo = NULL, *v = NULL, template; |