wezterm: better implemenation of set_by_os to avoid checking for os multiple times

This reverts commit c2fbb28170.

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
 }

dot_config/wezterm/wezterm.lua

@@ -8,18 +8,6 @@ local tab_fg = '#ebdbb2'
 local tab_bg = '#504945'
 local tab_bg_active = '#282828'
 
--- os specific stuff
-local myos = fn.set_by_os{
-    Darwin = {
-        font_size = 14,
-        window_decorations = "TITLE | RESIZE"
-    },
-    others = {
-        font_size = 10,
-        window_decorations = "RESIZE"
-    }
-}
-
 -- misc
 local cfg = wt.config_builder()
 cfg.leader = kb.leader
@@ -32,13 +20,18 @@ cfg.cursor_blink_rate = 0
 cfg.bold_brightens_ansi_colors = false
 cfg.font = wt.font('Cascadia Mono PL')
 cfg.harfbuzz_features = {"ss01", "ss19", "ss20"}
-cfg.font_size = myos.font_size
+cfg.font_size = fn.set_by_os{
+    Darwin = 14,
+    others = 10
+}
+
 -- 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 = myos.window_decorations
 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 = {