Change execute_command to return a list of results

This matches i3's behavior of returning a list of results that contain
the result of each command that was executed. Additionally, the
`parse_error` attribute has been added to the IPC JSON reply.

1 files changed, 28 insertions, 39 deletions

diff --git a/sway/commands.c b/sway/commands.c
index bffc18f6..34e66ce9 100644
--- a/sway/commands.c
+++ b/sway/commands.c
@@ -215,12 +215,9 @@ static void set_config_node(struct sway_node *node) {
         }
 }
 
-struct cmd_results *execute_command(char *_exec, struct sway_seat *seat,
+list_t *execute_command(char *_exec, struct sway_seat *seat,
                 struct sway_container *con) {
-        // Even though this function will process multiple commands we will only
-        // return the last error, if any (for now). (Since we have access to an
-        // error string we could e.g. concatenate all errors there.)
-        struct cmd_results *results = NULL;
+        list_t *res_list = create_list();
         char *exec = strdup(_exec);
         char *head = exec;
         char *cmdlist;
@@ -254,8 +251,8 @@ struct cmd_results *execute_command(char *_exec, struct sway_seat *seat,
                         char *error = NULL;
                         struct criteria *criteria = criteria_parse(head, &error);
                         if (!criteria) {
-                                results = cmd_results_new(CMD_INVALID, head,
-                                        "%s", error);
+                                list_add(res_list, cmd_results_new(CMD_INVALID, head,
+                                        "%s", error));
                                 free(error);
                                 goto cleanup;
                         }
@@ -291,10 +288,8 @@ struct cmd_results *execute_command(char *_exec, struct sway_seat *seat,
                         }
                         struct cmd_handler *handler = find_handler(argv[0], NULL, 0);
                         if (!handler) {
-                                if (results) {
-                                        free_cmd_results(results);
-                                }
-                                results = cmd_results_new(CMD_INVALID, cmd, "Unknown/invalid command");
+                                list_add(res_list, cmd_results_new(CMD_INVALID, cmd,
+                                                        "Unknown/invalid command"));
                                 free_argv(argc, argv);
                                 goto cleanup;
                         }
@@ -308,29 +303,21 @@ struct cmd_results *execute_command(char *_exec, struct sway_seat *seat,
                         if (!config->handler_context.using_criteria) {
                                 set_config_node(node);
                                 struct cmd_results *res = handler->handle(argc-1, argv+1);
-                                if (res->status != CMD_SUCCESS) {
+                                list_add(res_list, res);
+                                if (res->status == CMD_INVALID) {
                                         free_argv(argc, argv);
-                                        if (results) {
-                                                free_cmd_results(results);
-                                        }
-                                        results = res;
                                         goto cleanup;
                                 }
-                                free_cmd_results(res);
                         } else {
                                 for (int i = 0; i < views->length; ++i) {
                                         struct sway_view *view = views->items[i];
                                         set_config_node(&view->container->node);
                                         struct cmd_results *res = handler->handle(argc-1, argv+1);
-                                        if (res->status != CMD_SUCCESS) {
+                                        list_add(res_list, res);
+                                        if (res->status == CMD_INVALID) {
                                                 free_argv(argc, argv);
-                                                if (results) {
-                                                        free_cmd_results(results);
-                                                }
-                                                results = res;
                                                 goto cleanup;
                                         }
-                                        free_cmd_results(res);
                                 }
                         }
                         free_argv(argc, argv);
@@ -339,10 +326,7 @@ struct cmd_results *execute_command(char *_exec, struct sway_seat *seat,
 cleanup:
         free(exec);
         list_free(views);
-        if (!results) {
-                results = cmd_results_new(CMD_SUCCESS, NULL, NULL);
-        }
-        return results;
+        return res_list;
 }
 
 // this is like execute_command above, except:
@@ -574,20 +558,25 @@ void free_cmd_results(struct cmd_results *results) {
         free(results);
 }
 
-char *cmd_results_to_json(struct cmd_results *results) {
+char *cmd_results_to_json(list_t *res_list) {
         json_object *result_array = json_object_new_array();
-        json_object *root = json_object_new_object();
-        json_object_object_add(root, "success",
-                        json_object_new_boolean(results->status == CMD_SUCCESS));
-        if (results->input) {
-                json_object_object_add(
-                                root, "input", json_object_new_string(results->input));
-        }
-        if (results->error) {
-                json_object_object_add(
-                                root, "error", json_object_new_string(results->error));
+        for (int i = 0; i < res_list->length; ++i) {
+                struct cmd_results *results = res_list->items[i];
+                json_object *root = json_object_new_object();
+                json_object_object_add(root, "success",
+                                json_object_new_boolean(results->status == CMD_SUCCESS));
+                if (results->error) {
+                        json_object_object_add(root, "parse_error",
+                                        json_object_new_boolean(results->status == CMD_INVALID));
+                        json_object_object_add(
+                                        root, "error", json_object_new_string(results->error));
+                }
+                if (results->input) {
+                        json_object_object_add(
+                                        root, "input", json_object_new_string(results->input));
+                }
+                json_object_array_add(result_array, root);
         }
-        json_object_array_add(result_array, root);
         const char *json = json_object_to_json_string(result_array);
         char *res = strdup(json);
         json_object_put(result_array);