You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by rj...@apache.org on 2007/06/20 22:40:45 UTC
svn commit: r549225 - in /tomcat/connectors/trunk/jk/native:
common/jk_ajp12_worker.c common/jk_connect.c common/jk_lb_worker.c
nt_service/jk_nt_service.c
Author: rjung
Date: Wed Jun 20 13:40:44 2007
New Revision: 549225
URL: http://svn.apache.org/viewvc?view=rev&rev=549225
Log:
Fix a couple of compiler warnings
related to type conversion from chars to ints
for isspace(), isdigit() etc.
Modified:
tomcat/connectors/trunk/jk/native/common/jk_ajp12_worker.c
tomcat/connectors/trunk/jk/native/common/jk_connect.c
tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c
tomcat/connectors/trunk/jk/native/nt_service/jk_nt_service.c
Modified: tomcat/connectors/trunk/jk/native/common/jk_ajp12_worker.c
URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/common/jk_ajp12_worker.c?view=diff&rev=549225&r1=549224&r2=549225
==============================================================================
--- tomcat/connectors/trunk/jk/native/common/jk_ajp12_worker.c (original)
+++ tomcat/connectors/trunk/jk/native/common/jk_ajp12_worker.c Wed Jun 20 13:40:44 2007
@@ -542,7 +542,7 @@
}
name = line;
- while (isspace(*name) && *name) {
+ while (isspace((int)(*name)) && *name) {
name++; /* Skip leading white chars */
}
if (!*name) { /* Empty header name */
@@ -557,7 +557,7 @@
}
*value = '\0';
value++;
- while (isspace(*value) && *value) {
+ while (isspace((int)(*value)) && *value) {
value++; /* Skip leading white chars */
}
if (!*value) { /* Empty header value */
Modified: tomcat/connectors/trunk/jk/native/common/jk_connect.c
URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/common/jk_connect.c?view=diff&rev=549225&r1=549224&r2=549225
==============================================================================
--- tomcat/connectors/trunk/jk/native/common/jk_connect.c (original)
+++ tomcat/connectors/trunk/jk/native/common/jk_connect.c Wed Jun 20 13:40:44 2007
@@ -257,7 +257,7 @@
/* Check if we only have digits in the string */
for (x = 0; host[x] != '\0'; x++) {
- if (!isdigit(host[x]) && host[x] != '.') {
+ if (!isdigit((int)(host[x])) && host[x] != '.') {
break;
}
}
Modified: tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c
URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c?view=diff&rev=549225&r1=549224&r2=549225
==============================================================================
--- tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c (original)
+++ tomcat/connectors/trunk/jk/native/common/jk_lb_worker.c Wed Jun 20 13:40:44 2007
@@ -347,9 +347,9 @@
id_start; id_start = strstr(id_start + 1, name)) {
if (id_start == s->headers_values[i] ||
id_start[-1] == ';' ||
- id_start[-1] == ',' || isspace(id_start[-1])) {
+ id_start[-1] == ',' || isspace((int)id_start[-1])) {
id_start += strlen(name);
- while (*id_start && isspace(*id_start))
+ while (*id_start && isspace((int)(*id_start)))
++id_start;
if (*id_start == '=' && id_start[1]) {
/*
Modified: tomcat/connectors/trunk/jk/native/nt_service/jk_nt_service.c
URL: http://svn.apache.org/viewvc/tomcat/connectors/trunk/jk/native/nt_service/jk_nt_service.c?view=diff&rev=549225&r1=549224&r2=549225
==============================================================================
--- tomcat/connectors/trunk/jk/native/nt_service/jk_nt_service.c (original)
+++ tomcat/connectors/trunk/jk/native/nt_service/jk_nt_service.c Wed Jun 20 13:40:44 2007
@@ -382,7 +382,7 @@
if (dst >= szTrueName + sizeof(szTrueName) - 1) {
break;
}
- if (!isspace(*src) && *src != '/' && *src != '\\') {
+ if (!isspace((int)(*src)) && *src != '/' && *src != '\\') {
*(dst++) = *src;
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org