[Xcb] [PATCH libxcb v2 3/4] c_client: Add support for lists of FDs

Uli Schlachter psychon at znc.in
Sat Jul 22 11:12:45 UTC 2017

On 20.07.2017 19:15, Daniel Stone wrote:
> Hi Eric,
> On 17 June 2017 at 02:11, Eric Anholt <eric at anholt.net> wrote:
>> Daniel Stone <daniels at collabora.com> writes:
>>> Matching xcbgen changes, add support having a ListType which contains
>>> file descriptors. Use this to send a variable number of FDs to the
>>> server, including when the list size is not fixed.
>> I've verified that patch 1 and 2 don't change the generated code, that
>> this patch only cosmetically changes the code, and that the generated
>> code for patch 4 looks reasonable.
>> I find the python to be pretty hairy, but it seems fine, and it's really
>> the generated code that matters, so patch 1-3 are:
>> Reviewed-by: Eric Anholt <eric at anholt.net>
> Thanks for the review! FWIW, I find the Python quite hairy as well,
> but after a couple of attempted reworks, couldn't find anything which
> worked better.
> Uli, would you be OK pushing these first 3 support patches, or would
> you prefer I did, or ... ?

I'd think that it is less work overall if you push them since you
already have them in Git. Also, I usually try to stay away from the
Python code around XCB. :-)

So, feel free to push them.

If you have to type the letters "A-E-S" into your source code, you're
doing it wrong.

More information about the Xcb mailing list