diff options
author | Lev Walkin <vlm@lionet.info> | 2004-06-05 08:47:18 +0000 |
---|---|---|
committer | Lev Walkin <vlm@lionet.info> | 2004-06-05 08:47:18 +0000 |
commit | 67c2aec9f8ba2da0d0650e6173ac5f437205d999 (patch) | |
tree | 86eb85621ae5749210d77f8b2e531a85cd33c7ec | |
parent | 51b67e099c4c02f5dc328731a727d5bc761674f5 (diff) |
enforsed compilation with -W -Werror
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | asn1c/tests/check-31.c | 2 | ||||
-rw-r--r-- | asn1c/tests/check-35.c | 16 | ||||
-rw-r--r-- | asn1c/tests/check-41.c | 16 | ||||
-rwxr-xr-x | configure | 4 | ||||
-rw-r--r-- | configure.in | 4 |
6 files changed, 28 insertions, 18 deletions
@@ -1,4 +1,8 @@ +0.8.11: 2004-Jun-05 + + * Enforsed stricter conformance with C standards. + 0.8.10: 2004-Jun-02 * Added const qualifier where necessary. diff --git a/asn1c/tests/check-31.c b/asn1c/tests/check-31.c index cfebedcd..05380815 100644 --- a/asn1c/tests/check-31.c +++ b/asn1c/tests/check-31.c @@ -74,6 +74,8 @@ int bytes_compare(const void *bufferp, size_t size, void *key) { const uint8_t *buffer = bufferp; assert(buf_pos + size <= sizeof(buf1_reconstr)); + (void)key; /* Unused argument */ + fprintf(stderr, " writing %d (%d)\n", (int)size, buf_pos + (int)size); for(; size; buf_pos++, size--, buffer++) { diff --git a/asn1c/tests/check-35.c b/asn1c/tests/check-35.c index f29d877c..8fdaced0 100644 --- a/asn1c/tests/check-35.c +++ b/asn1c/tests/check-35.c @@ -133,13 +133,15 @@ check(T_t *tp, uint8_t *buf, int size, size_t consumed) { && strcmp(tp->b.choice.b2.buf, "z") == 0); } -int buf_pos; -int buf_size; +size_t buf_pos; +size_t buf_size; uint8_t *buf; static int buf_fill(const void *buffer, size_t size, void *app_key) { + (void)app_key; + if(buf_pos + size > buf_size) { fprintf(stderr, "%d + %d > %d\n", buf_pos, (int)size, buf_size); return -1; @@ -184,10 +186,10 @@ compare(T_t *tp, uint8_t *cmp_buf, int cmp_buf_size) { } static void -partial_read(uint8_t *buf, int size) { +partial_read(uint8_t *buf, size_t size) { T_t t, *tp; ber_dec_rval_t rval; - int i1, i2; + size_t i1, i2; uint8_t *buf1 = alloca(size); uint8_t *buf2 = alloca(size); uint8_t *buf3 = alloca(size); @@ -203,11 +205,11 @@ partial_read(uint8_t *buf, int size) { for(i1 = 0; i1 < size; i1++) { for(i2 = i1; i2 < size; i2++) { uint8_t *chunk1 = buf; - int size1 = i1; + size_t size1 = i1; uint8_t *chunk2 = buf + size1; - int size2 = i2 - i1; + size_t size2 = i2 - i1; uint8_t *chunk3 = buf + size1 + size2; - int size3 = size - size1 - size2; + size_t size3 = size - size1 - size2; fprintf(stderr, "\n%d:{%d, %d, %d}...\n", size, size1, size2, size3); diff --git a/asn1c/tests/check-41.c b/asn1c/tests/check-41.c index 09a9d62a..d36ad1ec 100644 --- a/asn1c/tests/check-41.c +++ b/asn1c/tests/check-41.c @@ -151,13 +151,15 @@ check(T_t *tp, uint8_t *buf, int size, size_t consumed) { */ } -int buf_pos; -int buf_size; +size_t buf_pos; +size_t buf_size; uint8_t *buf; static int buf_fill(const void *buffer, size_t size, void *app_key) { + (void)app_key; /* Unused argument */ + if(buf_pos + size > buf_size) { fprintf(stderr, "%d + %d > %d\n", buf_pos, (int)size, buf_size); return -1; @@ -202,10 +204,10 @@ compare(T_t *tp, uint8_t *cmp_buf, int cmp_buf_size) { } static void -partial_read(uint8_t *buf, int size) { +partial_read(uint8_t *buf, size_t size) { T_t t, *tp; ber_dec_rval_t rval; - int i1, i2; + size_t i1, i2; uint8_t *buf1 = alloca(size); uint8_t *buf2 = alloca(size); uint8_t *buf3 = alloca(size); @@ -221,11 +223,11 @@ partial_read(uint8_t *buf, int size) { for(i1 = 0; i1 < size; i1++) { for(i2 = i1; i2 < size; i2++) { uint8_t *chunk1 = buf; - int size1 = i1; + size_t size1 = i1; uint8_t *chunk2 = buf + size1; - int size2 = i2 - i1; + size_t size2 = i2 - i1; uint8_t *chunk3 = buf + size1 + size2; - int size3 = size - size1 - size2; + size_t size3 = size - size1 - size2; fprintf(stderr, "\n%d:{%d, %d, %d}...\n", size, size1, size2, size3); @@ -1650,7 +1650,7 @@ fi # Define the identity of the package. PACKAGE=asn1c -VERSION=0.8.10 +VERSION=0.8.11 cat >>confdefs.h <<_ACEOF #define PACKAGE "$PACKAGE" @@ -4756,7 +4756,7 @@ esac # Check whether --enable-autoconf or --disable-autoconf was given. if test "${enable_autoconf+set}" = set; then enableval="$enable_autoconf" - ADD_CFLAGS="-Werror" + ADD_CFLAGS="-Werror -W" fi; diff --git a/configure.in b/configure.in index a4ad1b06..ea31dbbf 100644 --- a/configure.in +++ b/configure.in @@ -2,7 +2,7 @@ dnl Process this file with autoconf to produce a configure script. AC_INIT(libasn1parser/asn1p_y.y) AC_CANONICAL_SYSTEM AC_PREREQ(2.53) -AM_INIT_AUTOMAKE(asn1c, 0.8.10) +AM_INIT_AUTOMAKE(asn1c, 0.8.11) AC_SUBST(PATH) @@ -45,7 +45,7 @@ esac AC_ARG_ENABLE(autoconf, [ --enable-Werror abort compilation after any C compiler warning], - ADD_CFLAGS="-Werror") + ADD_CFLAGS="-Werror -W") AC_SUBST(ADD_CFLAGS) dnl Add these flags if we're using GCC. |