From b8db6db8b092849a8578e0d7ba54f34072edd0f4 Mon Sep 17 00:00:00 2001 From: Ilia Mirkin Date: Wed, 9 Jul 2014 21:17:32 -0400 Subject: [PATCH] i965: forward-declare struct brw_context in brw_reg.h Commit 54e91e7420 introduced a function declaration that uses brw_context. While brw_context tends to get included in most files, it is not when compiling intel_asm_annotation.c resulting in the following warning: In file included from brw_shader.h:25:0, from brw_cfg.h:32, from intel_asm_annotation.c:24: brw_reg.h:122:39: warning: 'struct brw_context' declared inside parameter list [enabled by default] brw_reg.h:122:39: warning: its scope is only this definition or declaration, which is probably not what you want [enabled by default] Add a forward-declaration for struct brw_context to avoid the issue. Signed-off-by: Ilia Mirkin Reviewed-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_reg.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_reg.h b/src/mesa/drivers/dri/i965/brw_reg.h index 494ce8c4fa5..28d3d943327 100644 --- a/src/mesa/drivers/dri/i965/brw_reg.h +++ b/src/mesa/drivers/dri/i965/brw_reg.h @@ -52,6 +52,8 @@ extern "C" { #endif +struct brw_context; + /** Number of general purpose registers (VS, WM, etc) */ #define BRW_MAX_GRF 128 -- 2.30.2