Re: [Mesa-dev] [PATCH 1/3] mesa: Simplify some tests in update_array_format()

2015-03-09 Thread Fredrik Höglund
On Thursday 05 March 2015, Ian Romanick wrote: On 03/05/2015 10:56 AM, Fredrik Höglund wrote: There is no need to check if these extensions are supported here; if the data type is not supported, we will already have returned a GL_INVALID_ENUM error. From where would GL_INVALID_ENUM have

Re: [Mesa-dev] [PATCH 1/3] mesa: Simplify some tests in update_array_format()

2015-03-05 Thread Ian Romanick
On 03/05/2015 10:56 AM, Fredrik Höglund wrote: There is no need to check if these extensions are supported here; if the data type is not supported, we will already have returned a GL_INVALID_ENUM error. From where would GL_INVALID_ENUM have been generated? Is that the (typeBit

Re: [Mesa-dev] [PATCH 1/3] mesa: Simplify some tests in update_array_format()

2015-03-05 Thread Brian Paul
On 03/05/2015 03:56 PM, Ian Romanick wrote: On 03/05/2015 10:56 AM, Fredrik Höglund wrote: There is no need to check if these extensions are supported here; if the data type is not supported, we will already have returned a GL_INVALID_ENUM error. From where would GL_INVALID_ENUM have been

[Mesa-dev] [PATCH 1/3] mesa: Simplify some tests in update_array_format()

2015-03-05 Thread Fredrik Höglund
There is no need to check if these extensions are supported here; if the data type is not supported, we will already have returned a GL_INVALID_ENUM error. --- src/mesa/main/varray.c | 20 +--- 1 file changed, 5 insertions(+), 15 deletions(-) diff --git a/src/mesa/main/varray.c