From 631d6b59fdb193cc29d845d1baf30b729a4cb75a Mon Sep 17 00:00:00 2001
From: Nicholas Marriott <nicm@openbsd.org>
Date: Sun, 8 Apr 2012 06:47:26 +0000
Subject: [PATCH] Do not fire name timer when automatic-rename is off, from Tim
 Ruehsen a while ago.

---
 cmd-set-option.c | 13 +++++++++++++
 names.c          |  4 +---
 window.c         |  4 ++--
 3 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/cmd-set-option.c b/cmd-set-option.c
index 153d2d88..ff138873 100644
--- a/cmd-set-option.c
+++ b/cmd-set-option.c
@@ -87,6 +87,7 @@ cmd_set_option_exec(struct cmd *self, struct cmd_ctx *ctx)
 	struct winlink				*wl;
 	struct client				*c;
 	struct options				*oo;
+	struct window				*w;
 	const char				*optstr, *valstr;
 	u_int					 i;
 
@@ -147,6 +148,18 @@ cmd_set_option_exec(struct cmd *self, struct cmd_ctx *ctx)
 			return (-1);
 	}
 
+	/* Start or stop timers when automatic-rename changed. */
+	if (strcmp (oe->name, "automatic-rename") == 0) {
+		for (i = 0; i < ARRAY_LENGTH(&windows); i++) {
+			if ((w = ARRAY_ITEM(&windows, i)) == NULL)
+				continue;
+			if (options_get_number(&w->options, "automatic-rename"))
+				queue_window_name(w);
+			else if (event_initialized(&w->name_timer))
+				evtimer_del(&w->name_timer);
+		}
+	}
+
 	/* Update sizes and redraw. May not need it but meh. */
 	recalculate_sizes();
 	for (i = 0; i < ARRAY_LENGTH(&clients); i++) {
diff --git a/names.c b/names.c
index 5821d94a..469f9123 100644
--- a/names.c
+++ b/names.c
@@ -49,9 +49,7 @@ window_name_callback(unused int fd, unused short events, void *data)
 	struct window	*w = data;
 	char		*name, *wname;
 
-	queue_window_name(w);	/* XXX even if the option is off? */
-	if (!options_get_number(&w->options, "automatic-rename"))
-		return;
+	queue_window_name(w); /* stopped when option turned off */
 
 	if (w->active->screen != &w->active->base)
 		name = NULL;
diff --git a/window.c b/window.c
index 4a013405..9071972d 100644
--- a/window.c
+++ b/window.c
@@ -297,9 +297,9 @@ window_create1(u_int sx, u_int sy)
 	w->sx = sx;
 	w->sy = sy;
 
-	queue_window_name(w);
-
 	options_init(&w->options, &global_w_options);
+	if (options_get_number(&w->options, "automatic-rename"))
+		queue_window_name(w);
 
 	for (i = 0; i < ARRAY_LENGTH(&windows); i++) {
 		if (ARRAY_ITEM(&windows, i) == NULL) {