[jbossnative-commits] JBoss Native SVN: r966 - trunk/sight/native/share.

[jbossnative-commits at lists.jboss.org]

Author: mladen.turk at jboss.com
Date: 2007-09-03 04:22:55 -0400 (Mon, 03 Sep 2007)
New Revision: 966

Modified:
   trunk/sight/native/share/jnu.c
Log:
Fix JBNATIVE-43. We compare >= 0 and we cannot do that with size_t that is unsigned.

Modified: trunk/sight/native/share/jnu.c
===================================================================
--- trunk/sight/native/share/jnu.c	2007-09-03 07:01:53 UTC (rev 965)
+++ trunk/sight/native/share/jnu.c	2007-09-03 08:22:55 UTC (rev 966)
@@ -118,11 +118,11 @@
 
 char *sight_rtrim(char *s)
 {
-    size_t i;
+    int i;
     /* check for empty strings */
     if (!s || !*s)
         return NULL;
-    for (i = strlen(s) - 1; i >= 0 && (apr_iscntrl(s[i]) ||
+    for (i = (int)strlen(s) - 1; i >= 0 && (apr_iscntrl(s[i]) ||
          apr_isspace(s[i])); i--)
         ;
     s[i + 1] = '\0';
@@ -134,7 +134,7 @@
 
 char *sight_ltrim(char *s)
 {
-    size_t i;
+    int i;
     /* check for empty strings */
     if (!s || !*s)
         return NULL;
@@ -149,11 +149,11 @@
 
 char *sight_trim(char *s)
 {
-    size_t i;
+    int i;
     /* check for empty strings */
     if (!s || !*s)
         return NULL;
-    for (i = strlen(s) - 1; i >= 0 && (apr_iscntrl(s[i]) ||
+    for (i = (int)strlen(s) - 1; i >= 0 && (apr_iscntrl(s[i]) ||
          apr_isspace(s[i])); i--)
         ;
     s[i + 1] = '\0';