[Xcb] xinput.xml: patches for padding at the end of requests, replies and structs
Ran Benita
ran234 at gmail.com
Tue Aug 19 16:54:53 PDT 2014
On Fri, Aug 15, 2014 at 08:26:51PM +0200, Christian Linhart wrote:
> Hi again,
>
> In the next twelve posts I'll send the xinput-proto patches
> for the following category:
> "padding at the end of requests, replies and structs"
These (12) patches all look good to me. I hope the compiler is smart
enough to optimize away all of the always-zero double padding that the
generator inserts here. I bet it does.
Reviewed-by: Ran Benita <ran234 at gmail.com>
> These pads will be useful when we use assertions to compare
> request/reply-length with interpretation of the data using xcb-protocol defs.
>
> BTW, Ran, thank you for your suggestions for
> making the patches easier to review.
> I have applied your suggestions.
>
> I have also increased the diff-context to 7 lines.
> I hope this helps as well.
That was helpful, thanks.
Btw, for the future, you can do away with the "file-positions may become
wrong" by linking to a specific tag or commit, e.g.
http://cgit.freedesktop.org/xorg/proto/inputproto/tree/XIproto.h?id=inputproto-2.3.1#n596
Ran
> Chris
>
> P.S.: I will post the patches for the remaining categories later.
>
>
> _______________________________________________
> Xcb mailing list
> Xcb at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/xcb
More information about the Xcb
mailing list