<p dir="ltr">Looks good!<br>
Reviewed-by: Lars Hamre <<a href="mailto:chemecse@gmail.com">chemecse@gmail.com</a>></p>
<div class="gmail_quote">On Mar 29, 2016 01:30, "Kenneth Graunke" <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The 'f' suffix isn't required, and in fact it isn't technically allowed<br>
in GLSL 1.10 either.<br>
<br>
Cc: Lars Hamre <<a href="mailto:chemecse@gmail.com">chemecse@gmail.com</a>><br>
---<br>
 tests/spec/arb_texture_cube_map_array/cubemap-lod.c | 4 ++--<br>
 1 file changed, 2 insertions(+), 2 deletions(-)<br>
<br>
Hi Lars,<br>
<br>
I noticed this Piglit test regressed when applying your Mesa patch.<br>
It turns out the test was broken :)<br>
<br>
 --Ken<br>
<br>
diff --git a/tests/spec/arb_texture_cube_map_array/cubemap-lod.c b/tests/spec/arb_texture_cube_map_array/cubemap-lod.c<br>
index ceb347b..2816f9e 100644<br>
--- a/tests/spec/arb_texture_cube_map_array/cubemap-lod.c<br>
+++ b/tests/spec/arb_texture_cube_map_array/cubemap-lod.c<br>
@@ -80,7 +80,7 @@ static const char *frag_shader_biased =<br>
  "uniform samplerCubeArray tex; \n"<br>
  "void main()\n"<br>
  "{\n"<br>
- " gl_FragColor = texture(tex, gl_TexCoord[0], 3.0f);\n"<br>
+ " gl_FragColor = texture(tex, gl_TexCoord[0], 3.0);\n"<br>
  "}\n";<br>
<br>
 static const char *frag_shader_explicit =<br>
@@ -88,7 +88,7 @@ static const char *frag_shader_explicit =<br>
  "uniform samplerCubeArray tex; \n"<br>
  "void main()\n"<br>
  "{\n"<br>
- " gl_FragColor = textureLod(tex, gl_TexCoord[0], 3.0f);\n"<br>
+ " gl_FragColor = textureLod(tex, gl_TexCoord[0], 3.0);\n"<br>
  "}\n";<br>
<br>
 static GLuint frag_shader_cube_array_biased;<br>
--<br>
2.7.4<br>
<br>
</blockquote></div>