mesa: Generate correct error code in glDrawBuffers()
authorAnuj Phogat <anuj.phogat@gmail.com>
Tue, 28 Jan 2014 01:23:32 +0000 (17:23 -0800)
committerAnuj Phogat <anuj.phogat@gmail.com>
Tue, 28 Jan 2014 23:30:55 +0000 (15:30 -0800)
commit33034755583edfb7c5b773b8e38a9dfa8d317821
tree0a79e5ef3a9711c362b849983d0860bed7af83ac
parentfaee376869734e0d41209e8985084a4380190652
mesa: Generate correct error code in glDrawBuffers()

OpenGL 3.3 spec expects GL_INVALID_OPERATION:
 "For both the default framebuffer and framebuffer objects, the
  constants FRONT, BACK, LEFT, RIGHT, and FRONT AND BACK are not
  valid in the bufs array passed to DrawBuffers, and will result
  in the error INVALID OPERATION."

But OpenGL 4.0 spec changed the error code to GL_INVALID_ENUM:
 "For both the default framebuffer and framebuffer objects, the
  constants FRONT, BACK, LEFT, RIGHT, and FRONT_AND_BACK are not
  valid in the bufs array passed to DrawBuffers, and will result
  in the error INVALID_ENUM."

This patch changes the behaviour to match OpenGL 4.0 spec
Fixes Khronos OpenGL CTS draw_buffers_api.test.

V2: Update the comment in code.

Cc: mesa-stable@lists.freedesktop.org
Signed-off-by: Anuj Phogat <anuj.phogat@gmail.com>
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
src/mesa/main/buffers.c