Reviewed-by: Ilia Mirkin <imir...@alum.mit.edu>
On Thu, May 7, 2015 at 1:07 AM, Tapani Pälli <tapani.pa...@intel.com> wrote: > This was really the original purpose, for enabling the path for > ES3.1 tests without the extension being set. Set also fallthrough > comment for Coverity (caught by Matt). > > v2: .. and test the right way, not wrong one (Ilia Mirkin) > > Signed-off-by: Tapani Pälli <tapani.pa...@intel.com> > --- > src/mesa/main/shader_query.cpp | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/main/shader_query.cpp b/src/mesa/main/shader_query.cpp > index 6e46553..3445f89 100644 > --- a/src/mesa/main/shader_query.cpp > +++ b/src/mesa/main/shader_query.cpp > @@ -28,6 +28,7 @@ > * \author Ian Romanick <ian.d.roman...@intel.com> > */ > > +#include "main/context.h" > #include "main/core.h" > #include "glsl_symbol_table.h" > #include "ir.h" > @@ -986,8 +987,9 @@ _mesa_program_resource_prop(struct gl_shader_program > *shProg, > case GL_ACTIVE_VARIABLES: > return get_buffer_property(shProg, res, prop, val, caller); > case GL_REFERENCED_BY_COMPUTE_SHADER: > - if (!ctx->Extensions.ARB_compute_shader) > + if (!_mesa_has_compute_shaders(ctx)) > goto invalid_enum; > + /* fallthrough */ > case GL_REFERENCED_BY_VERTEX_SHADER: > case GL_REFERENCED_BY_GEOMETRY_SHADER: > case GL_REFERENCED_BY_FRAGMENT_SHADER: > -- > 2.1.0 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev