mesa: Update DispatchComputeIndirect errors for indirect parameter
authorJordan Justen <jordan.l.justen@intel.com>
Tue, 13 Oct 2015 22:04:54 +0000 (15:04 -0700)
committerJordan Justen <jordan.l.justen@intel.com>
Mon, 2 Nov 2015 23:25:37 +0000 (15:25 -0800)
commit44c399f20af49607c799326ad4dd0f74c5214d4c
treea03dbb2eadb2ca067625f132dccc99ae5c6544c2
parent0b19f651958c3888588190c8c8a9e701173a2aa2
mesa: Update DispatchComputeIndirect errors for indirect parameter

There is some discrepancy between the return values for some error
cases for the DispatchComputeIndirect call in the ARB_compute_shader
specification. Regarding the indirect parameter, in one place the
extension spec lists that the error returned for invalid values should
be INVALID_OPERATION, while later it specifies INVALID_VALUE.

The OpenGL 4.3 and OpenGLES 3.1 specifications appear to be consistent
in requiring the INVALID_VALUE error return in this case.

Here we update the code to match the main specifications, and update
the citations use the main specification rather than the extension
specification.

v2:
 * Updates based on review from Iago

Signed-off-by: Jordan Justen <jordan.l.justen@intel.com>
Cc: Iago Toral Quiroga <itoral@igalia.com>
Cc: Marta Lofstedt <marta.lofstedt@intel.com>
Reviewed-by: Marta Lofstedt <marta.lofstedt@intel.com>
src/mesa/main/api_validate.c