[PATCH 04/19] Use C99 designated initializers in dix Replies
Keith Packard
keithp at keithp.com
Mon Jul 2 10:47:00 PDT 2012
Alan Coopersmith <alan.coopersmith at oracle.com> writes:
Ok, I like the general plan here, but there are a mish-mash of styles
which doesn't really seem neater than the original code.
If we can, indeed, use this form:
> +
> + reply = (xQueryTreeReply) {
> + .type = X_Reply,
> + .sequenceNumber = client->sequence,
> + .root = pWin->drawable.pScreen->root->drawable.id,
> + .parent = (pWin->parent) ? pWin->parent->drawable.id : (Window) None
> + };
then I'd like to see that used uniformly as it will eliminate the need
to create artificial blocks to place new declarations and initializers
in, and will make the patch a lot easier to read -- you'd be replacing a
sequence of assignment statements with a single struct assignment.
--
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 827 bytes
Desc: not available
URL: <http://lists.x.org/archives/xorg-devel/attachments/20120702/915e58af/attachment.pgp>
More information about the xorg-devel
mailing list