This seems foolishly cosmetic on the surface - and will reorder log
messages in certain failure cases. "request could not be marshalled"
will now appear after logging the request that failed to marshal
instead of before.
The real point of this is that a follow up patch will make
wl_closure_send() set fds to -1 as it buffers them for send, so
they can be more easily cleaned up.
Doing that while leaving this order unchanged would result in
printing -1 for fds instead of their value.
Signed-off-by: Derek Foreman <der...@osg.samsung.com>
---
src/wayland-server.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/wayland-server.c b/src/wayland-server.c
index 82a3b01..61e0315 100644
--- a/src/wayland-server.c
+++ b/src/wayland-server.c
@@ -220,11 +220,11 @@ handle_array(struct wl_resource *resource, uint32_t
opcode,
return;
}
+ log_closure(resource, closure, true);
+
if (send_func(closure, resource->client->connection))
resource->client->error = 1;
- log_closure(resource, closure, true);
-
wl_closure_destroy(closure);
}
--
2.15.0
_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel
