From 232b220d0859bd28d4436ba9cbd90efa482cc658 Mon Sep 17 00:00:00 2001
From: Wez Furlong <wez@wezfurlong.org>
Date: Wed, 1 Dec 2021 08:45:12 -0700
Subject: [PATCH] Avoid looping if window:set_config_overrides doesn't change
 the config

Assuming that the window config reloaded hook doesn't actually change
anything, this will avoid a cycle where we keep triggering the hook
over and over.
---
 wezterm-gui/src/termwindow/mod.rs | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/wezterm-gui/src/termwindow/mod.rs b/wezterm-gui/src/termwindow/mod.rs
index 723a8d9f1..e2ee9d095 100644
--- a/wezterm-gui/src/termwindow/mod.rs
+++ b/wezterm-gui/src/termwindow/mod.rs
@@ -810,8 +810,10 @@ impl TermWindow {
                     .context("send GetConfigOverrides response")?;
             }
             TermWindowNotif::SetConfigOverrides(value) => {
-                self.config_overrides = value;
-                self.config_was_reloaded();
+                if value != self.config_overrides {
+                    self.config_overrides = value;
+                    self.config_was_reloaded();
+                }
             }
             TermWindowNotif::CancelOverlayForPane(pane_id) => {
                 self.cancel_overlay_for_pane(pane_id);