# HG changeset patch # User Eric Blake # Date 1221510663 21600 # Node ID 86be59f0ff1ba7c9521f148e9f206f464874064b # Parent d435a10d69fe9e048dfafe3c4feeb3278599b283 tests: avoid some compiler warnings * tests/test-memchr.c (main): Pass NULL indirectly. * tests/test-closein.c (main): Avoid unused variable. Signed-off-by: Eric Blake diff --git a/ChangeLog b/ChangeLog --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2008-09-15 Eric Blake + + tests: avoid some compiler warnings + * tests/test-memchr.c (main): Pass NULL indirectly. + * tests/test-closein.c (main): Avoid unused variable. + 2008-09-15 Bruno Haible * m4/errno_h.m4 (gl_HEADER_ERRNO_H_BODY): Test for all the macros that @@ -158,7 +164,7 @@ __MSVCRT__. 2008-09-13 Yoann Vandoorselaere - Bruno Haible + Bruno Haible * lib/glob.c: Don't include on native Windows. (WINDOWS32): New macro. @@ -263,7 +269,7 @@ Approved by Eric Blake. 2008-09-04 Ralf Wildenhues - Bruno Haible + Bruno Haible Make gnulib-tool work with native 'sed' on AIX. * gnulib-tool (sed_noop): New variable. @@ -273,7 +279,7 @@ Reported by Albert Chin . 2008-09-04 Albert Chin - Bruno Haible + Bruno Haible * m4/include_next.m4 (gl_INCLUDE_NEXT): Add check whether #include_next also works after #include directives. diff --git a/tests/test-closein.c b/tests/test-closein.c --- a/tests/test-closein.c +++ b/tests/test-closein.c @@ -1,5 +1,5 @@ /* Test of closein module. - Copyright (C) 2007 Free Software Foundation, Inc. + Copyright (C) 2007, 2008 Free Software Foundation, Inc. This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -36,8 +36,7 @@ main (int argc, char **argv) { char buf[7]; - int i = -1; - atexit(close_stdin); + atexit (close_stdin); program_name = argv[0]; /* close_stdin currently relies on ftell, but mingw ftell is @@ -48,6 +47,6 @@ close (0); if (argc > 1) - i = fread (buf, 1, 6, stdin); + fread (buf, 1, 6, stdin); return 0; } diff --git a/tests/test-memchr.c b/tests/test-memchr.c --- a/tests/test-memchr.c +++ b/tests/test-memchr.c @@ -41,6 +41,8 @@ int main () { + void *nil = NULL; /* Use to avoid gcc attribute((nonnull)) warnings. */ + size_t n = 0x100000; char *input = malloc (n); ASSERT (input); @@ -56,7 +58,7 @@ ASSERT (MEMCHR (input, 'a', n) == input); ASSERT (MEMCHR (input, 'a', 0) == NULL); - ASSERT (MEMCHR (NULL, 'a', 0) == NULL); + ASSERT (MEMCHR (nil, 'a', 0) == NULL); ASSERT (MEMCHR (input, 'b', n) == input + 1); ASSERT (MEMCHR (input, 'c', n) == input + 2);