server: Remove superfluous goto label 39/295139/3 accepted/tizen/unified/20230706.152204
authorSeunghun Lee <shiin.lee@samsung.com>
Mon, 3 Jul 2023 08:01:27 +0000 (17:01 +0900)
committerSeunghun Lee <shiin.lee@samsung.com>
Tue, 4 Jul 2023 01:52:29 +0000 (01:52 +0000)
The goto label was introduced by 1c4afff, and is no longer necessary
after 812e138.

Change-Id: If3735215a8f16c4530cc1270562d6999dd0ce5f7

src/wayland-server.c

index 4b32960..c0ba0a9 100644 (file)
@@ -388,12 +388,12 @@ wl_client_connection_data(int fd, uint32_t mask, void *data)
                wl_log("Mask has hangup flag set, client_proc(%s) PID(%d) mask[%x]\n",
                           client->proc_name, client->pid, mask);
                wl_client_destroy(client);
-               goto end;
+               return 1;
        }
 
        if (mask & WL_EVENT_ERROR) {
                destroy_client_with_error(client, "socket error");
-               goto end;
+               return 1;
        }
 
        if (mask & WL_EVENT_WRITABLE) {
@@ -407,7 +407,7 @@ wl_client_connection_data(int fd, uint32_t mask, void *data)
                if (len < 0 && errno != EAGAIN) {
                        destroy_client_with_error(
                            client, "failed to flush client connection");
-                       goto end;
+                       return 1;
                } else if (len >= 0) {
                        wl_event_source_fd_update(client->source,
                                                  WL_EVENT_READABLE);
@@ -425,7 +425,7 @@ wl_client_connection_data(int fd, uint32_t mask, void *data)
                if (len == 0 || (len < 0 && errno != EAGAIN)) {
                        destroy_client_with_error(
                            client, "failed to read client connection");
-                       goto end;
+                       return 1;
                }
        }
 
@@ -528,7 +528,6 @@ wl_client_connection_data(int fd, uint32_t mask, void *data)
                                          "error in client communication");
        }
 
-end:
        return 1;
 }