[PATCH weston 1/4] Handle requests using outputs that have been unplugged

Neil Roberts neil at linux.intel.com
Tue May 20 02:56:09 PDT 2014


Jason Ekstrand <jason at jlekstrand.net> writes:

> The destructor passed in here should be 0, why is it ~0? Also, it
> might be a good idea to throw it in a #define or enum for clarity.

The ~0 is meant to signify ‘there is no destructor’, ie, it's a fake
opcode that will never match a request. The reason is that I put this
patch first before the separate patch to handle the wl_output.release
request which changes it to 0. Yes, it would probably be good to have a
define for this but as it was only in one temporary patch I thought it
wasn't worth adding. Maybe just reordering the patches would make it
cleaner.

Regards,
- Neil


More information about the wayland-devel mailing list