[Mesa-dev] [PATCH 8/9] mesa: Add support for glGetIntegeri_v from GL_ARB_uniform_buffer_object.
Brian Paul
brianp at vmware.com
Wed Jun 20 06:43:49 PDT 2012
On 06/19/2012 01:26 PM, Eric Anholt wrote:
> On Tue, 19 Jun 2012 08:10:25 -0600, Brian Paul<brianp at vmware.com> wrote:
>> On 06/18/2012 07:35 PM, Eric Anholt wrote:
>>> Fixes piglit ARB_uniform_buffer_object/getintegeri_v.
>>> ---
>>> src/mesa/main/get.c | 24 ++++++++++++++++++++++++
>>> 1 file changed, 24 insertions(+)
>>>
>>> diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
>>> index 933bfe7..4798c02 100644
>>> --- a/src/mesa/main/get.c
>>> +++ b/src/mesa/main/get.c
>>> @@ -2566,6 +2566,30 @@ find_value_indexed(const char *func, GLenum pname, int index, union value *v)
>>> goto invalid_enum;
>>> v->value_int = ctx->TransformFeedback.CurrentObject->BufferNames[index];
>>> return TYPE_INT;
>>> +
>>> + case GL_UNIFORM_BUFFER_BINDING:
>>> + if (index>= ctx->Const.MaxUniformBufferBindings)
>>> + goto invalid_value;
>>> + if (!ctx->Extensions.ARB_uniform_buffer_object)
>>> + goto invalid_enum;
>>
>> I think it's a bit more natural to do the extension check before the
>> index check.
>
> Since all the other enums are handled in this order, too, I'd rather
> that be an independent change. It does seem like a silly ordering,
> though.
Kenneth spotted the same thing. Reordering in a follow-on commit
sounds fine.
-Brian
More information about the mesa-dev
mailing list