[PATCH wayland] connection: Print the content of arrays in closures

Daniel Stone daniel at fooishbar.org
Fri Dec 1 17:08:15 UTC 2017


Hi,

On 10 July 2017 at 19:28, Emmanuel Gil Peyrot
<emmanuel.peyrot at collabora.com> wrote:
> The current behaviour when WAYLAND_DEBUG is set is to print “array”,
> which is quite unhelpful.
>
> This patch prints a list of the bytes present in the array.  It doesn’t
> try to interpret it as uint32_t or anything, leaving that to the reader
> because this information isn’t present in the protocol description.

To be honest, I'm not really wild about this one. All the array users
I know of (key/button, xdg surface states) are uint32_t. I'd prefer to
add a pretty-printing hint to the protocol - this could specify how to
interpret arrays, and also scratch my long-standing itch of printing
uints with %x rather than %u when it makes sense to ...

Cheers,
Daniel


More information about the wayland-devel mailing list