[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Undefined __GNUC_GNU_INLINE__
From: |
Eric Blake |
Subject: |
Re: Undefined __GNUC_GNU_INLINE__ |
Date: |
Fri, 18 Oct 2013 09:40:47 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.0 |
On 10/18/2013 09:30 AM, Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> When testing with old gcc, I found out that gnulib uses
> __GNUC_GNU_INLINE__ without checking whether it's defined first. I
> propose following fix:
>
> === modified file 'm4/extern-inline.m4'
> --- m4/extern-inline.m4 2013-04-11 19:12:46 +0000
> +++ m4/extern-inline.m4 2013-10-18 14:55:19 +0000
> @@ -32,7 +32,7 @@
> # define _GL_INLINE inline
> # define _GL_EXTERN_INLINE extern inline
> #elif 2 < __GNUC__ + (7 <= __GNUC_MINOR__) && !defined __APPLE__
> -# if __GNUC_GNU_INLINE__
> +# if defined (__GNUC_GNU_INLINE__) && __GNUC_GNU_INLINE__
This has no semantic change, unless you are compiling with -Wundef; and
we generally state that use of -Wundef for gnulib .c files is
unsupported. Or is your complaint that this snippet of code gets
included into header files that get included in your files, and not just
gnulib.c files, and where we should try to honor -Wundef?
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature