The presence of EXT_texture_integer implies EXT_gpu_shader4, so I added "ctx->Version >= 30 || ctx->Extensions.EXT_texture_integer" everywhere to allow drivers to disable EXT_texture_integer but not lose integer textures from GL 3.0.
Marek On Mon, Jun 16, 2014 at 12:49 PM, Neil Roberts <n...@linux.intel.com> wrote: > Ilia Mirkin <imir...@alum.mit.edu> writes: > >>> + if (ctx->Version >= 30 || ctx->Extensions.EXT_texture_integer) { >> >> Just ctx->Extensions.EXT_texture_integer should be enough here, no? > > I'm reluctant to change this because every other place in the code that > checks for integer textures does it in the same way. Perhaps if we > wanted to change it then we should do it in a separate patch and change > it everywhere. I think it might theoretically make sense to expose GL 3 > without GL_EXT_texture_integer in some cases because the extension > implies supporting luminance integer textures whereas GL 3 does not. > > This bit of code implies that Mesa treats the two types of support for > integer textures distinctly: > > http://is.gd/HM3SIw > > Regards, > - Neil > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev