🐛 use panic() instead of process::exit() to terminate only the plugin…

… process
friedow · Oct 27, 2023 · fbc1da1 · fbc1da1
1 parent 77b8209
commit fbc1da1
Show file tree

Hide file tree

Showing 10 changed files with 19 additions and 18 deletions.
diff --git a/README.md b/README.md
@@ -38,6 +38,7 @@ This project is currently in a very early state and tailored to my needs and dai
 - [ ] unit converter (https://github.com/printfn/fend)
 - [x] display resource monitor (cpu, gpu, ram, disks)
   - [ ] split those into seperate files
+  - [ ] color background of entries based on the usage percentage
 - [x] display battery state
 - [x] display date and time
 - [ ] display weather

diff --git a/client/src/plugin/applications.rs b/client/src/plugin/applications.rs
@@ -177,7 +177,7 @@ impl Plugin {
                 target: self.plugin.id.as_str(),
                 "{:?}", error
             );
-            std::process::exit(1);
+            panic!();
         }
 
         let search_result = self.search(&String::from(""));

diff --git a/client/src/plugin/brave/bookmarks.rs b/client/src/plugin/brave/bookmarks.rs
@@ -29,7 +29,7 @@ impl Plugin {
                 "{:?}", error
             );
             // TODO: process exit terminates the parent process as well! This is unwanted
-            std::process::exit(1);
+            panic!();
         }
 
         return Plugin {
@@ -65,7 +65,7 @@ impl Plugin {
                 target: self.plugin.id.as_str(),
                 "{:?}", error,
             );
-            std::process::exit(1);
+            panic!();
         }
 
         loop {

diff --git a/client/src/plugin/brave/progressive_web_apps.rs b/client/src/plugin/brave/progressive_web_apps.rs
@@ -28,7 +28,7 @@ impl Plugin {
                 target: "brave-progressive-web-apps",
                 "{:?}", error
             );
-            std::process::exit(1);
+            panic!();
         }
 
         return Plugin {
@@ -72,7 +72,7 @@ impl Plugin {
                 target: self.plugin.id.as_str(),
                 "{:?}", error,
             );
-            std::process::exit(1);
+            panic!();
         }
 
         loop {

diff --git a/client/src/plugin/clock.rs b/client/src/plugin/clock.rs
@@ -64,7 +64,7 @@ impl Plugin {
                 target: self.plugin.id.as_str(),
                 "{:?}", error
             );
-            std::process::exit(1);
+            panic!();
         }
 
         loop {

diff --git a/client/src/plugin/git_repositories.rs b/client/src/plugin/git_repositories.rs
@@ -65,7 +65,7 @@ impl Plugin {
                 target: self.plugin.id.as_str(),
                 "{:?}", error,
             );
-            std::process::exit(1);
+            panic!();
         }
 
         loop {

diff --git a/client/src/plugin/resource_monitor.rs b/client/src/plugin/resource_monitor.rs
@@ -74,7 +74,7 @@ impl Plugin {
                 target: "resource-monitor",
                 "{:?}", error,
             );
-            std::process::exit(1);
+            panic!();
         }
 
         let update_entries_result = self.update_entries();

diff --git a/client/src/plugin/utils.rs b/client/src/plugin/utils.rs
@@ -40,7 +40,7 @@ where
             error = log::as_error!(error);
             "Could not read HOME environment variable",
         );
-        std::process::exit(1);
+        panic!();
     }
     let home_directory = home_directory_result.unwrap();
 
@@ -53,7 +53,7 @@ where
             error = log::as_error!(error);
             "Error while opening index file",
         );
-        std::process::exit(1);
+        panic!();
     }
     let index_file = index_file_result.unwrap();
 
@@ -64,7 +64,7 @@ where
             error = log::as_error!(error);
             "Error while reading index file",
         );
-        std::process::exit(1);
+        panic!();
     }
     let git_repository_paths = git_repository_paths_result.unwrap();
     git_repository_paths

diff --git a/client/src/plugin/windows.rs b/client/src/plugin/windows.rs
@@ -31,7 +31,7 @@ impl Plugin {
                 target: "windows",
                 "{:?}", error,
             );
-            std::process::exit(1);
+            panic!();
         }
         let mut sway = connection_result.unwrap();
 
@@ -41,7 +41,7 @@ impl Plugin {
                 target: "windows",
                 "{:?}", error,
             );
-            std::process::exit(1);
+            panic!();
         }
         let entries = entries_result.unwrap();
 
@@ -107,7 +107,7 @@ impl Plugin {
                 target: self.plugin.id.as_str(),
                 "{:?}", error,
             );
-            std::process::exit(1);
+            panic!();
         }
 
         loop {

diff --git a/services/index-git-repositories/src/main.rs b/services/index-git-repositories/src/main.rs
@@ -44,7 +44,7 @@ fn write_index_file(git_repository_paths: Vec<&str>) {
             error = log::as_error!(error);
             "Could read HOME environment variable",
         );
-        std::process::exit(1);
+        panic!();
     }
     let home_directory = home_directory_result.unwrap();
 
@@ -54,7 +54,7 @@ fn write_index_file(git_repository_paths: Vec<&str>) {
             error = log::as_error!(error);
             "Error while creating cache directory",
         );
-        std::process::exit(1);
+        panic!();
     }
 
     let index_file_path = cache_directory_path.join("git-repositories-index.json");
@@ -65,7 +65,7 @@ fn write_index_file(git_repository_paths: Vec<&str>) {
             error = log::as_error!(error);
             "Error while creating index file",
         );
-        std::process::exit(1);
+        panic!();
     }
     let index_file = index_file_result.unwrap();
 
@@ -75,6 +75,6 @@ fn write_index_file(git_repository_paths: Vec<&str>) {
             error = log::as_error!(error);
             "Error while writing index file",
         );
-        std::process::exit(1);
+        panic!();
     }
 }