Remove now-unused "input" argument of cmd_results_new

Patch tested by compiling with `__attribute__ ((format (printf, 2, 3)))`
applied to `cmd_results_new`.

String usage constants have been converted from pointers to arrays when
encountered. General handler format strings were sometimes modified to
include the old input string, especially for unknown command errors.

1 files changed, 6 insertions, 5 deletions

diff --git a/sway/commands/exec_always.c b/sway/commands/exec_always.c
index 9ec28d81..1ed6c7e6 100644
--- a/sway/commands/exec_always.c
+++ b/sway/commands/exec_always.c
@@ -15,7 +15,9 @@
 
 struct cmd_results *cmd_exec_always(int argc, char **argv) {
         struct cmd_results *error = NULL;
-        if (!config->active || config->validating) return cmd_results_new(CMD_DEFER, NULL, NULL);
+        if (!config->active || config->validating) {
+                return cmd_results_new(CMD_DEFER, NULL);
+        }
         if ((error = checkarg(argc, argv[-1], EXPECTED_AT_LEAST, 1))) {
                 return error;
         }
@@ -71,7 +73,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, argv[-1], "fork() failed");
+                return cmd_results_new(CMD_FAILURE, "fork() failed");
         }
         close(fd[1]); // close write
         ssize_t s = 0;
@@ -85,9 +87,8 @@ 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, argv[-1],
-                        "Second fork() failed");
+                return cmd_results_new(CMD_FAILURE, "Second fork() failed");
         }
 
-        return cmd_results_new(CMD_SUCCESS, NULL, NULL);
+        return cmd_results_new(CMD_SUCCESS, NULL);
 }