[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls |
Date: |
Mon, 15 Apr 2013 08:18:21 +0100 |
On 15 April 2013 03:31, Alexey Kardashevskiy <address@hidden> wrote:
> The assert.h header file from Fedora18 does not have #ifdef-#endif
> brackets around __assertXXXX function so it cannot compile with
> the -Wredundant-decls switch on.
>
> Some Linux distributions (such as Debian Wheezy) still do have those
> brackets arounb __assertXXXX functions (#ifndef _ASSERT_H_DECLS) but
> the version of assert.h on http://sourceware.org/git/?p=glibc.git does not
> so we should not be using -Wredundant-decls.
>
> The patch removes it.
This commit message seems to be missing any mention of
which versions of pixman this change breaks and why
it's OK now to break compiling against them...
> Signed-off-by: Alexey Kardashevskiy <address@hidden>
> ---
> include/ui/qemu-pixman.h | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/include/ui/qemu-pixman.h b/include/ui/qemu-pixman.h
> index b032f52..6f473f9 100644
> --- a/include/ui/qemu-pixman.h
> +++ b/include/ui/qemu-pixman.h
> @@ -7,13 +7,7 @@
> #define QEMU_PIXMAN_H
>
> /* pixman-0.16.0 headers have a redundant declaration */
...and if it's a correct change it should be removing the
comment as well, since it would no longer apply.
> -#ifdef CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE
> -#pragma GCC diagnostic ignored "-Wredundant-decls"
> -#endif
> #include <pixman.h>
> -#ifdef CONFIG_PRAGMA_DIAGNOSTIC_AVAILABLE
> -#pragma GCC diagnostic error "-Wredundant-decls"
> -#endif
>
> #include "qemu/typedefs.h"
>
> --
> 1.7.10.4
thanks
-- PMM
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, (continued)
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Markus Armbruster, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Paolo Bonzini, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Markus Armbruster, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Alexey Kardashevskiy, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Paolo Bonzini, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Markus Armbruster, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Markus Armbruster, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Alexey Kardashevskiy, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Kevin Wolf, 2013/04/16
- Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls, Markus Armbruster, 2013/04/16
Re: [Qemu-devel] [PATCH] pixman: remove -Wredundand-decls,
Peter Maydell <=