<p dir="ltr">On Mar 26, 2013 3:28 AM, "Dave Airlie" <<a href="mailto:airlied@gmail.com">airlied@gmail.com</a>> wrote:<br>
><br>
> > +++ b/src/wayland-private.h<br>
> > @@ -74,15 +74,9 @@ int wl_connection_write(struct wl_connection *connection, const void *data, size<br>
> >  int wl_connection_queue(struct wl_connection *connection,<br>
> >                         const void *data, size_t count);<br>
> ><br>
> > -union wl_argument {<br>
> > -       int32_t i;<br>
> > -       uint32_t u;<br>
> > -       wl_fixed_t f;<br>
> > -       const char *s;<br>
> > -       struct wl_object *o;<br>
> > -       uint32_t n;<br>
> > -       struct wl_array *a;<br>
> > -       int32_t h;<br>
> > +struct wl_dispatcher {<br>
><br>
> Care to document why n is different from u and h from i?<br>
><br>
> at first glance I can't see what they are for.</p>
<p dir="ltr">David,<br>
Thanks for pointing this out as it should probably be documented.  Those are the standard abbreviations for the 8 Wayland data types. Specifically, "n" stands for new_id, "h" stands for file descriptor. While not strictly needed, the redundancy serves to make things more explicit in the code. When I actually send out the patches, I'll put comments in the union to that effect.</p>

<p dir="ltr">Thanks for reviewing,<br>
--Jason Ekstrand</p>