<p dir="ltr"><br>
On Jun 10, 2015 9:57 AM, "Neil Roberts" <<a href="mailto:neil@linux.intel.com">neil@linux.intel.com</a>> wrote:<br>
><br>
> Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> writes:<br>
><br>
> >     _mesa_meta_fb_tex_blit_begin(ctx, &blit);<br>
> > +   ctx->Extensions.ARB_texture_stencil8 = true;<br>
><br>
> Maybe you could put assert(ctx->Extensions.ARB_texture_stencil8==false)<br>
> just before setting it to true so that we'll definitely remember to<br>
> remove it if we eventually enable the extension. Otherwise as it stands<br>
> if we forget about this it would probably not break any tests but the<br>
> extension would mysteriously disable itself and no-one would notice<br>
> because they would probably check the extensions just once upfront.</p>
<p dir="ltr">I'll second that of it's not already pushed.</p>
<p dir="ltr">> - Neil<br>
> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</p>