[Mesa-dev] [PATCH 06/11] glsl: Remove unused include from glsl_symbol_table.h

thomashelland90 at gmail.com thomashelland90 at gmail.com
Mon Jun 9 15:57:37 PDT 2014


From: Thomas Helland <thomashelland90 at gmail.com>

Only function-defs use glsl_type so forward declare instead.
Compile-tested on my Ivy-bridge system.

IWYU also suggests removing #include <new>, and this compiles fine.
I'm not familiar enough with memory management in C/C++ that I feel
comfortable removing this. Insights would be appreciated.

Signed-off-by: Thomas Helland <thomashelland90 at gmail.com>
---
 src/glsl/glsl_symbol_table.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/glsl/glsl_symbol_table.h b/src/glsl/glsl_symbol_table.h
index f323fc3..39b84e4 100644
--- a/src/glsl/glsl_symbol_table.h
+++ b/src/glsl/glsl_symbol_table.h
@@ -32,9 +32,9 @@ extern "C" {
 #include "program/symbol_table.h"
 }
 #include "ir.h"
-#include "glsl_types.h"
 
 class symbol_table_entry;
+class glsl_type;
 
 /**
  * Facade class for _mesa_symbol_table
-- 
2.0.0



More information about the mesa-dev mailing list