[Mesa-dev] [PATCH] mesa/main: fix up _mesa_has_rg_textures for gles2

[Erik Faye-Lund]

Thanks, pushed :)

On Tue, 2018-12-04 at 15:03 -0500, Marek Olšák wrote:
> Reviewed-by: Marek Olšák <marek.olsak at amd.com>
> 
> Marek
> 
> On Tue, Dec 4, 2018 at 6:51 AM Erik Faye-Lund <
> erik.faye-lund at collabora.com> wrote:
> > rg-textures are supported in GLES 2.0 if EXT_texture_rg, so let's
> > make
> > sure the enums are accepted.
> > 
> > Fixes: 510b6424607 "mesa/main: do not allow rg-textures enums
> > before gles3"
> > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108936
> > Signed-off-by: Erik Faye-Lund <erik.faye-lund at collabora.com>
> > ---
> > Whoops, seems I missed an extension to enable RG-textures. Sorry
> > for the
> > breakage!
> > 
> >  src/mesa/main/context.h | 3 ++-
> >  1 file changed, 2 insertions(+), 1 deletion(-)
> > 
> > diff --git a/src/mesa/main/context.h b/src/mesa/main/context.h
> > index cdda8cf2012..7de10e9924b 100644
> > --- a/src/mesa/main/context.h
> > +++ b/src/mesa/main/context.h
> > @@ -378,7 +378,8 @@ _mesa_has_packed_float(const struct gl_context
> > *ctx)
> >  static inline bool
> >  _mesa_has_rg_textures(const struct gl_context *ctx)
> >  {
> > -   return _mesa_has_ARB_texture_rg(ctx) || _mesa_is_gles3(ctx);
> > +   return _mesa_has_ARB_texture_rg(ctx) ||
> > _mesa_has_EXT_texture_rg(ctx) ||
> > +          _mesa_is_gles3(ctx);
> >  }
> > 
> >  static inline bool