If you're going to abolish the clamping entirely (which is admittedly a matter of taste), I'd rather remove the glsl_feature_level variable and assign to consts->GLSLVersion directly.

Cheers,
Nicolai

On 22.03.2016 22:21, Edward O'Callaghan wrote:
Signed-off-by: Edward O'Callaghan <eocallag...@alterapraxis.com>
---
  src/mesa/state_tracker/st_extensions.c | 4 +---
  1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/mesa/state_tracker/st_extensions.c 
b/src/mesa/state_tracker/st_extensions.c
index b03f531..6645189 100644
--- a/src/mesa/state_tracker/st_extensions.c
+++ b/src/mesa/state_tracker/st_extensions.c
@@ -846,10 +846,8 @@ void st_init_extensions(struct pipe_screen *screen,

     /* Figure out GLSL support. */
     glsl_feature_level = screen->get_param(screen, 
PIPE_CAP_GLSL_FEATURE_LEVEL);
-
+   /* Set GLSLVersion to PIPE_CAP_GLSL_FEATURE_LEVEL */
     consts->GLSLVersion = glsl_feature_level;
-   if (glsl_feature_level >= 410)
-      consts->GLSLVersion = 410;

     _mesa_override_glsl_version(consts);


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to