[PATCH] scanner: Add GCC pragmas to disable -Wredundant-decls

Kristian Høgsberg hoegsberg at gmail.com
Tue Feb 18 14:28:46 PST 2014


On Sat, Feb 15, 2014 at 01:29:16PM -0500, Jasper St. Pierre wrote:
> The code very intentionally emits a lot of redundant declarations
> to simplify the scanner code. Somebody building with -Wredundant-decls
> would have compile errors, so emit special pragmas to turn those
> warnings off.
> 
> These pragmas should be ignored outside of gcc/clang.

Looks good, applied.

Kristian

> ---
>  src/scanner.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/src/scanner.c b/src/scanner.c
> index 243ddfd..e8bfc7c 100644
> --- a/src/scanner.c
> +++ b/src/scanner.c
> @@ -1165,11 +1165,13 @@ emit_code(struct protocol *protocol)
>  	       "#include <stdint.h>\n"
>  	       "#include \"wayland-util.h\"\n\n");
>  
> +	printf("#pragma GCC diagnostic push\n"
> +	       "#pragma GCC diagnostic ignored \"-Wredundant-decls\"\n");
>  	wl_list_for_each(i, &protocol->interface_list, link) {
>  		emit_types_forward_declarations(protocol, &i->request_list);
>  		emit_types_forward_declarations(protocol, &i->event_list);
>  	}
> -	printf("\n");
> +	printf("#pragma GCC diagnostic pop\n\n");
>  
>  	printf("static const struct wl_interface *types[] = {\n");
>  	emit_null_run(protocol);
> -- 
> 1.8.5.3
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list