Undo eww-state hotreloading change

it caused variables not to be linked to the state after the reload

crates/eww/src/app.rs

@@ -1,4 +1,11 @@
-use crate::{EwwPaths, config, daemon_response::DaemonResponseSender, display_backend, error_handling_ctx, eww_state::{self, EwwState}, script_var_handler::*};
+use crate::{
+    config,
+    daemon_response::DaemonResponseSender,
+    display_backend, error_handling_ctx,
+    eww_state::{self},
+    script_var_handler::*,
+    EwwPaths,
+};
 use anyhow::*;
 use debug_stub_derive::*;
 use eww_shared_util::VarName;
@@ -6,7 +13,7 @@ use gdk::WindowExt;
 use gtk::{ContainerExt, CssProviderExt, GtkWindowExt, StyleContextExt, WidgetExt};
 use itertools::Itertools;
 use simplexpr::dynval::DynVal;
-use std::collections::HashMap;
+use std::collections::{HashMap, HashSet};
 use tokio::sync::mpsc::UnboundedSender;
 use yuck::{
     config::window_geometry::{AnchorPoint, WindowGeometry},
@@ -65,6 +72,9 @@ pub struct App {
     pub eww_state: eww_state::EwwState,
     pub eww_config: config::EwwConfig,
     pub open_windows: HashMap<String, EwwWindow>,
+    /// Window names that are supposed to be open, but failed.
+    /// When reloading the config, these should be opened again.
+    pub failed_windows: HashSet<String>,
     pub css_provider: gtk::CssProvider,
 
     #[debug_stub = "ScriptVarHandler(...)"]
@@ -179,7 +189,9 @@ impl App {
 
     fn stop_application(&mut self) {
         self.script_var_handler.stop_all();
-        self.open_windows.drain().for_each(|(_, w)| w.close());
+        for (_, window) in self.open_windows.drain() {
+            window.close();
+        }
         gtk::main_quit();
     }
 
@@ -193,12 +205,10 @@ impl App {
             self.script_var_handler.stop_for_variable(unused_var.clone());
         }
 
-        let window = self
-            .open_windows
+        self.open_windows
             .remove(window_name)
             .with_context(|| format!("Tried to close window named '{}', but no such window was open", window_name))?;
 
-        window.close();
         self.eww_state.clear_window_state(window_name);
 
         Ok(())
@@ -212,18 +222,19 @@ impl App {
         monitor: Option<i32>,
         anchor: Option<AnchorPoint>,
     ) -> Result<()> {
+        self.failed_windows.remove(window_name);
         log::info!("Opening window {}", window_name);
 
+        // if an instance of this is already running, close it
+        let _ = self.close_window(window_name);
+
+        let open_result: Result<_> = try {
             let mut window_def = self.eww_config.get_window(window_name)?.clone();
             window_def.geometry = window_def.geometry.map(|x| x.override_if_given(anchor, pos, size));
 
             let root_widget =
                 window_def.widget.render(&mut self.eww_state, window_name, &self.eww_config.get_widget_definitions())?;
 
-        // once generating the root widget has succeeded
-        // remove and close existing window with the same name
-        let _ = self.close_window(window_name);
-
             root_widget.get_style_context().add_class(&window_name.to_string());
 
             let monitor_geometry =
@@ -234,32 +245,35 @@ impl App {
 
             // initialize script var handlers for variables that where not used before opening this window.
             // TODO somehow make this less shit
-        for newly_used_var in self.variables_only_used_in(window_name).filter_map(|var| self.eww_config.get_script_var(var).ok())
+            for newly_used_var in
+                self.variables_only_used_in(window_name).filter_map(|var| self.eww_config.get_script_var(var).ok())
             {
                 self.script_var_handler.add(newly_used_var.clone());
             }
+        };
 
+        if let Err(err) = open_result {
+            self.failed_windows.insert(window_name.to_string());
+            Err(err)
+        } else {
             Ok(())
         }
+    }
 
     /// Load the given configuration, reloading all script-vars and attempting to reopen all windows that where opened.
     pub fn load_config(&mut self, config: config::EwwConfig) -> Result<()> {
+        // TODO the reload procedure is kinda bad.
+        // It should probably instead prepare a new eww_state and everything, and then swap the instances once everything has worked.
+
         log::info!("Reloading windows");
         // refresh script-var poll stuff
         self.script_var_handler.stop_all();
 
         self.eww_config = config;
+        self.eww_state.clear_all_window_states();
 
-        let new_state = EwwState::from_default_vars(self.eww_config.generate_initial_state()?);
-        let old_state = std::mem::replace(&mut self.eww_state, new_state);
-        for (key, value) in old_state.get_variables() {
-            if self.eww_state.get_variables().contains_key(key) {
-                self.eww_state.update_variable(key.clone(), value.clone())
-            }
-        }
-
-        let windows = self.open_windows.clone();
-        for (window_name, _) in windows {
+        let window_names: Vec<String> = self.open_windows.keys().chain(self.failed_windows.iter()).cloned().collect();
+        for window_name in &window_names {
             self.open_window(&window_name, None, None, None, None)?;
         }
         Ok(())

crates/eww/src/config/eww_config.rs

@@ -67,7 +67,7 @@ impl EwwConfig {
     }
 
     pub fn get_window(&self, name: &String) -> Result<&EwwWindowDefinition> {
-        self.windows.get(name).with_context(|| format!("No window named '{}' exists", name))
+        self.windows.get(name).with_context(|| format!("No window named '{}' exists in config", name))
     }
 
     pub fn get_script_var(&self, name: &VarName) -> Result<&ScriptVarDefinition> {

crates/eww/src/server.rs

@@ -1,12 +1,7 @@
 use crate::{EwwPaths, app::{self, DaemonCommand}, config, daemon_response, error_handling_ctx, eww_state::*, ipc_server, script_var_handler, util};
 use anyhow::*;
 
-use std::{
-    collections::HashMap,
-    os::unix::io::AsRawFd,
-    path::Path,
-    sync::{atomic::Ordering, Arc},
-};
+use std::{collections::{HashMap, HashSet}, os::unix::io::AsRawFd, path::Path, sync::{atomic::Ordering, Arc}};
 use tokio::sync::mpsc::*;
 
 pub fn initialize_server(paths: EwwPaths, action: Option<DaemonCommand>) -> Result<ForkResult> {
@@ -58,6 +53,7 @@ pub fn initialize_server(paths: EwwPaths, action: Option<DaemonCommand>) -> Resu
         eww_state: EwwState::from_default_vars(eww_config.generate_initial_state()?),
         eww_config,
         open_windows: HashMap::new(),
+        failed_windows: HashSet::new(),
         css_provider: gtk::CssProvider::new(),
         script_var_handler,
         app_evt_send: ui_send.clone(),