On 05/30/2013 09:56 AM, Eric Anholt wrote:
Brian Paul <bri...@vmware.com> writes:

For formats such as GL_COMPRESSED_SRGB_S3TC_DXT1_EXT we need to
have both the GL_EXT_texture_sRGB and GL_EXT_texture_compression_s3tc
extensions.  This patch adds the missing check for the later.

Found when checking out https://bugs.freedesktop.org/show_bug.cgi?id=65173

Ah, this patch was a bit confusing without the context of "if
(ctx->Extensions.EXT_texture_sRGB) {" wrapping the switch statement.

Reviewed-by: Eric Anholt <e...@anholt.net>

On a similar note, looks like we're allowing a lot of extension-provided
L/LA/A/I formats on core.

Is that something you (or another else) can take care of?

-Brian

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to