On 7 November 2013 13:04, Keith Packard <[email protected]> wrote:
> Daniel Martin <[email protected]> writes:
>
>> The <fd> tag is not necessary. It should be an ordinary field - as it
>> gets handled.
>
> I prefer using a separate tag as '<field>' tags now universally mean
> bytes encoded on the wire, and '<fd>' elements are strictly *not* on the
> wire in the usual sense.
>
> It's not a different data type, it's taking an entirely different route
> over the network.
Damn, so I was at the correct line:
...
new_fd = Field(self, module.get_type_name('INT32'), fd_name, True,
False, False, None, True)
...
but didn't read it correctly, as the first False is for 'wire'. m(
(Lot's of places in the code would benefit from keyword arguments.)
Still I think that an attribute would've been better. Anyways,
if you could add the modification for the xml schema it's:
Reviewed-by: Daniel Martin <[email protected]>
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
