Jason Ekstrand <ja...@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 _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel