client: Simplify some logic in queue_event
authorDerek Foreman <derekf@osg.samsung.com>
Thu, 13 Apr 2017 16:51:47 +0000 (11:51 -0500)
committerDaniel Stone <daniels@collabora.com>
Mon, 4 Dec 2017 22:21:45 +0000 (22:21 +0000)
Both the blocks in this if/else clause do the same thing, so combine
the comparisons into one.

No functional change.

Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-by: Bryce Harrington <bryce@osg.samsung.com>
Reviewed-by: Daniel Stone <daniels@collabora.com>
src/wayland-client.c

index 3d7361e..8fb2dbd 100644 (file)
@@ -1228,10 +1228,7 @@ queue_event(struct wl_display *display, int len)
                return 0;
 
        proxy = wl_map_lookup(&display->objects, id);
-       if (proxy == WL_ZOMBIE_OBJECT) {
-               wl_connection_consume(display->connection, size);
-               return size;
-       } else if (proxy == NULL) {
+       if (!proxy || proxy == WL_ZOMBIE_OBJECT) {
                wl_connection_consume(display->connection, size);
                return size;
        }