Thanks for the diff - I've committed a more comprehensive diff that makes it clean with WARNINGS=Yes.
On Mon, 12 May 2014, [email protected] wrote: > From: Brent Cook <[email protected]> > > --- > base64/base64test.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/base64/base64test.c b/base64/base64test.c > index a7d167e..fedbdcd 100644 > --- a/base64/base64test.c > +++ b/base64/base64test.c > @@ -19,6 +19,7 @@ > > #include <err.h> > #include <stdio.h> > +#include <string.h> > > #define BUF_SIZE 128 > > @@ -205,7 +206,7 @@ base64_encoding_test(int test_no, struct base64_test > *bt, int test_nl) > > len = BIO_write(bio_mem, bt->in, bt->in_len); > if (len != bt->in_len) { > - fprintf(stderr, "FAIL: test %i - only wrote %i out of %i " > + fprintf(stderr, "FAIL: test %i - only wrote %i out of %zu " > "characters\n", test_no, len, bt->in_len); > failure = 1; > goto done; > @@ -296,7 +297,7 @@ base64_decoding_test(int test_no, struct base64_test > *bt, int test_nl) len = BIO_read(bio_mem, buf, BUF_SIZE); > if (len != bt->valid_len && (bt->in_len != 0 || len != -1)) { > fprintf(stderr, "FAIL: test %i - decoding resulted in %i " > - "characters instead of %i\n", test_no, len, bt->valid_len); > + "characters instead of %zu\n", test_no, len, bt->valid_len); > fprintf(stderr, " input: "); > for (i = 0; i < inlen; i++) > fprintf(stderr, "%c", input[i]); -- "Action without study is fatal. Study without action is futile." -- Mary Ritter Beard
