From f4f8d3b5ede9da429025e51650db4fbad48171b5 Mon Sep 17 00:00:00 2001
From: nicm <nicm>
Date: Wed, 25 Aug 2021 10:15:15 +0000
Subject: Ignore client creating session when working out size if it is a
 control client.

---
 resize.c | 7 +++++++
 1 file changed, 7 insertions(+)

(limited to 'resize.c')

diff --git a/resize.c b/resize.c
index 38cda23d..91dea48e 100644
--- a/resize.c
+++ b/resize.c
@@ -243,6 +243,13 @@ default_window_size(struct client *c, struct session *s, struct window *w,
 		}
 	}
 
+        /*
+         * Ignore the given client if it is a control client - the creating
+         * client should only affect the size if it is not a control client.
+         */
+        if (c != NULL && (c->flags & CLIENT_CONTROL))
+                c = NULL;
+
 	/*
 	 * Look for a client to base the size on. If none exists (or the type
 	 * is manual), use the default-size option.
-- 
cgit 


From 24636be42b4b0463afe5c72e1d982f28729a0579 Mon Sep 17 00:00:00 2001
From: nicm <nicm>
Date: Wed, 25 Aug 2021 10:18:01 +0000
Subject: Improve some logging.

---
 resize.c | 29 ++++++++++++++++-------------
 1 file changed, 16 insertions(+), 13 deletions(-)

(limited to 'resize.c')

diff --git a/resize.c b/resize.c
index 91dea48e..f321e7d2 100644
--- a/resize.c
+++ b/resize.c
@@ -51,7 +51,7 @@ resize_window(struct window *w, u_int sx, u_int sy, int xpixel, int ypixel)
 	if (sy < w->layout_root->sy)
 		sy = w->layout_root->sy;
 	window_resize(w, sx, sy, xpixel, ypixel);
-	log_debug("%s: @%u resized to %u,%u; layout %u,%u", __func__, w->id,
+	log_debug("%s: @%u resized to %ux%u; layout %ux%u", __func__, w->id,
 	    sx, sy, w->layout_root->sx, w->layout_root->sy);
 
 	/* Restore the window zoom state. */
@@ -186,6 +186,10 @@ clients_calculate_size(int type, int current, struct client *c,
 		log_debug("%s: after %s (%ux%u), size is %ux%u", __func__,
 		    loop->name, cx, cy, *sx, *sy);
 	}
+	if (*sx != UINT_MAX && *sy != UINT_MAX)
+		log_debug("%s: calculated size %ux%u", __func__, *sx, *sy);
+	else
+		log_debug("%s: no calculated size", __func__);
 
 	/* Return whether a suitable size was found. */
 	if (type == WINDOW_SIZE_LARGEST) {
@@ -231,16 +235,14 @@ default_window_size(struct client *c, struct session *s, struct window *w,
 	 * Latest clients can use the given client if suitable. If there is no
 	 * client and no window, use the default size as for manual type.
 	 */
-	if (type == WINDOW_SIZE_LATEST) {
-		if (c != NULL && !ignore_client_size(c)) {
-			*sx = c->tty.sx;
-			*sy = c->tty.sy - status_line_size(c);
-			*xpixel = c->tty.xpixel;
-			*ypixel = c->tty.ypixel;
-			log_debug("%s: using %ux%u from %s", __func__, *sx, *sy,
-			    c->name);
-			goto done;
-		}
+	if (type == WINDOW_SIZE_LATEST && c != NULL && !ignore_client_size(c)) {
+		*sx = c->tty.sx;
+		*sy = c->tty.sy - status_line_size(c);
+		*xpixel = c->tty.xpixel;
+		*ypixel = c->tty.ypixel;
+		log_debug("%s: using %ux%u from %s", __func__, *sx, *sy,
+		    c->name);
+		goto done;
 	}
 
         /*
@@ -304,7 +306,7 @@ recalculate_size(struct window *w, int now)
 	 */
 	if (w->active == NULL)
 		return;
-	log_debug("%s: @%u is %u,%u", __func__, w->id, w->sx, w->sy);
+	log_debug("%s: @%u is %ux%u", __func__, w->id, w->sx, w->sy);
 
 	/*
 	 * Type is manual, smallest, largest, latest. Current is the
@@ -335,6 +337,7 @@ recalculate_size(struct window *w, int now)
 	 * size.
 	 */
 	if (!changed) {
+		log_debug("%s: @%u no size change", __func__, w->id);
 		tty_update_window_offset(w);
 		return;
 	}
@@ -344,7 +347,7 @@ recalculate_size(struct window *w, int now)
 	 * the size immediately. Otherwise set the flag and it will be done
 	 * later.
 	 */
-	log_debug("%s: @%u new size %u,%u", __func__, w->id, sx, sy);
+	log_debug("%s: @%u new size %ux%u", __func__, w->id, sx, sy);
 	if (now || type == WINDOW_SIZE_MANUAL)
 		resize_window(w, sx, sy, xpixel, ypixel);
 	else {
-- 
cgit