sway

commit 5ecedc7199fcd9788ae2d96f0ed1f25a2cbe7364
parent 46e5aeac5016f336aa1184b2752146c04ece21d9
Author: Drew DeVault <sir@cmpwn.com>
Date:   Sun,  8 Oct 2017 10:57:41 -0400

Remove destroyed views from scratchpad

Fixes #1363

Diffstat:
M
sway/container.c
 | 
4
++--
M
sway/handlers.c
 | 
37
++++++++++++++++++++++++-------------

2 files changed, 26 insertions(+), 15 deletions(-)
diff --git a/sway/container.c b/sway/container.c
@@ -516,11 +516,11 @@ swayc_t *destroy_view(swayc_t *view) {
 		return NULL;
 	}
 	sway_log(L_DEBUG, "Destroying view '%p'", view);
-	swayc_t *parent = view->parent;
 	free_swayc(view);
 
 	// Destroy empty containers
-	if (parent->type == C_CONTAINER) {
+	swayc_t *parent = view->parent;
+	if (parent && parent->type == C_CONTAINER) {
 		return destroy_container(parent);
 	}
 	return parent;
diff --git a/sway/handlers.c b/sway/handlers.c
@@ -553,22 +553,24 @@ static void handle_view_destroyed(wlc_handle handle) {
 		bool fullscreen = swayc_is_fullscreen(view);
 		remove_view_from_scratchpad(view);
 		swayc_t *parent = destroy_view(view);
-		if (fullscreen) {
-			parent->fullscreen = NULL;
-		}
+		if (parent) {
+			if (fullscreen) {
+				parent->fullscreen = NULL;
+			}
 
-		ipc_event_window(parent, "close");
+			ipc_event_window(parent, "close");
 
-		// Destroy empty workspaces
-		if (parent->type == C_WORKSPACE &&
-			parent->children->length == 0 &&
-			parent->floating->length == 0 &&
-			swayc_active_workspace() != parent &&
-			!parent->visible) {
-			parent = destroy_workspace(parent);
-		}
+			// Destroy empty workspaces
+			if (parent->type == C_WORKSPACE &&
+				parent->children->length == 0 &&
+				parent->floating->length == 0 &&
+				swayc_active_workspace() != parent &&
+				!parent->visible) {
+				parent = destroy_workspace(parent);
+			}
 
-		arrange_windows(parent, -1, -1);
+			arrange_windows(parent, -1, -1);
+		}
 	} else {
 		// Is it unmanaged?
 		int i;
@@ -584,6 +586,15 @@ static void handle_view_destroyed(wlc_handle handle) {
 				}
 			}
 		}
+		// Is it in the scratchpad?
+		for (i = 0; i < scratchpad->length; ++i) {
+			swayc_t *item = scratchpad->items[i];
+			if (item->handle == handle) {
+				list_del(scratchpad, i);
+				destroy_view(item);
+				break;
+			}
+		}
 	}
 	set_focused_container(get_focused_view(&root_container));
 }