Commit 5bf38bfb authored by Erik Faye-Lund 's avatar Erik Faye-Lund

mesa/main: correct validation for GL_RGB565

Technically speaking, this validation was incorrect, because GL_RGB565
is only supported in OpenGL ES 1.x if OES_framebuffer_object is
supported. This couldn't lead to any real incorrect behavior, because
all drivers support OES_framebuffer_object. But let's keep the code
self-documenting, by correcting the check as per the spec.
Signed-off-by: Erik Faye-Lund 's avatarErik Faye-Lund <>
Reviewed-by: 's avatarEmil Velikov <>
parent 4b218984
......@@ -2312,7 +2312,9 @@ _mesa_base_tex_format(const struct gl_context *ctx, GLint internalFormat)
if (_mesa_has_ARB_ES2_compatibility(ctx) || _mesa_is_gles(ctx)) {
if (_mesa_has_ARB_ES2_compatibility(ctx) ||
_mesa_has_OES_framebuffer_object(ctx) ||
ctx->API == API_OPENGLES2) {
switch (internalFormat) {
case GL_RGB565:
return GL_RGB;
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment