Mesa (master): i965: forward-declare struct brw_context in brw_reg.h

Ilia Mirkin imirkin at kemper.freedesktop.org
Thu Jul 10 02:36:06 UTC 2014


Module: Mesa
Branch: master
Commit: b8db6db8b092849a8578e0d7ba54f34072edd0f4
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=b8db6db8b092849a8578e0d7ba54f34072edd0f4

Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Wed Jul  9 21:17:32 2014 -0400

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 <imirkin at alum.mit.edu>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>

---

 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 494ce8c..28d3d94 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
 




More information about the mesa-commit mailing list