From 3168e01c952d83be2261ace25b83c46e746dafe0 Mon Sep 17 00:00:00 2001
From: Von Random <von@mechanus.net>
Date: Tue, 25 Feb 2025 14:01:02 +0200
Subject: [PATCH] wezterm: better implemenation of set_by_os to avoid checking
 for os multiple times

---
 dot_config/wezterm/functions.lua | 5 +++--
 dot_config/wezterm/wezterm.lua   | 5 +++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/dot_config/wezterm/functions.lua b/dot_config/wezterm/functions.lua
index 1827dd2..f9a7b0a 100644
--- a/dot_config/wezterm/functions.lua
+++ b/dot_config/wezterm/functions.lua
@@ -5,11 +5,12 @@ local function get_os()
 end
 
 local function set_by_os(values)
-    local my_os = get_os()
-    if values[my_os] then return values[my_os] end
+    if values[MY_OS] then return values[MY_OS] end
     return values.others
 end
 
+MY_OS = get_os()
+
 return {
     set_by_os = set_by_os
 }
diff --git a/dot_config/wezterm/wezterm.lua b/dot_config/wezterm/wezterm.lua
index 19dba86..c6776d4 100644
--- a/dot_config/wezterm/wezterm.lua
+++ b/dot_config/wezterm/wezterm.lua
@@ -26,11 +26,12 @@ cfg.font_size = fn.set_by_os{
 }
 
 -- visuals
+-- maybe enable these back one day
+-- cfg.window_decorations = 'RESIZE'
+-- cfg.hide_tab_bar_if_only_one_tab = false
 cfg.native_macos_fullscreen_mode = false
-cfg.window_decorations = 'RESIZE'
 cfg.xcursor_theme = 'Adwaita'
 cfg.use_fancy_tab_bar = false
-cfg.hide_tab_bar_if_only_one_tab = false
 cfg.show_new_tab_button_in_tab_bar = false
 cfg.tab_max_width = 24
 cfg.window_padding = {