bug-gnulib
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

allocsa compiler warning


From: Eric Blake
Subject: allocsa compiler warning
Date: Mon, 28 May 2007 08:51:16 -0600
User-agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.10) Gecko/20070221 Thunderbird/1.5.0.10 Mnenhy/0.7.5.666

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

I'm checking this in, so that test-allocsa can compile on cygwin with -Werror:

../../tests/test-allocsa.c: In function `do_allocation':
../../tests/test-allocsa.c:33: warning: left-hand operand of comma
expression has no effect
make[4]: *** [test-allocsa.o] Error 1

2007-05-28  Eric Blake  <address@hidden>

        * lib/allocsa.h (safe_alloca): Avoid compiler warning.

- --
Don't work too hard, make some time for fun as well!

Eric Blake             address@hidden
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (Cygwin)
Comment: Public key at home.comcast.net/~ericblake/eblake.gpg
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFGWuxh84KuGfSFAYARAsNGAJ9aU9Ffn0M3OgByVzNLBvwoEQqNaACeIsoF
/8GN4YlMDpCN1G2AtTyAi9s=
=eihB
-----END PGP SIGNATURE-----
Index: lib/allocsa.h
===================================================================
RCS file: /sources/gnulib/gnulib/lib/allocsa.h,v
retrieving revision 1.9
diff -u -p -r1.9 allocsa.h
--- lib/allocsa.h       6 Apr 2007 14:36:56 -0000       1.9
+++ lib/allocsa.h       28 May 2007 13:41:01 -0000
@@ -46,7 +46,7 @@ extern "C" {
    This must be a macro, not an inline function.  */
 # define safe_alloca(N) ((N) < 4032 ? alloca (N) : NULL)
 #else
-# define safe_alloca(N) ((N), NULL)
+# define safe_alloca(N) ((void) (N), NULL)
 #endif
 
 /* allocsa(N) is a safe variant of alloca(N).  It allocates N bytes of

reply via email to

[Prev in Thread] Current Thread [Next in Thread]