From 1b9176c42779fcfd3eea7b962b7a9df44e337744 Mon Sep 17 00:00:00 2001 From: Consolatis <35009135+Consolatis@users.noreply.github.com> Date: Wed, 6 Mar 2024 04:46:40 +0100 Subject: [PATCH] src/action.c: remove assumption about MoveToOutput direction being valid --- src/action.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/action.c b/src/action.c index 8e152c79..1f8289e0 100644 --- a/src/action.c +++ b/src/action.c @@ -926,7 +926,6 @@ actions_run(struct view *activator, struct server *server, if (output_name) { target = output_from_name(view->server, output_name); } else { - /* Config parsing makes sure that direction is a valid direction */ enum view_edge edge = action_get_int(action, "direction", 0); bool wrap = action_get_bool(action, "wrap", false); target = view_get_adjacent_output(view, edge, wrap); @@ -934,7 +933,8 @@ actions_run(struct view *activator, struct server *server, if (!target) { /* * Most likely because we're already on the - * output furthest in the requested direction. + * output furthest in the requested direction + * or the output or direction was invalid. */ wlr_log(WLR_DEBUG, "Invalid output"); break; -- 2.52.0