[PATCH 1/3] protocol: Add an event to specify the name of an output

Neil Roberts neil at linux.intel.com
Tue May 6 06:43:22 PDT 2014


Pekka Paalanen <ppaalanen at gmail.com> writes:

> were you aware of […] perhaps? :-)

Ah, no, I hadn't noticed those patches. Oops!

> Your series is almost identical to Quanxian's, except:
> - wording differences, of course
> - patch 2 checks if output->name is set before sending (can it ever be
>   NULL?)
> - patch 2 missed setting the actual protocol object version
> - you add support to weston-info in patch 3
> - Quanxian adds empty handlers to window.c and desktop-shell.c which
>   isn't really necessary

Ok, it sounds like it would make the most sense to use Quanxian's
patches without the empty handlers and then land my weston-info patch on
top.

Regards,
- Neil


More information about the wayland-devel mailing list