Re: [PATCH wayland 1/3] server: Create the socket FD after taking the lock

Yeah, this looks good to me. It does change the order of a couple of system calls, but it looks ok to me. Reviewed-by: Jason Ekstrand On Wed, May 7, 2014 at 9:25 AM, Jasper St. Pierre wrote: > --- > src/wayland-server.c | 15 +++ > 1 file changed, 7 insertions(+), 8 deletions(-)

[PATCH wayland 1/3] server: Create the socket FD after taking the lock

--- src/wayland-server.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/src/wayland-server.c b/src/wayland-server.c index e850d48..d0fd280 100644 --- a/src/wayland-server.c +++ b/src/wayland-server.c @@ -1061,12 +1061,6 @@ wl_display_add_socket(struct wl_displa

[PATCH wayland 1/3] server: Create the socket FD after taking the lock

This removes a small cleanup path. --- src/wayland-server.c | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/src/wayland-server.c b/src/wayland-server.c index e850d48..d299000 100644 --- a/src/wayland-server.c +++ b/src/wayland-server.c @@ -1061,12 +1061,6 @@ wl_d