On 09/05/2013 11:32 AM, Paul Berry wrote: > On 4 September 2013 11:29, Ian Romanick <[email protected] > <mailto:[email protected]>> wrote: > > From: Ian Romanick <[email protected] > <mailto:[email protected]>> > > There is no GL_TEXTURE_CUBE_MAP_SEAMLESS in any version of OpenGL ES or > in any extension that applies to OpenGL ES. The same error check > already occurs for glTexParameteri. > > Signed-off-by: Ian Romanick <[email protected] > <mailto:[email protected]>> > Cc: Maxence Le Dore <[email protected] > <mailto:[email protected]>> > --- > src/mesa/main/samplerobj.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/main/samplerobj.c b/src/mesa/main/samplerobj.c > index 39cfcd0..c3b612c 100644 > --- a/src/mesa/main/samplerobj.c > +++ b/src/mesa/main/samplerobj.c > @@ -569,7 +569,8 @@ static GLuint > set_sampler_cube_map_seamless(struct gl_context *ctx, > struct gl_sampler_object *samp, > GLboolean param) > { > - if (!ctx->Extensions.AMD_seamless_cubemap_per_texture) > + if (!_mesa_is_desktop_gl(ctx) > + || !ctx->Extensions.AMD_seamless_cubemap_per_texture) > return INVALID_PNAME; > > if (samp->CubeMapSeamless == param) > -- > 1.8.1.4 > > > Should we add a similar check to these functions too? > > - _mesa_GetSamplerParameteriv() > - _mesa_GetSamplerParameterfv() > - _mesa_GetSamplerParameterIiv() > - _mesa_GetSamplerParameterIuiv()
Yes. I forgot about the getters. _______________________________________________ mesa-dev mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-dev
