From cdb00739cabd0d55dbdde6be67d110142d0ee35f Mon Sep 17 00:00:00 2001 From: nirenjan Date: Mon, 16 Mar 2026 22:50:11 -0700 Subject: [PATCH] fix(x52ctl): Remove unnecessary goto statements The `goto cleanup` statements are not really needed, and can be safely replaced with `break` statements in x52ctl. --- daemon/x52ctl.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/daemon/x52ctl.c b/daemon/x52ctl.c index a90e107..f200e37 100644 --- a/daemon/x52ctl.c +++ b/daemon/x52ctl.c @@ -168,7 +168,7 @@ int main(int argc, char **argv) if (send_command(sock_fd, sargc, sargv)) { rc = EXIT_FAILURE; - goto cleanup; + break; } fputs("\n> ", stdout); @@ -177,12 +177,9 @@ int main(int argc, char **argv) } else { if (send_command(sock_fd, argc - optind, &argv[optind])) { rc = EXIT_FAILURE; - goto cleanup; } } - -cleanup: close(sock_fd); return rc; }