koma/i3
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #4698 from orestisfl/user_output_names_find_next

Browse Source 
user_output_names_find_next: Always initialize target_output
This commit is contained in:
Ingo Bürk 2021-11-27 23:38:13 +01:00 committed by GitHub
commit 298c3aede9
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 19 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
release-notes/bugfixes/4-failed-workspace-output Normal file
View File

@ -0,0 +1 @@
fix wrong failed reply on move workspace to output

16
src/commands.c
View File

@ -1052,6 +1052,14 @@ static Output *user_output_names_find_next(user_output_names_head *names, Output
Output *target_output = NULL;
user_output_name *uo;
TAILQ_FOREACH (uo, names, user_output_names) {
if (!target_output) {
/* The first available output from the list is used in 2 cases:
* 1. When we must wrap around the user list. For example, if user
* specifies outputs A B C and C is `current_output`.
* 2. When the current output is not in the user list. For example,
* user specifies A B C and D is `current_output`. */
target_output = get_output_from_string(current_output, uo->name);
}
if (strcasecmp(output_primary_name(current_output), uo->name) == 0) {
/* The current output is in the user list */
while (true) {
@ -1071,14 +1079,6 @@ static Output *user_output_names_find_next(user_output_names_head *names, Output
}
break;
}
if (!target_output) {
/* The first available output from the list is used in 2 cases:
* 1. When we must wrap around the user list. For example, if user
* specifies outputs A B C and C is `current_output`.
* 2. When the current output is not in the user list. For example,
* user specifies A B C and D is `current_output`. */
target_output = get_output_from_string(current_output, uo->name);
}
}
return target_output;
}

10
testcases/t/543-move-workspace-to-multiple-outputs.t
View File

@ -39,6 +39,16 @@ sub is_ws {
is(get_output_for_workspace("$ws_num"), "fake-$out_num", "Workspace $ws_num -> $out_num: $msg");
}
###############################################################################
# Test moving workspace to same output
# See issue #4691
###############################################################################
is_ws(1, 0, 'sanity check');
my $reply = cmd '[con_mark=aa] move workspace to output fake-0';
is_ws(1, 0, 'workspace did not move');
ok($reply->[0]->{success}, 'reply success');
###############################################################################
# Test using "next" special keyword
###############################################################################