From 74a7d2562dd93eb47097caf5330fe75f990ff32a Mon Sep 17 00:00:00 2001
From: William McKinnon <contact@willmckinnon.com>
Date: Sun, 25 Feb 2024 17:15:55 -0500
Subject: [PATCH] improved performance

---
 sway/server.c | 26 +++++++++++++++++---------
 1 file changed, 17 insertions(+), 9 deletions(-)

diff --git a/sway/server.c b/sway/server.c
index ef7e4f5b..78be989b 100644
--- a/sway/server.c
+++ b/sway/server.c
@@ -63,22 +63,30 @@ static void handle_drm_lease_request(struct wl_listener *listener, void *data) {
 	}
 }
 
+float get_fastest_output_refresh_s() {
+	float fastest_output_refresh_s = 0.0166667; // fallback to 60 Hz
+	for (int i = 0; i < root->outputs->length; ++i) {
+		struct sway_output *output = root->outputs->items[i];
+		if (output->refresh_nsec > 0) {
+			fastest_output_refresh_s = MIN(fastest_output_refresh_s, output->refresh_sec);
+		}
+	}
+	return fastest_output_refresh_s;
+}
+
+
 static int animation_timer(void *data) {
 	struct sway_server *server = data;
-	float fastest_output_refresh_s = 1.0 / 60.0; // fallback to 60 Hz
+	float fastest_output_refresh_s = get_fastest_output_refresh_s();
 
 	for (int i = 0; i < server->animated_containers->length; i++) {
 		struct sway_container *con = server->animated_containers->items[i];
 		bool is_closing = con->alpha > con->target_alpha;
 
-		for (int i = 0; i < con->outputs->length; ++i) {
-			struct sway_output *output = root->outputs->items[i];
-			fastest_output_refresh_s = MIN(fastest_output_refresh_s, output->refresh_sec);
-			float alpha_step = config->animation_duration ?
-				(con->max_alpha * output->refresh_sec) / config->animation_duration : con->max_alpha;
-			con->alpha = is_closing ? MAX(con->alpha - alpha_step, con->target_alpha) :
-				MIN(con->alpha + alpha_step, con->target_alpha);
-		}
+		float alpha_step = config->animation_duration ?
+			(con->max_alpha * fastest_output_refresh_s) / config->animation_duration : con->max_alpha;
+		con->alpha = is_closing ? MAX(con->alpha - alpha_step, con->target_alpha) :
+			MIN(con->alpha + alpha_step, con->target_alpha);
 
 		if (con->alpha == con->target_alpha) {
 			list_del(server->animated_containers, i);