[Mesa-dev] [PATCH] glx: Add GLXBadProfileARB to the error string list

[Ian Romanick]

On 10/17/2012 11:52 AM, Brian Paul wrote:
> On 10/17/2012 12:38 PM, Adam Jackson wrote:
>> Note: This is a candidate for the stable branches.
>>
>> Signed-off-by: Adam Jackson<ajax at redhat.com>
>> ---
>>   src/glx/glxext.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/src/glx/glxext.c b/src/glx/glxext.c
>> index 4d59a06..e55c1c1 100644
>> --- a/src/glx/glxext.c
>> +++ b/src/glx/glxext.c
>> @@ -81,6 +81,7 @@ static /* const */ char *error_list[] = {
>>      "GLXBadPbuffer",
>>      "GLXBadCurrentDrawable",
>>      "GLXBadWindow",
>> +   "GLXBadProfileARB",
>>   };
>>
>>   #ifdef GLX_USE_APPLEGL
>
> Reviewed-by: Brian Paul <brianp at vmware.com>
>
> But could we also add an assertion somewhere to catch missing strings in
> the future?  Like assert(ARRAY_SIZE(error_list) == __GLX_NUMBER_ERRORS)?

Either that or some sort of unit test that happens in 'make check'.