Hi Derek, On 6 December 2017 at 17:22, Derek Foreman <der...@osg.samsung.com> wrote: > This initializes all the fd arguments in closures to -1 and clears > them back to -1 when they've been dispatched or serialized. > > This means that any valid fd in a closure is currently libwayland's > responsibility to close in the case of an error.
As with the previous, I'm totally happy with this and intend to land, but a good follow-on might be to have wl_closure_{invoke,dispatch} take full ownership of the closure, i.e. unconditionally destroy it before return. All its users do this anyway. Cheers, Daniel _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/wayland-devel