On 01/09/2018 04:30 PM, Eric Engestrom wrote:
On Tuesday, 2018-01-09 09:48:16 +0200, Tapani Pälli wrote:
Patch moves existing functionality from shader_cache_read_program_metadata
to a helper function.
Signed-off-by: Tapani Pälli <tapani.pa...@intel.com>
---
src/compiler/Makefile.sources | 1 +
src/compiler/glsl/meson.build | 1 +
src/compiler/glsl/program.cpp | 88 ++++++++++++++++++++++++++++++++++++++
src/compiler/glsl/program.h | 12 ++++--
src/compiler/glsl/shader_cache.cpp | 55 +-----------------------
5 files changed, 100 insertions(+), 57 deletions(-)
create mode 100644 src/compiler/glsl/program.cpp
[snip]
diff --git a/src/compiler/glsl/program.h b/src/compiler/glsl/program.h
index 480379b10b..ae349113e0 100644
--- a/src/compiler/glsl/program.h
+++ b/src/compiler/glsl/program.h
@@ -25,18 +25,22 @@
#ifndef GLSL_PROGRAM_H
#define GLSL_PROGRAM_H
-#ifdef __cplusplus
-extern "C" {
-#endif
-
struct gl_context;
struct gl_shader;
struct gl_shader_program;
+#ifdef __cplusplus
+extern "C" {
+#endif
+
Unrelated hunk, but also I'm not convinced this is right. Either C++
does mangle struct names and this needs to be inside the `extern "C"`,
or it doesn't and moving it outside won't change anything, right?
Yes this is not needed, I can't recall what happened here but will
remove this hunk!
// Tapani
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev