From 02638c1f447c9497345412a67d869e0acfedb3e4 Mon Sep 17 00:00:00 2001
From: Tony Crisci <tony@dubstepdish.com>
Date: Mon, 1 Aug 2016 22:57:33 -0400
Subject: [PATCH] Refactor logic to determine if stacked/tabbed con

Add a new function `swayc_is_tabbed_or_stacked()` to determine whether or not
the given container has layout of stacked or tabbed. This is done commonly
enough that it should have its own method to help readability and code quality.

No logic changes.
---
 include/container.h |  5 +++++
 sway/border.c       |  4 ++--
 sway/commands.c     |  2 +-
 sway/container.c    | 14 +++++++++++---
 sway/focus.c        |  2 +-
 sway/handlers.c     |  2 +-
 sway/layout.c       |  3 +--
 7 files changed, 22 insertions(+), 10 deletions(-)

diff --git a/include/container.h b/include/container.h
index 4dd7f3a0e..4636ddbf8 100644
--- a/include/container.h
+++ b/include/container.h
@@ -141,6 +141,11 @@ enum visibility_mask {
 	VISIBLE = true
 } visible;
 
+/**
+ * Whether this con is tabbed or stacked
+ */
+bool swayc_is_tabbed_or_stacked(swayc_t *container);
+
 /**
  * Allocates a new output container.
  */
diff --git a/sway/border.c b/sway/border.c
index f681e4f14..f74c3f53e 100644
--- a/sway/border.c
+++ b/sway/border.c
@@ -425,7 +425,7 @@ void render_view_borders(wlc_handle view) {
 	// if we are not the only child in the container, always draw borders,
 	// regardless of the border setting on the individual view
 	if (!c || (c->border_type == B_NONE
-			&& !((c->parent->layout == L_TABBED || c->parent->layout == L_STACKED)
+			&& !(swayc_is_tabbed_or_stacked(c->parent)
 				&& c->parent->children->length > 1))) {
 		return;
 	}
@@ -440,7 +440,7 @@ bool should_hide_top_border(swayc_t *con, double y) {
 	// sharing top border with tabbed titlebar
 	swayc_t *par = con->parent;
 	while (par->type != C_WORKSPACE) {
-		if (par->layout == L_TABBED || par->layout == L_STACKED) {
+		if (swayc_is_tabbed_or_stacked(par)) {
 			return con->y == y;
 		}
 		con = par;
diff --git a/sway/commands.c b/sway/commands.c
index f73bd21c9..178a14536 100644
--- a/sway/commands.c
+++ b/sway/commands.c
@@ -1968,7 +1968,7 @@ static struct cmd_results *cmd_layout(int argc, char **argv) {
 			parent->layout = default_layout(output);
 		}
 	} else {
-		if (parent->layout != L_TABBED && parent->layout != L_STACKED) {
+		if (!swayc_is_tabbed_or_stacked(parent)) {
 			parent->prev_layout = parent->layout;
 		}
 
diff --git a/sway/container.c b/sway/container.c
index c922bac3b..f6a4b62a9 100644
--- a/sway/container.c
+++ b/sway/container.c
@@ -99,6 +99,14 @@ static void update_root_geometry() {
 	root_container.height = height;
 }
 
+bool swayc_is_tabbed_or_stacked(swayc_t *container) {
+	if (!ASSERT_NONNULL(container)) {
+		return false;
+	}
+
+	return container->layout == L_STACKED || container->layout == L_TABBED;
+}
+
 // New containers
 
 swayc_t *new_output(wlc_handle handle) {
@@ -682,7 +690,7 @@ swayc_t *container_under_pointer(void) {
 		int len;
 		// if tabbed/stacked go directly to focused container, otherwise search
 		// children
-		if (lookup->layout == L_TABBED || lookup->layout == L_STACKED) {
+		if (swayc_is_tabbed_or_stacked(lookup)) {
 			lookup = lookup->focused;
 			continue;
 		}
@@ -888,7 +896,7 @@ swayc_t *swayc_tabbed_stacked_ancestor(swayc_t *view) {
 	}
 	while (view->type != C_WORKSPACE && view->parent && view->parent->type != C_WORKSPACE) {
 		view = view->parent;
-		if (view->layout == L_TABBED || view->layout == L_STACKED) {
+		if (swayc_is_tabbed_or_stacked(view)) {
 			parent = view;
 		}
 	}
@@ -900,7 +908,7 @@ swayc_t *swayc_tabbed_stacked_parent(swayc_t *con) {
 	if (!ASSERT_NONNULL(con)) {
 		return NULL;
 	}
-	if (con->parent && (con->parent->layout == L_TABBED || con->parent->layout == L_STACKED)) {
+	if (con->parent && swayc_is_tabbed_or_stacked(con->parent)) {
 		return con->parent;
 	}
 	return NULL;
diff --git a/sway/focus.c b/sway/focus.c
index 6583f8023..6f001d4cd 100644
--- a/sway/focus.c
+++ b/sway/focus.c
@@ -151,7 +151,7 @@ bool set_focused_container(swayc_t *c) {
 		}
 		// set focus if view_focus is unlocked
 		wlc_view_focus(c->handle);
-		if (c->parent->layout != L_TABBED && c->parent->layout != L_STACKED) {
+		if (!swayc_is_tabbed_or_stacked(c->parent)) {
 			update_container_border(c);
 		}
 
diff --git a/sway/handlers.c b/sway/handlers.c
index 806923716..12953777b 100644
--- a/sway/handlers.c
+++ b/sway/handlers.c
@@ -396,7 +396,7 @@ static bool handle_view_created(wlc_handle handle) {
 		// layout is tabbed/stacked, add a container around newview
 		swayc_t *parent_container = newview->parent;
 		if (parent_container->type == C_WORKSPACE && parent_container->children->length == 1 &&
-				(parent_container->layout == L_TABBED || parent_container->layout == L_STACKED)) {
+				swayc_is_tabbed_or_stacked(parent_container)) {
 			swayc_t *container = new_container(newview, parent_container->layout);
 			set_focused_container(newview);
 			arrange_windows(container, -1, -1);
diff --git a/sway/layout.c b/sway/layout.c
index 42954ebc8..8f48e4baa 100644
--- a/sway/layout.c
+++ b/sway/layout.c
@@ -793,8 +793,7 @@ static void arrange_windows_r(swayc_t *container, double width, double height) {
 		y = container->y;
 
 		// add gaps to top level tapped/stacked container
-		if (container->parent->type == C_WORKSPACE &&
-			(container->layout == L_TABBED || container->layout == L_STACKED)) {
+		if (container->parent->type == C_WORKSPACE && swayc_is_tabbed_or_stacked(container)) {
 			update_geometry(container);
 			width = container->border_geometry.size.w;
 			height = container->border_geometry.size.h;