Remove register_config_handler, pass config directly to ::new()

src/app.rs

@@ -58,10 +58,7 @@ impl App {
     frame_rate: f64,
     log_receiver: mpsc::UnboundedReceiver<String>,
   ) -> Result<Self> {
-    let home = Home::new();
-    let fps = FpsCounter::default();
     let config = Config::new()?;
-    let mode = Mode::Home;
     let datadir = config.config._data_dir.join("default");
     let future_clients = config.accounts.clone().map(|conf| {
       let datadir = datadir.clone();
@@ -123,6 +120,10 @@ impl App {
     }
     let clients = NonEmpty::collect(clients).expect("map on NonEmpty returned empty vec");
 
+    let home = Home::new(config.clone());
+    let fps = FpsCounter::default();
+    let mode = Mode::Home;
+
     let mut app = Self {
       tick_rate,
       frame_rate,
@@ -154,10 +155,6 @@ impl App {
       component.register_action_handler(action_tx.clone())?;
     }
 
-    for component in self.components.iter_mut() {
-      component.register_config_handler(self.config.clone())?;
-    }
-
     for component in self.components.iter_mut() {
       component.init(tui.size()?)?;
     }

src/components.rs

@@ -32,19 +32,6 @@ pub trait Component {
   fn register_action_handler(&mut self, tx: UnboundedSender<Action>) -> Result<()> {
     Ok(())
   }
-  /// Register a configuration handler that provides configuration settings if necessary.
-  ///
-  /// # Arguments
-  ///
-  /// * `config` - Configuration settings.
-  ///
-  /// # Returns
-  ///
-  /// * `Result<()>` - An Ok result or an error.
-  #[allow(unused_variables)]
-  fn register_config_handler(&mut self, config: Config) -> Result<()> {
-    Ok(())
-  }
   /// Initialize the component with a specified area if necessary.
   ///
   /// # Arguments

src/components/buflist.rs

@@ -25,14 +25,16 @@ use crate::{
   config::{Config, KeyBindings},
 };
 
-#[derive(Default)]
 pub struct Buflist {
   command_tx: Option<UnboundedSender<Action>>,
-  config: OnceCell<Config>,
+  config: Config,
 }
 impl Buflist {
-  pub fn new() -> Self {
-    Self::default()
+  pub fn new(config: Config) -> Self {
+    Buflist {
+      command_tx: None,
+      config,
+    }
   }
 }
 
@@ -42,13 +44,6 @@ impl Component for Buflist {
     Ok(())
   }
 
-  fn register_config_handler(&mut self, config: Config) -> Result<()> {
-    self
-      .config
-      .set(config)
-      .context("Buflist config was already set")
-  }
-
   fn update(&mut self, action: &Action) -> Result<Option<Action>> {
     Ok(None)
   }

src/components/home.rs

@@ -30,18 +30,23 @@ use crate::{
   config::{Config, KeyBindings},
 };
 
-#[derive(Default)]
 pub struct Home {
   command_tx: Option<UnboundedSender<Action>>,
-  config: OnceCell<Config>,
+  config: Config,
   buflist: Buflist,
   backlog: Backlog,
   textarea: TextArea<'static>,
 }
 
 impl Home {
-  pub fn new() -> Self {
-    let mut self_ = Self::default();
+  pub fn new(config: Config) -> Self {
+    let mut self_ = Home {
+      command_tx: None,
+      buflist: Buflist::new(config.clone()),
+      backlog: Backlog::default(),
+      textarea: TextArea::default(),
+      config,
+    };
     self_.configure_textarea();
     self_
   }
@@ -61,15 +66,6 @@ impl Component for Home {
     Ok(())
   }
 
-  fn register_config_handler(&mut self, config: Config) -> Result<()> {
-    self.buflist.register_config_handler(config.clone())?;
-    self
-      .config
-      .set(config)
-      .context("Home config was already set")?;
-    Ok(())
-  }
-
   fn handle_key_events(&mut self, key: KeyEvent) -> Result<Option<Action>> {
     match key {
       KeyEvent {