Thanks, I'll modify the commit message a bit and push this in.

On 03/26/2018 05:13 PM, Lin Johnson wrote:
Ext_color_buffer_half_float is using type GL_HALF_FLOAT
and data_type GL_FLOAT. This fix Android CTS test
android.view.cts.PixelCopyTest
v2: remove commtens of Ext_color_buffer_half_float.
As Ext_color_buffer__float can use type GL_HALF_FLOAT

Reviewed-by:   Palli, Tapani <[email protected]>
Signed-off-by: Lin Johnson <[email protected]>
---
  src/mesa/main/readpix.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/src/mesa/main/readpix.c b/src/mesa/main/readpix.c
index 6ce340ddf9bb..4407f13289e2 100644
--- a/src/mesa/main/readpix.c
+++ b/src/mesa/main/readpix.c
@@ -920,6 +920,8 @@ read_pixels_es3_error_check(GLenum format, GLenum type,
     case GL_RGBA:
        if (type == GL_FLOAT && data_type == GL_FLOAT)
           return GL_NO_ERROR; /* EXT_color_buffer_float */
+      if (type == GL_HALF_FLOAT && data_type == GL_FLOAT)
+         return GL_NO_ERROR;
        if (type == GL_UNSIGNED_BYTE && data_type == GL_UNSIGNED_NORMALIZED)
           return GL_NO_ERROR;
        if (internalFormat == GL_RGB10_A2 &&

_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to