[Xcb] [PUSHED] [PATCH 2/2] glx: Add IsEnabled request

Christian Linhart chris at DemoRecorder.com
Sun Sep 13 01:59:41 PDT 2015


Hi Robert and Peter,

This looks good.
The server-code  of IsEnabled is almost identical to IsList.
Therefore, the xml should be almost identical, too.
Which is the case in your patch.

Therefore:
Reviewed-by: Christian Linhart <chris at demorecorder.com>

I have pushed this patch now because it has waited enough time for additional reviews.

Cheers,

Chris

On 08/31/15 16:29, Peter Harris wrote:
> On 2015-08-30 08:30, Christian Linhart wrote:
>> Hi Robert,
>>
>> Can you please post a link to the spec of this request? (or to sourcecode in server or xlib which currently implements this request)
> Server source code:
> http://cgit.freedesktop.org/xorg/xserver/tree/glx/indirect_dispatch.c#n2215
>
> Mesa's XML (which is a different format from XCB XML):
> http://cgit.freedesktop.org/mesa/mesa/tree/src/mapi/glapi/gen/gl_API.xml#n2843
>
> Both of these agree with the proposed patch. Therefore,
>
> Reviewed-by: Peter Harris <pharris at opentext.com>
>
> Peter Harris



More information about the Xcb mailing list