[Xcb] [PATCH] Add RandR 1.3 requests
Julien Cristau
jcristau at debian.org
Tue Apr 21 07:57:36 PDT 2009
Hi,
One thing I forgot to mention in the patch was that whenever this goes
in and there's a libxcb release, it'd be nice to bump the version-info
for libxcb-randr since this adds new interface.
On Tue, Apr 21, 2009 at 10:37:06 -0400, Peter Harris wrote:
> Julien Cristau wrote:
> > + <request name="GetPanning" opcode="28">
> > + <field type="CRTC" name="crtc" />
> > + <reply>
> > + <field type="CARD8" name="status" /> <!-- enum SetConfig -->
>
> enum is an attribute now. That should probably read
>
> + <field type="CARD8" name="status" enum="SetConfig" />
>
> (same with the other instances)
Ah, right, I didn't remember what the status was on this.
> I haven't merged my "enumify everything" branch, since nobody has asked
> for it and I haven't finished enumifying all the extensions yet. If you
> like, I can merge that branch now so the rest of randr is enumified (and
> <enum name="SetConfig"> is actually defined) before this goes in.
>
> http://cgit.freedesktop.org/~peterh/xcbproto-enum/
>
> What do you think?
>
Sounds fine to me (haven't reviewed the branch, but what could possibly
go wrong? ;) ).
BTW, I noticed that subpixel_order in render was missing an enum, does
your branch fix this? If not I'll try to send a patch.
Cheers,
Julien
More information about the Xcb
mailing list