diff options
author | Ryan Dwyer <[email protected]> | 2018-10-23 20:33:40 +1000 |
---|---|---|
committer | GitHub <[email protected]> | 2018-10-23 20:33:40 +1000 |
commit | e33671fd9a478a917250b2e3b889c9ee0e49c65c (patch) | |
tree | 32f4a9782328486f529c09fe2bb8092c25b9993d /sway/commands/exec_always.c | |
parent | 970501f05e8a1835b880953888ad3022fb2e9ada (diff) | |
parent | 9227cb7d673b012a0c6730800d528f9d729972b6 (diff) |
Merge pull request #2941 from ianyfan/commands
commands: clean up checkargs function +other small changes
Diffstat (limited to 'sway/commands/exec_always.c')
-rw-r--r-- | sway/commands/exec_always.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/sway/commands/exec_always.c b/sway/commands/exec_always.c index de78dd83..8bdeceeb 100644 --- a/sway/commands/exec_always.c +++ b/sway/commands/exec_always.c @@ -16,7 +16,7 @@ struct cmd_results *cmd_exec_always(int argc, char **argv) { struct cmd_results *error = NULL; if (!config->active) return cmd_results_new(CMD_DEFER, NULL, NULL); - if ((error = checkarg(argc, "exec_always", EXPECTED_MORE_THAN, 0))) { + if ((error = checkarg(argc, argv[-1], EXPECTED_AT_LEAST, 1))) { return error; } @@ -24,7 +24,7 @@ struct cmd_results *cmd_exec_always(int argc, char **argv) { if (strcmp(argv[0], "--no-startup-id") == 0) { wlr_log(WLR_INFO, "exec switch '--no-startup-id' not supported, ignored."); --argc; ++argv; - if ((error = checkarg(argc, "exec_always", EXPECTED_MORE_THAN, 0))) { + if ((error = checkarg(argc, argv[-1], EXPECTED_AT_LEAST, 1))) { return error; } } @@ -71,7 +71,7 @@ struct cmd_results *cmd_exec_always(int argc, char **argv) { } else if (pid < 0) { close(fd[0]); close(fd[1]); - return cmd_results_new(CMD_FAILURE, "exec_always", "fork() failed"); + return cmd_results_new(CMD_FAILURE, argv[-1], "fork() failed"); } close(fd[1]); // close write ssize_t s = 0; @@ -85,7 +85,7 @@ struct cmd_results *cmd_exec_always(int argc, char **argv) { wlr_log(WLR_DEBUG, "Child process created with pid %d", child); root_record_workspace_pid(child); } else { - return cmd_results_new(CMD_FAILURE, "exec_always", + return cmd_results_new(CMD_FAILURE, argv[-1], "Second fork() failed"); } |