The activity flag could already be set, so queue the callback always (if

nicm · nicm · commit 98967c5ec97f · 2015-11-19T14:55:25.000Z
not already queued) rather than only if the flag is being added. Fixes a
problem reported by tim@
diff --git a/alerts.c b/alerts.c
@@ -132,15 +132,15 @@ alerts_queue(struct window *w, int flags)
 	if (!event_initialized(&w->alerts_timer))
 		evtimer_set(&w->alerts_timer, alerts_timer, w);
 
-	if (w->flags & flags)
-		return;
-	w->flags |= flags;
-	log_debug("@%u alerts flags added %#x", w->id, flags);
-
-	if (!alerts_fired && alerts_enabled(w, flags)) {
-		log_debug("alerts check queued (by @%u)", w->id);
-		event_once(-1, EV_TIMEOUT, alerts_callback, NULL, NULL);
-		alerts_fired = 1;
+	if (!alerts_fired) {
+		w->flags |= flags;
+		log_debug("@%u alerts flags added %#x", w->id, flags);
+
+		if (alerts_enabled(w, flags)) {
+			log_debug("alerts check queued (by @%u)", w->id);
+			event_once(-1, EV_TIMEOUT, alerts_callback, NULL, NULL);
+			alerts_fired = 1;
+		}
 	}
 }
 
