From f813ca6ada78047ddea7471cb9873f9198ea7e64 Mon Sep 17 00:00:00 2001
From: a-kenji <aks.kenji@protonmail.com>
Date: Thu, 18 Jul 2024 23:37:55 +0200
Subject: [PATCH] =?UTF-8?q?=F0=9F=90=9B=20Change=20settings=20to=20be=20lo?=
 =?UTF-8?q?aded=20only=20once=20from=20disk=20(#165)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Add the `get_or_init` function to `[Settings]` that will cache the
result in memory once it reads the settings file from disk.
---
 client/src/component/entry.rs       | 2 +-
 client/src/component/query_input.rs | 8 ++++----
 client/src/main.rs                  | 8 ++++----
 client/src/settings.rs              | 6 ++++++
 4 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/client/src/component/entry.rs b/client/src/component/entry.rs
index 456e05f..6b8b0c4 100644
--- a/client/src/component/entry.rs
+++ b/client/src/component/entry.rs
@@ -41,7 +41,7 @@ impl iced::widget::container::StyleSheet for Style {
     type Style = iced::Theme;
 
     fn appearance(&self, _style: &Self::Style) -> iced::widget::container::Appearance {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
 
         iced::widget::container::Appearance {
             background: None,
diff --git a/client/src/component/query_input.rs b/client/src/component/query_input.rs
index f8a1e6d..60f9ff7 100644
--- a/client/src/component/query_input.rs
+++ b/client/src/component/query_input.rs
@@ -49,22 +49,22 @@ impl iced::widget::text_input::StyleSheet for Style {
     }
 
     fn placeholder_color(&self, _style: &Self::Style) -> iced::Color {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
         crate::settings::hexcolor(&color_settings.color.surface)
     }
 
     fn value_color(&self, _style: &Self::Style) -> iced::Color {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
         crate::settings::hexcolor(&color_settings.color.text)
     }
 
     fn disabled_color(&self, _style: &Self::Style) -> iced::Color {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
         crate::settings::hexcolor(&color_settings.color.surface)
     }
 
     fn selection_color(&self, _style: &Self::Style) -> iced::Color {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
         crate::settings::hexcolor(&color_settings.color.surface)
     }
 }
diff --git a/client/src/main.rs b/client/src/main.rs
index b480263..1047234 100644
--- a/client/src/main.rs
+++ b/client/src/main.rs
@@ -529,7 +529,7 @@ impl iced::application::StyleSheet for SandboxStyle {
     type Style = iced::Theme;
 
     fn appearance(&self, _style: &Self::Style) -> iced::application::Appearance {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
 
         iced::application::Appearance {
             background_color: iced::Color::TRANSPARENT,
@@ -543,7 +543,7 @@ impl iced::widget::container::StyleSheet for ApplicationWrapperStyle {
     type Style = iced::Theme;
 
     fn appearance(&self, _style: &Self::Style) -> iced::widget::container::Appearance {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
         iced::widget::container::Appearance {
             background: Some(iced::Background::Color(settings::hexcolor(
                 &color_settings.color.background,
@@ -561,7 +561,7 @@ impl iced::widget::scrollable::StyleSheet for ScrollableStyle {
     type Style = iced::Theme;
 
     fn active(&self, _style: &Self::Style) -> iced::widget::scrollable::Scrollbar {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
         iced::widget::scrollable::Scrollbar {
             background: None,
             border_radius: iced::BorderRadius::from(0.),
@@ -581,7 +581,7 @@ impl iced::widget::scrollable::StyleSheet for ScrollableStyle {
         _style: &Self::Style,
         _is_mouse_over_scrollbar: bool,
     ) -> iced::widget::scrollable::Scrollbar {
-        let color_settings = crate::settings::Settings::new();
+        let color_settings = crate::settings::Settings::get_or_init();
         iced::widget::scrollable::Scrollbar {
             background: None,
             border_radius: iced::BorderRadius::from(0.),
diff --git a/client/src/settings.rs b/client/src/settings.rs
index 4b27f1d..8dbe7d3 100644
--- a/client/src/settings.rs
+++ b/client/src/settings.rs
@@ -1,5 +1,6 @@
 use hex_color::HexColor;
 use serde::Deserialize;
+use std::sync::OnceLock;
 
 pub fn hexcolor(color: &str) -> iced::Color {
     let hex_col = HexColor::parse(color).unwrap_or_else(|_| {
@@ -327,6 +328,11 @@ impl Settings {
         }
         config_result.unwrap()
     }
+
+    pub fn get_or_init() -> &'static Self {
+        static SETTINGS: OnceLock<Settings> = OnceLock::new();
+        SETTINGS.get_or_init(Self::new)
+    }
 }
 
 impl std::convert::TryFrom<crate::cli::CliArgs> for Settings {