diff --git a/sway/tree/workspace.c b/sway/tree/workspace.c index 9a8e53d25..49a830e7f 100644 --- a/sway/tree/workspace.c +++ b/sway/tree/workspace.c @@ -202,11 +202,10 @@ static bool workspace_valid_on_output(const char *output_name, } for (int i = 0; i < wsc->outputs->length; i++) { - if (output_match_name_or_id(output, wsc->outputs->items[i])) { - return true; - } - if (output_by_name_or_id(wsc->outputs->items[i])) { - return false; // a higher-priority output is available + struct sway_output *ws_output = + output_by_name_or_id(wsc->outputs->items[i]); + if (ws_output) { + return ws_output == output; } } @@ -323,15 +322,16 @@ char *workspace_next_name(const char *output_name) { } bool found = false; for (int j = 0; j < wsc->outputs->length; ++j) { - if (output_match_name_or_id(output, wsc->outputs->items[j])) { - found = true; - free(target); - target = strdup(wsc->workspace); + struct sway_output *ws_output = + output_by_name_or_id(wsc->outputs->items[j]); + if (ws_output) { + if (ws_output == output) { + found = true; + free(target); + target = strdup(wsc->workspace); + } break; } - if (output_by_name_or_id(wsc->outputs->items[j])) { - break; // a higher-priority output is available - } } if (found) { break;