- (dtucker) [openbsd-compat/setenv.c] Make offsets size_t to prevent

    compiler warnings on some platforms.  Based on a discussion with otto@

diff --git a/ChangeLog b/ChangeLog
index 54e0e4d9295e77ad1d5d3d4e325a878abbe0d589..ff14caf44668cc5fded16198c89df921cfb0fc9a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -171,6 +171,8 @@
   - (dtucker) [umac.c] STORE_UINT32_REVERSED and endian_convert are never used
     on big endian machines, so ifdef them for little-endian only to prevent
     unused function warnings on big-endians.
+  - (dtucker) [openbsd-compat/setenv.c] Make offsets size_t to prevent
+    compiler warnings on some platforms.  Based on a discussion with otto@
 
 20080611
  - (djm) [channels.c configure.ac]

diff --git a/openbsd-compat/setenv.c b/openbsd-compat/setenv.c
index b52a99c2ccd2356bbde868a6bdf5b6708924a89c..e2a8b6dd3ca972e76eb324caac9e8e2146ac8e0a 100644 (file)
--- a/openbsd-compat/setenv.c
+++ b/openbsd-compat/setenv.c
@@ -47,7 +47,7 @@ extern char **environ;
  *     Explicitly removes '=' in argument name.
  */
 static char *
-__findenv(const char *name, int *offset)
+__findenv(const char *name, size_t *offset)
 {
        extern char **environ;
        int len, i;
@@ -82,7 +82,7 @@ setenv(const char *name, const char *value, int rewrite)
 {
        static char **lastenv;                  /* last value of environ */
        char *C;
-       int l_value, offset;
+       size_t l_value, offset;
 
        if (*value == '=')                      /* no `=' in value */
                ++value;
@@ -133,7 +133,7 @@ void
 unsetenv(const char *name)
 {
        char **P;
-       int offset;
+       size_t offset;
 
        while (__findenv(name, &offset))        /* if set multiple times */
                for (P = &environ[offset];; ++P)