Skip to content

Commit

Permalink
Fix off-by-one error when checking workspace_layout arguments
Browse files Browse the repository at this point in the history
  • Loading branch information
oranenj committed Mar 18, 2017
1 parent ec50b92 commit a608859
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions sway/commands/workspace_layout.c
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,13 @@ struct cmd_results *cmd_workspace_layout(int argc, char **argv) {
if ((error = checkarg(argc, "workspace_layout auto", EXPECTED_EQUAL_TO, 2))) {
return error;
}
if (strcasecmp(argv[0], "left") == 0) {
if (strcasecmp(argv[1], "left") == 0) {
config->default_layout = L_AUTO_LEFT;
} else if (strcasecmp(argv[0], "right") == 0) {
} else if (strcasecmp(argv[1], "right") == 0) {
config->default_layout = L_AUTO_RIGHT;
} else if (strcasecmp(argv[0], "top") == 0) {
} else if (strcasecmp(argv[1], "top") == 0) {
config->default_layout = L_AUTO_TOP;
} else if (strcasecmp(argv[0], "bottom") == 0) {
} else if (strcasecmp(argv[1], "bottom") == 0) {
config->default_layout = L_AUTO_BOTTOM;
} else {
return cmd_results_new(CMD_INVALID, "workspace_layout auto", "Expected 'workspace_layout auto <left|right|top|bottom>'");
Expand Down

0 comments on commit a608859

Please sign in to comment.