[Mesa-dev] [Bug 98134] dEQP-GLES31.functional.debug.negative_coverage.get_error.buffer.draw_buffers wants a different GL error code

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=98134

Tapani Pälli <lemody at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Tapani Pälli <lemody at gmail.com> ---
commit a1652a059edc5a5f0f4b0836ba310a22e094bd85
Author: Tapani Pälli <tapani.palli at intel.com>
Date:   Fri Oct 7 10:08:21 2016 +0300

    mesa: fix error handling in DrawBuffers

    Patch rearranges error checking so that enum checking provided via
    destmask happens before other checks. It needs to be done in this
    order because other error checks do not work properly if there were
    invalid enums passed.

    Patch also refines one existing check and it's documentation to match
    GLES 3.0 spec (also in later specs). This was somewhat mysteriously
    referring to desktop GL but had a check for gles3.

    Fixes following dEQP tests:

      
dEQP-GLES31.functional.debug.negative_coverage.get_error.buffer.draw_buffers

    no CI regressions observed.

    Signed-off-by: Tapani Pälli <tapani.palli at intel.com>
    Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98134
    Cc: "12.0 13.0" <mesa-stable at lists.freedesktop.org>
    Reviewed-by: Emil Velikov <emil.velikov at collabora.com>

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161025/ac601b7f/attachment.html>