From 4c755711758a29ca502b53577dfa74d714655648 Mon Sep 17 00:00:00 2001 From: Hendrik Wolff Date: Tue, 4 Nov 2025 14:09:50 +0100 Subject: [PATCH] sway/cmd/create_output: Add create_result struct --- sway/commands/create_output.c | 27 ++++++++++++++++----------- 1 file changed, 16 insertions(+), 11 deletions(-) diff --git a/sway/commands/create_output.c b/sway/commands/create_output.c index 79283fd11..18c1c377b 100644 --- a/sway/commands/create_output.c +++ b/sway/commands/create_output.c @@ -9,23 +9,28 @@ #include "sway/server.h" #include "log.h" +struct create_result { + bool done; + struct wlr_output *new_output; +}; + static void create_output(struct wlr_backend *backend, void *data) { - bool *done = data; - if (*done) { + struct create_result *result = data; + if (result->done) { return; } if (wlr_backend_is_wl(backend)) { - wlr_wl_output_create(backend); - *done = true; + result->new_output = wlr_wl_output_create(backend); + result->done = true; } else if (wlr_backend_is_headless(backend)) { - wlr_headless_add_output(backend, 1920, 1080); - *done = true; + result->new_output = wlr_headless_add_output(backend, 1920, 1080); + result->done = true; } #if WLR_HAS_X11_BACKEND else if (wlr_backend_is_x11(backend)) { - wlr_x11_output_create(backend); - *done = true; + result->new_output = wlr_x11_output_create(backend); + result->done = true; } #endif } @@ -37,10 +42,10 @@ struct cmd_results *cmd_create_output(int argc, char **argv) { sway_assert(wlr_backend_is_multi(server.backend), "Expected a multi backend"); - bool done = false; - wlr_multi_for_each_backend(server.backend, create_output, &done); + struct create_result result = { false, NULL }; + wlr_multi_for_each_backend(server.backend, create_output, &result); - if (!done) { + if (!result.done) { return cmd_results_new(CMD_INVALID, "Can only create outputs for Wayland, X11 or headless backends"); }