diff options
author | Clement Tsang <34804052+ClementTsang@users.noreply.github.com> | 2023-01-20 00:08:02 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-01-20 00:08:02 -0500 |
commit | 4870ff365a91b68eb0b9528c75375f01ce539e75 (patch) | |
tree | beaa2b145556107558a9c319f38c877585c4b5ef /src | |
parent | 7f7a328977eda28725f2b67d7f8ab2e3697fa654 (diff) |
refactor: change name of some stuff, add some comments (#992)
* some quick refactoring first
* add todo for bug report template
Diffstat (limited to 'src')
-rw-r--r-- | src/app.rs | 10 | ||||
-rw-r--r-- | src/app/states.rs | 10 | ||||
-rw-r--r-- | src/lib.rs | 5 | ||||
-rw-r--r-- | src/options.rs | 6 | ||||
-rw-r--r-- | src/widgets/process_table.rs | 10 |
5 files changed, 18 insertions, 23 deletions
@@ -12,7 +12,7 @@ pub use states::*; use typed_builder::*; use unicode_segmentation::{GraphemeCursor, UnicodeSegmentation}; -use crate::widgets::{ProcWidget, ProcWidgetMode}; +use crate::widgets::{ProcWidgetMode, ProcWidgetState}; use crate::{ constants, data_conversion::ConvertedData, @@ -1190,7 +1190,7 @@ impl App { .proc_state .get_mut_widget_state(self.current_widget.widget_id) { - proc_widget_state.select_column(ProcWidget::CPU); + proc_widget_state.select_column(ProcWidgetState::CPU); } } } @@ -1200,7 +1200,7 @@ impl App { .proc_state .get_mut_widget_state(self.current_widget.widget_id) { - proc_widget_state.select_column(ProcWidget::MEM); + proc_widget_state.select_column(ProcWidgetState::MEM); } } else if let Some(disk) = self .disk_state @@ -1215,7 +1215,7 @@ impl App { .proc_state .get_mut_widget_state(self.current_widget.widget_id) { - proc_widget_state.select_column(ProcWidget::PID_OR_COUNT); + proc_widget_state.select_column(ProcWidgetState::PID_OR_COUNT); } } else if let Some(disk) = self .disk_state @@ -1240,7 +1240,7 @@ impl App { .proc_state .get_mut_widget_state(self.current_widget.widget_id) { - proc_widget_state.select_column(ProcWidget::PROC_NAME_OR_CMD); + proc_widget_state.select_column(ProcWidgetState::PROC_NAME_OR_CMD); } } else if let Some(disk) = self .disk_state diff --git a/src/app/states.rs b/src/app/states.rs index e4141e4f..30e20291 100644 --- a/src/app/states.rs +++ b/src/app/states.rs @@ -9,7 +9,7 @@ use crate::{ utils::gen_util::str_width, widgets::{ BatteryWidgetState, CpuWidgetState, DiskTableWidget, MemWidgetState, NetWidgetState, - ProcWidget, TempWidgetState, + ProcWidgetState, TempWidgetState, }, }; @@ -251,19 +251,19 @@ impl AppSearchState { } pub struct ProcState { - pub widget_states: HashMap<u64, ProcWidget>, + pub widget_states: HashMap<u64, ProcWidgetState>, } impl ProcState { - pub fn init(widget_states: HashMap<u64, ProcWidget>) -> Self { + pub fn init(widget_states: HashMap<u64, ProcWidgetState>) -> Self { ProcState { widget_states } } - pub fn get_mut_widget_state(&mut self, widget_id: u64) -> Option<&mut ProcWidget> { + pub fn get_mut_widget_state(&mut self, widget_id: u64) -> Option<&mut ProcWidgetState> { self.widget_states.get_mut(&widget_id) } - pub fn get_widget_state(&self, widget_id: u64) -> Option<&ProcWidget> { + pub fn get_widget_state(&self, widget_id: u64) -> Option<&ProcWidgetState> { self.widget_states.get(&widget_id) } } @@ -113,11 +113,6 @@ pub fn handle_key_event_or_break( ) -> bool { // debug!("KeyEvent: {:?}", event); - // TODO: [PASTE] Note that this does NOT support some emojis like flags. This is due to us - // catching PER CHARACTER right now WITH A forced throttle! This means multi-char will not work. - // We can solve this (when we do paste probably) while keeping the throttle (mainly meant for movement) - // by throttling after *bulk+singular* actions, not just singular ones. - if event.modifiers.is_empty() { // Required catch for searching - otherwise you couldn't search with q. if event.code == KeyCode::Char('q') && !app.is_in_search_widget() { diff --git a/src/options.rs b/src/options.rs index 5c160310..08690537 100644 --- a/src/options.rs +++ b/src/options.rs @@ -24,7 +24,7 @@ use crate::{ utils::error::{self, BottomError}, widgets::{ BatteryWidgetState, CpuWidgetState, DiskTableWidget, MemWidgetState, NetWidgetState, - ProcWidget, ProcWidgetMode, TempWidgetState, + ProcWidgetMode, ProcWidgetState, TempWidgetState, }, }; @@ -198,7 +198,7 @@ pub fn build_app( let mut cpu_state_map: HashMap<u64, CpuWidgetState> = HashMap::new(); let mut mem_state_map: HashMap<u64, MemWidgetState> = HashMap::new(); let mut net_state_map: HashMap<u64, NetWidgetState> = HashMap::new(); - let mut proc_state_map: HashMap<u64, ProcWidget> = HashMap::new(); + let mut proc_state_map: HashMap<u64, ProcWidgetState> = HashMap::new(); let mut temp_state_map: HashMap<u64, TempWidgetState> = HashMap::new(); let mut disk_state_map: HashMap<u64, DiskTableWidget> = HashMap::new(); let mut battery_state_map: HashMap<u64, BatteryWidgetState> = HashMap::new(); @@ -326,7 +326,7 @@ pub fn build_app( proc_state_map.insert( widget.widget_id, - ProcWidget::new( + ProcWidgetState::new( &app_config_fields, mode, is_case_sensitive, diff --git a/src/widgets/process_table.rs b/src/widgets/process_table.rs index 46d6ef60..03b74ad0 100644 --- a/src/widgets/process_table.rs +++ b/src/widgets/process_table.rs @@ -72,7 +72,7 @@ type ProcessTable = SortDataTable<ProcWidgetData, ProcColumn>; type SortTable = DataTable<Cow<'static, str>, SortTableColumn>; type StringPidMap = FxHashMap<String, Vec<Pid>>; -pub struct ProcWidget { +pub struct ProcWidgetState { pub mode: ProcWidgetMode, /// The state of the search box. @@ -92,7 +92,7 @@ pub struct ProcWidget { pub force_update_data: bool, } -impl ProcWidget { +impl ProcWidgetState { pub const PID_OR_COUNT: usize = 0; pub const PROC_NAME_OR_CMD: usize = 1; pub const CPU: usize = 2; @@ -222,7 +222,7 @@ impl ProcWidget { let id_pid_map = FxHashMap::default(); - let mut table = ProcWidget { + let mut table = ProcWidgetState { proc_search: process_search_state, table, sort_table, @@ -240,7 +240,7 @@ impl ProcWidget { pub fn is_using_command(&self) -> bool { self.table .columns - .get(ProcWidget::PROC_NAME_OR_CMD) + .get(ProcWidgetState::PROC_NAME_OR_CMD) .map(|col| matches!(col.inner(), ProcColumn::Command)) .unwrap_or(false) } @@ -248,7 +248,7 @@ impl ProcWidget { pub fn is_mem_percent(&self) -> bool { self.table .columns - .get(ProcWidget::MEM) + .get(ProcWidgetState::MEM) .map(|col| matches!(col.inner(), ProcColumn::MemoryPercent)) .unwrap_or(false) } |