Refactor destroy functions and save workspaces when there's no outputs

This changes the destroy functions to the following:

* output_begin_destroy
* output_destroy
* workspace_begin_destroy
* workspace_destroy
* container_begin_destroy
* container_destroy
* view_begin_destroy
* view_destroy

The terminology was `destroy` and `free`, and it has been changed to
`begin_destroy` and `destroy` respectively.

When the last output is disconnected, its workspaces will now be stashed
in the root. Upon connection of a new output they will be restored.

There is a new function `workspace_consider_destroy` which decides
whether the given workspace should be destroyed or not (ie. empty and
not visible).

Calling container_begin_destroy will no longer automatically reap the
parents. In some places we want to reap the parents and in some we
don't, so this is left to the caller.

container_reap_empty_recursive and container_reap_empty have been
combined into one function and it will recurse up the tree.

1 files changed, 16 insertions, 1 deletions

diff --git a/sway/desktop/transaction.c b/sway/desktop/transaction.c
index f82e5ef2..c18529fb 100644
--- a/sway/desktop/transaction.c
+++ b/sway/desktop/transaction.c
@@ -54,7 +54,22 @@ static void transaction_destroy(struct sway_transaction *transaction) {
                         con->instruction = NULL;
                 }
                 if (con->destroying && con->ntxnrefs == 0) {
-                        container_free(con);
+                        switch (con->type) {
+                        case C_ROOT:
+                                break;
+                        case C_OUTPUT:
+                                output_destroy(con);
+                                break;
+                        case C_WORKSPACE:
+                                workspace_destroy(con);
+                                break;
+                        case C_CONTAINER:
+                        case C_VIEW:
+                                container_destroy(con);
+                                break;
+                        case C_TYPES:
+                                break;
+                        }
                 }
                 free(instruction);
         }