Ecosystem/gupaxx
1
0
Fork 0
mirror of https://github.com/Cyrix126/gupaxx.git synced 2024-12-22 14:49:21 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: high cpu usage when moving mouse on Tab with service ready

Browse source 
Check if the program is running outside of Gupaxx only if the user wants
to start it. Do not check at all times.
This commit is contained in:
Cyrix126 2024-12-11 21:56:53 +01:00
parent 77cc9faf62
commit a920c53d56
1 changed files with 12 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
src/app/panels/bottom.rs
View file

@ -12,7 +12,7 @@ use crate::utils::constants::*;
use crate::utils::errors::{ErrorButtons, ErrorFerris}; use crate::utils::errors::{ErrorButtons, ErrorFerris};
use crate::utils::regex::Regexes; use crate::utils::regex::Regexes;
use egui::*; use egui::*;
use log::debug; use log::{debug, error};
use crate::app::Tab; use crate::app::Tab;
use crate::helper::ProcessState::*; use crate::helper::ProcessState::*;
@ -300,6 +300,16 @@ impl crate::app::App {
.on_disabled_hover_text(text_err) .on_disabled_hover_text(text_err)
.clicked() .clicked()
{ {
// check if process is running outside of Gupaxx, warn about it and do not start it.
if process_running(name) {
error!("Process already running outside: {}", name);
self.error_state.set(
PROCESS_OUTSIDE,
ErrorFerris::Error,
ErrorButtons::Okay,
);
return;
}
let _ = self.og.lock().unwrap().update_absolute_path(); let _ = self.og.lock().unwrap().update_absolute_path();
let _ = self.state.update_absolute_path(); let _ = self.state.update_absolute_path();
// start process // start process
@ -550,13 +560,8 @@ impl crate::app::App {
}); });
} }
pub fn start_ready(&self, state: &ProcessStateGui) -> Result<(), String> { pub fn start_ready(&self, state: &ProcessStateGui) -> Result<(), String> {
// check if process running outside
let name = state.name;
if process_running(name) {
return Err(format!("Error: {}", PROCESS_OUTSIDE));
}
// custom check and var // custom check and var
let name = state.name;
let path = match name { let path = match name {
ProcessName::Node => { ProcessName::Node => {
// check path of DB valid, empty valid. // check path of DB valid, empty valid.