From a71d991695fb3874d6cb329291cfcfa986e0eee4 Mon Sep 17 00:00:00 2001 From: Clement Tsang <34804052+ClementTsang@users.noreply.github.com> Date: Thu, 7 May 2020 01:09:24 -0400 Subject: [PATCH] refactor: avoid using max checks, use saturating_sub instead --- Cargo.lock | 16 ++++++------- src/app.rs | 6 ++--- src/app/query.rs | 2 +- src/app/states.rs | 2 +- src/canvas.rs | 9 ++++--- src/canvas/dialogs/dd_dialog.rs | 19 +++++++-------- src/canvas/dialogs/help_dialog.rs | 8 +++---- src/canvas/widgets/basic_table_arrows.rs | 9 +++---- src/canvas/widgets/battery_display.rs | 17 +++++--------- src/canvas/widgets/cpu_basic.rs | 12 ++++------ src/canvas/widgets/cpu_graph.rs | 30 ++++++++++-------------- src/canvas/widgets/disk_table.rs | 16 +++++-------- src/canvas/widgets/mem_basic.rs | 4 +--- src/canvas/widgets/mem_graph.rs | 16 ++++--------- src/canvas/widgets/network_graph.rs | 14 ++++------- src/canvas/widgets/process_table.rs | 30 +++++++++--------------- src/canvas/widgets/temp_table.rs | 17 +++++--------- 17 files changed, 89 insertions(+), 138 deletions(-) diff --git a/Cargo.lock b/Cargo.lock index 8c489dd6..2f736631 100644 --- a/Cargo.lock +++ b/Cargo.lock @@ -179,7 +179,7 @@ dependencies = [ "strsim 0.8.0 (registry+https://github.com/rust-lang/crates.io-index)", "textwrap 0.11.0 (registry+https://github.com/rust-lang/crates.io-index)", "unicode-width 0.1.7 (registry+https://github.com/rust-lang/crates.io-index)", - "vec_map 0.8.1 (registry+https://github.com/rust-lang/crates.io-index)", + "vec_map 0.8.2 (registry+https://github.com/rust-lang/crates.io-index)", ] [[package]] @@ -423,7 +423,7 @@ dependencies = [ "proc-macro-hack 0.5.15 (registry+https://github.com/rust-lang/crates.io-index)", "proc-macro2 1.0.12 (registry+https://github.com/rust-lang/crates.io-index)", "quote 1.0.4 (registry+https://github.com/rust-lang/crates.io-index)", - "syn 1.0.18 (registry+https://github.com/rust-lang/crates.io-index)", + "syn 1.0.19 (registry+https://github.com/rust-lang/crates.io-index)", ] [[package]] @@ -1064,7 +1064,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index" dependencies = [ "proc-macro2 1.0.12 (registry+https://github.com/rust-lang/crates.io-index)", "quote 1.0.4 (registry+https://github.com/rust-lang/crates.io-index)", - "syn 1.0.18 (registry+https://github.com/rust-lang/crates.io-index)", + "syn 1.0.19 (registry+https://github.com/rust-lang/crates.io-index)", ] [[package]] @@ -1103,7 +1103,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index" [[package]] name = "syn" -version = "1.0.18" +version = "1.0.19" source = "registry+https://github.com/rust-lang/crates.io-index" dependencies = [ "proc-macro2 1.0.12 (registry+https://github.com/rust-lang/crates.io-index)", @@ -1192,7 +1192,7 @@ source = "registry+https://github.com/rust-lang/crates.io-index" dependencies = [ "proc-macro2 1.0.12 (registry+https://github.com/rust-lang/crates.io-index)", "quote 1.0.4 (registry+https://github.com/rust-lang/crates.io-index)", - "syn 1.0.18 (registry+https://github.com/rust-lang/crates.io-index)", + "syn 1.0.19 (registry+https://github.com/rust-lang/crates.io-index)", ] [[package]] @@ -1236,7 +1236,7 @@ dependencies = [ [[package]] name = "vec_map" -version = "0.8.1" +version = "0.8.2" source = "registry+https://github.com/rust-lang/crates.io-index" [[package]] @@ -1424,7 +1424,7 @@ dependencies = [ "checksum slab 0.4.2 (registry+https://github.com/rust-lang/crates.io-index)" = "c111b5bd5695e56cffe5129854aa230b39c93a305372fdbb2668ca2394eea9f8" "checksum smallvec 1.4.0 (registry+https://github.com/rust-lang/crates.io-index)" = "c7cb5678e1615754284ec264d9bb5b4c27d2018577fd90ac0ceb578591ed5ee4" "checksum strsim 0.8.0 (registry+https://github.com/rust-lang/crates.io-index)" = "8ea5119cdb4c55b55d432abb513a0429384878c15dde60cc77b1c99de1a95a6a" -"checksum syn 1.0.18 (registry+https://github.com/rust-lang/crates.io-index)" = "410a7488c0a728c7ceb4ad59b9567eb4053d02e8cc7f5c0e0eeeb39518369213" +"checksum syn 1.0.19 (registry+https://github.com/rust-lang/crates.io-index)" = "e8e5aa70697bb26ee62214ae3288465ecec0000f05182f039b477001f08f5ae7" "checksum sysinfo 0.14.1 (registry+https://github.com/rust-lang/crates.io-index)" = "d070254b7172eee9eb3990bea8f72aeabfe1226c40bf71a52e4fe75777812e9a" "checksum textwrap 0.11.0 (registry+https://github.com/rust-lang/crates.io-index)" = "d326610f408c7a4eb6f51c37c330e496b08506c9457c9d34287ecc38809fb060" "checksum thread_local 1.0.1 (registry+https://github.com/rust-lang/crates.io-index)" = "d40c6d1b69745a6ec6fb1ca717914848da4b44ae29d9b3080cbee91d72a69b14" @@ -1440,7 +1440,7 @@ dependencies = [ "checksum unicode-xid 0.2.0 (registry+https://github.com/rust-lang/crates.io-index)" = "826e7639553986605ec5979c7dd957c7895e93eabed50ab2ffa7f6128a75097c" "checksum uom 0.26.0 (registry+https://github.com/rust-lang/crates.io-index)" = "4cec796ec5f7ac557631709079168286056205c51c60aac33f51764bdc7b8dc4" "checksum uom 0.27.0 (registry+https://github.com/rust-lang/crates.io-index)" = "51fc04fb44bcb7806da71885872cb15d123b681e459a476ef8a0bab287bee0cd" -"checksum vec_map 0.8.1 (registry+https://github.com/rust-lang/crates.io-index)" = "05c78687fb1a80548ae3250346c3db86a80a7cdd77bda190189f2d0a0987c81a" +"checksum vec_map 0.8.2 (registry+https://github.com/rust-lang/crates.io-index)" = "f1bddf1187be692e79c5ffeab891132dfb0f236ed36a43c7ed39f1165ee20191" "checksum void 1.0.2 (registry+https://github.com/rust-lang/crates.io-index)" = "6a02e4885ed3bc0f2de90ea6dd45ebcbb66dacffe03547fadbb0eeae2770887d" "checksum wait-timeout 0.2.0 (registry+https://github.com/rust-lang/crates.io-index)" = "9f200f5b12eb75f8c1ed65abd4b2db8a6e1b138a20de009dacee265a2498f3f6" "checksum wasi 0.9.0+wasi-snapshot-preview1 (registry+https://github.com/rust-lang/crates.io-index)" = "cccddf32554fecc6acb585f82a32a72e28b48f8c4c1883ddfeeeaa96f7d8e519" diff --git a/src/app.rs b/src/app.rs index 02ed2292..e6abe093 100644 --- a/src/app.rs +++ b/src/app.rs @@ -1,4 +1,4 @@ -use std::{cmp::max, collections::HashMap, time::Instant}; +use std::{collections::HashMap, time::Instant}; use unicode_segmentation::GraphemeCursor; use unicode_width::{UnicodeWidthChar, UnicodeWidthStr}; @@ -166,7 +166,7 @@ impl App { >= cpu_widget_state.num_cpus_shown as u64 { let new_position = - max(0, cpu_widget_state.num_cpus_shown as i64 - 1) as u64; + cpu_widget_state.num_cpus_shown.saturating_sub(1) as u64; cpu_widget_state.scroll_state.current_scroll_position = new_position; cpu_widget_state.scroll_state.previous_scroll_position = 0; } @@ -184,7 +184,7 @@ impl App { >= cpu_widget_state.num_cpus_shown as u64 { let new_position = - max(0, cpu_widget_state.num_cpus_shown as i64 - 1) as u64; + cpu_widget_state.num_cpus_shown.saturating_sub(1) as u64; cpu_widget_state.scroll_state.current_scroll_position = new_position; cpu_widget_state.scroll_state.previous_scroll_position = 0; } diff --git a/src/app/query.rs b/src/app/query.rs index 863298db..0e98d835 100644 --- a/src/app/query.rs +++ b/src/app/query.rs @@ -412,7 +412,7 @@ impl ProcessQuery for ProcWidgetState { #[derive(Debug)] pub struct Query { - /// Remember, AND > OR, but and must come after or then. + /// Remember, AND > OR, but AND must come after OR when we parse. pub query: And, } diff --git a/src/app/states.rs b/src/app/states.rs index 67ab5a5f..98913e61 100644 --- a/src/app/states.rs +++ b/src/app/states.rs @@ -205,7 +205,7 @@ impl ProcWidgetState { self.process_search_state.search_state.error_message = None; } else { let parsed_query = self.parse_query(); - debug!("PQ: {:#?}", parsed_query); + // debug!("PQ: {:#?}", parsed_query); if let Ok(parsed_query) = parsed_query { self.process_search_state.search_state.query = Some(parsed_query); diff --git a/src/canvas.rs b/src/canvas.rs index a7df9ea9..e141214e 100644 --- a/src/canvas.rs +++ b/src/canvas.rs @@ -1,5 +1,4 @@ use itertools::izip; -use std::cmp::max; use std::collections::HashMap; use tui::{ @@ -66,7 +65,7 @@ pub struct Painter { layout_constraints: Vec>>>, widget_layout: BottomLayout, derived_widget_draw_locs: Vec>>>, - table_height_offset: i64, + table_height_offset: u16, } impl Painter { @@ -150,7 +149,7 @@ impl Painter { layout_constraints, widget_layout, derived_widget_draw_locs: Vec::new(), - table_height_offset: 4 + table_gap as i64, + table_height_offset: 4 + table_gap, } } @@ -209,7 +208,7 @@ impl Painter { terminal.draw(|mut f| { if app_state.help_dialog_state.is_showing_help { let gen_help_len = GENERAL_HELP_TEXT.len() as u16 + 3; - let border_len = (max(0, f.size().height as i64 - gen_help_len as i64)) as u16 / 2; + let border_len = f.size().height.saturating_sub(gen_help_len) / 2; let vertical_dialog_chunk = Layout::default() .direction(Direction::Vertical) .constraints( @@ -245,7 +244,7 @@ impl Painter { self.draw_help_dialog(&mut f, app_state, middle_dialog_chunk[1]); } else if app_state.delete_dialog_state.is_showing_dd { - let bordering = (max(0, f.size().height as i64 - 7) as u16) / 2; + let bordering = f.size().height.saturating_sub(7) / 2; let vertical_dialog_chunk = Layout::default() .direction(Direction::Vertical) .constraints( diff --git a/src/canvas/dialogs/dd_dialog.rs b/src/canvas/dialogs/dd_dialog.rs index 2e952fae..d4b605d6 100644 --- a/src/canvas/dialogs/dd_dialog.rs +++ b/src/canvas/dialogs/dd_dialog.rs @@ -1,5 +1,3 @@ -use std::cmp::max; - use tui::{ backend::Backend, layout::{Alignment, Rect}, @@ -61,13 +59,11 @@ impl KillDialog for Painter { }, ]; - let repeat_num = max( - 0, - draw_loc.width as i32 - DD_BASE.chars().count() as i32 - 2, - ); let dd_title = format!( " Confirm Kill Process ─{}─ Esc to close ", - "─".repeat(repeat_num as usize) + "─".repeat( + usize::from(draw_loc.width).saturating_sub(DD_BASE.chars().count() + 2) + ) ); f.render_widget( @@ -102,11 +98,12 @@ impl KillDialog for Painter { dd_err ))]; - let repeat_num = max( - 0, - draw_loc.width as i32 - DD_ERROR_BASE.chars().count() as i32 - 2, + let error_title = format!( + " Error ─{}─ Esc to close ", + "─".repeat( + usize::from(draw_loc.width).saturating_sub(DD_ERROR_BASE.chars().count() + 2) + ) ); - let error_title = format!(" Error ─{}─ Esc to close ", "─".repeat(repeat_num as usize)); f.render_widget( Paragraph::new(dd_text.iter()) diff --git a/src/canvas/dialogs/help_dialog.rs b/src/canvas/dialogs/help_dialog.rs index c75aef10..a11e19e1 100644 --- a/src/canvas/dialogs/help_dialog.rs +++ b/src/canvas/dialogs/help_dialog.rs @@ -1,4 +1,3 @@ -use std::cmp::max; use unicode_width::UnicodeWidthStr; use tui::{ @@ -22,11 +21,10 @@ impl HelpDialog for Painter { fn draw_help_dialog( &self, f: &mut Frame<'_, B>, app_state: &mut App, draw_loc: Rect, ) { - let repeat_num = max( - 0, - draw_loc.width as i32 - HELP_BASE.chars().count() as i32 - 2, + let help_title = format!( + " Help ─{}─ Esc to close ", + "─".repeat(usize::from(draw_loc.width).saturating_sub(HELP_BASE.chars().count() + 2)) ); - let help_title = format!(" Help ─{}─ Esc to close ", "─".repeat(repeat_num as usize)); if app_state.is_force_redraw { // We must also recalculate how many lines are wrapping to properly get scrolling to work on diff --git a/src/canvas/widgets/basic_table_arrows.rs b/src/canvas/widgets/basic_table_arrows.rs index abfb1126..b3a6fcf3 100644 --- a/src/canvas/widgets/basic_table_arrows.rs +++ b/src/canvas/widgets/basic_table_arrows.rs @@ -1,5 +1,3 @@ -use std::cmp::max; - use crate::{ app::{ layout_manager::{BottomWidget, BottomWidgetType}, @@ -56,10 +54,9 @@ impl BasicTableArrows for Painter { let left_name = left_table.get_pretty_name(); let right_name = right_table.get_pretty_name(); - let num_spaces = max( - 0, - draw_loc.width as i64 - 2 - 4 - (left_name.len() + right_name.len()) as i64, - ) as usize; + let num_spaces = usize::from(draw_loc.width) + .saturating_sub(6 + left_name.len() + right_name.len()) + as usize; let arrow_text = vec![ Text::Styled(format!("\n◄ {}", left_name).into(), self.colours.text_style), diff --git a/src/canvas/widgets/battery_display.rs b/src/canvas/widgets/battery_display.rs index 8d4c7a5f..943ea9a8 100644 --- a/src/canvas/widgets/battery_display.rs +++ b/src/canvas/widgets/battery_display.rs @@ -1,5 +1,3 @@ -use std::cmp::max; - use crate::{ app::App, canvas::{drawing_utils::calculate_basic_use_bars, Painter}, @@ -31,15 +29,12 @@ impl BatteryDisplayWidget for Painter { let is_on_widget = widget_id == app_state.current_widget.widget_id; let title = if app_state.is_expanded { const TITLE_BASE: &str = " Battery ── Esc to go back "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - let result_title = format!( + format!( " Battery ─{}─ Esc to go back ", - "─".repeat(repeat_num as usize) - ); - result_title + "─".repeat( + usize::from(draw_loc.width).saturating_sub(TITLE_BASE.chars().count() + 2) + ) + ) } else { " Battery ".to_string() }; @@ -74,7 +69,7 @@ impl BatteryDisplayWidget for Painter { .get(battery_widget_state.currently_selected_battery_index) { // Assuming a 50/50 split in width - let bar_length = max(0, (draw_loc.width as i64 - 2) / 2 - 8) as usize; + let bar_length = (draw_loc.width.saturating_sub(2) / 2).saturating_sub(8) as usize; let charge_percentage = battery_details.charge_percentage; let num_bars = calculate_basic_use_bars(charge_percentage, bar_length); let bars = format!( diff --git a/src/canvas/widgets/cpu_basic.rs b/src/canvas/widgets/cpu_basic.rs index a3299d7f..858b1732 100644 --- a/src/canvas/widgets/cpu_basic.rs +++ b/src/canvas/widgets/cpu_basic.rs @@ -1,4 +1,4 @@ -use std::cmp::{max, min}; +use std::cmp::min; use crate::{ app::App, @@ -58,12 +58,10 @@ impl CpuBasicWidget for Painter { .split(draw_loc); // +9 due to 3 + 4 + 2 columns for the name & space + percentage + bar bounds - let margin_space = 2; - let remaining_width = max( - 0, - draw_loc.width as i64 - - ((9 + margin_space) * REQUIRED_COLUMNS - margin_space) as i64, - ) as usize; + const MARGIN_SPACE: usize = 2; + let remaining_width = usize::from(draw_loc.width) + .saturating_sub((9 + MARGIN_SPACE) * REQUIRED_COLUMNS - MARGIN_SPACE) + as usize; let bar_length = remaining_width / REQUIRED_COLUMNS; diff --git a/src/canvas/widgets/cpu_graph.rs b/src/canvas/widgets/cpu_graph.rs index 1ee9fa5a..73b8a9c0 100644 --- a/src/canvas/widgets/cpu_graph.rs +++ b/src/canvas/widgets/cpu_graph.rs @@ -170,14 +170,12 @@ impl CpuGraphWidget for Painter { let title = if app_state.is_expanded && !cpu_widget_state.is_showing_tray { const TITLE_BASE: &str = " CPU ── Esc to go back "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - let result_title = - format!(" CPU ─{}─ Esc to go back ", "─".repeat(repeat_num as usize)); - - result_title + format!( + " CPU ─{}─ Esc to go back ", + "─".repeat( + usize::from(draw_loc.width).saturating_sub(TITLE_BASE.chars().count() + 2) + ) + ) } else { " CPU ".to_string() }; @@ -217,9 +215,8 @@ impl CpuGraphWidget for Painter { cpu_widget_state.is_legend_hidden = false; let cpu_data: &mut [ConvertedCpuData] = &mut app_state.canvas_data.cpu_data; - let num_rows = max(0, i64::from(draw_loc.height) - self.table_height_offset) as u64; let start_position = get_start_position( - num_rows, + draw_loc.height.saturating_sub(self.table_height_offset) as u64, &cpu_widget_state.scroll_state.scroll_direction, &mut cpu_widget_state.scroll_state.previous_scroll_position, cpu_widget_state.scroll_state.current_scroll_position, @@ -304,13 +301,12 @@ impl CpuGraphWidget for Painter { let title = if cpu_widget_state.is_showing_tray { const TITLE_BASE: &str = " Esc to close "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - let result_title = format!("{} Esc to close ", "─".repeat(repeat_num as usize)); - - result_title + format!( + "{} Esc to close ", + "─".repeat( + usize::from(draw_loc.width).saturating_sub(TITLE_BASE.chars().count() + 2) + ) + ) } else { "".to_string() }; diff --git a/src/canvas/widgets/disk_table.rs b/src/canvas/widgets/disk_table.rs index fda54d97..a9cabf9b 100644 --- a/src/canvas/widgets/disk_table.rs +++ b/src/canvas/widgets/disk_table.rs @@ -39,9 +39,8 @@ impl DiskTableWidget for Painter { ) { if let Some(disk_widget_state) = app_state.disk_state.widget_states.get_mut(&widget_id) { let disk_data: &mut [Vec] = &mut app_state.canvas_data.disk_data; - let num_rows = max(0, i64::from(draw_loc.height) - self.table_height_offset) as u64; let start_position = get_start_position( - num_rows, + draw_loc.height.saturating_sub(self.table_height_offset) as u64, &disk_widget_state.scroll_state.scroll_direction, &mut disk_widget_state.scroll_state.previous_scroll_position, disk_widget_state.scroll_state.current_scroll_position, @@ -66,15 +65,12 @@ impl DiskTableWidget for Painter { let title = if app_state.is_expanded { const TITLE_BASE: &str = " Disk ── Esc to go back "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - let result_title = format!( + format!( " Disk ─{}─ Esc to go back ", - "─".repeat(repeat_num as usize) - ); - result_title + "─".repeat( + usize::from(draw_loc.width).saturating_sub(TITLE_BASE.chars().count() + 2) + ) + ) } else if app_state.app_config_fields.use_basic_mode { String::new() } else { diff --git a/src/canvas/widgets/mem_basic.rs b/src/canvas/widgets/mem_basic.rs index 132fd91d..7c1b749a 100644 --- a/src/canvas/widgets/mem_basic.rs +++ b/src/canvas/widgets/mem_basic.rs @@ -1,5 +1,3 @@ -use std::cmp::max; - use crate::{ app::App, canvas::{drawing_utils::*, Painter}, @@ -41,7 +39,7 @@ impl MemBasicWidget for Painter { } // +9 due to 3 + 4 + 2 + 2 columns for the name & space + percentage + bar bounds + margin spacing - let bar_length = max(0, draw_loc.width as i64 - 11) as usize; + let bar_length = draw_loc.width.saturating_sub(11) as usize; let ram_use_percentage = if let Some(mem) = mem_data.last() { mem.1 } else { diff --git a/src/canvas/widgets/mem_graph.rs b/src/canvas/widgets/mem_graph.rs index 697d9688..7224484f 100644 --- a/src/canvas/widgets/mem_graph.rs +++ b/src/canvas/widgets/mem_graph.rs @@ -1,5 +1,3 @@ -use std::cmp::max; - use crate::{app::App, canvas::Painter, constants::*}; use tui::{ @@ -93,16 +91,12 @@ impl MemGraphWidget for Painter { let title = if app_state.is_expanded { const TITLE_BASE: &str = " Memory ── Esc to go back "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - let result_title = format!( + format!( " Memory ─{}─ Esc to go back ", - "─".repeat(repeat_num as usize) - ); - - result_title + "─".repeat( + usize::from(draw_loc.width).saturating_sub(TITLE_BASE.chars().count() + 2) + ) + ) } else { " Memory ".to_string() }; diff --git a/src/canvas/widgets/network_graph.rs b/src/canvas/widgets/network_graph.rs index d4efb3ba..2cc0307f 100644 --- a/src/canvas/widgets/network_graph.rs +++ b/src/canvas/widgets/network_graph.rs @@ -111,16 +111,12 @@ impl NetworkGraphWidget for Painter { let title = if app_state.is_expanded { const TITLE_BASE: &str = " Network ── Esc to go back "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - let result_title = format!( + format!( " Network ─{}─ Esc to go back ", - "─".repeat(repeat_num as usize) - ); - - result_title + "─".repeat( + usize::from(draw_loc.width).saturating_sub(TITLE_BASE.chars().count() + 2) + ) + ) } else { " Network ".to_string() }; diff --git a/src/canvas/widgets/process_table.rs b/src/canvas/widgets/process_table.rs index 2b1e728f..0be27f71 100644 --- a/src/canvas/widgets/process_table.rs +++ b/src/canvas/widgets/process_table.rs @@ -1,5 +1,3 @@ -use std::cmp::max; - use crate::{ app::{self, App}, canvas::{ @@ -83,11 +81,10 @@ impl ProcessTableWidget for Painter { // hit the process we've currently scrolled to. // We also need to move the list - we can // do so by hiding some elements! - let num_rows = max(0, i64::from(draw_loc.height) - self.table_height_offset) as u64; let is_on_widget = widget_id == app_state.current_widget.widget_id; let position = get_start_position( - num_rows, + draw_loc.height.saturating_sub(self.table_height_offset) as u64, &proc_widget_state.scroll_state.scroll_direction, &mut proc_widget_state.scroll_state.previous_scroll_position, proc_widget_state.scroll_state.current_scroll_position, @@ -96,7 +93,7 @@ impl ProcessTableWidget for Painter { // Sanity check let start_position = if position >= process_data.len() as u64 { - std::cmp::max(0, process_data.len() as i64 - 1) as u64 + process_data.len().saturating_sub(1) as u64 } else { position }; @@ -195,16 +192,13 @@ impl ProcessTableWidget for Painter { .is_enabled { const TITLE_BASE: &str = " Processes ── Esc to go back "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - let result_title = format!( + format!( " Processes ─{}─ Esc to go back ", - "─".repeat(repeat_num as usize) - ); - - result_title + "─".repeat( + usize::from(draw_loc.width) + .saturating_sub(TITLE_BASE.chars().count() + 2) + ) + ) } else { " Processes ".to_string() } @@ -439,11 +433,9 @@ impl ProcessTableWidget for Painter { let title = if draw_border { const TITLE_BASE: &str = " Esc to close "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - format!("{} Esc to close ", "─".repeat(repeat_num as usize)) + let repeat_num = + usize::from(draw_loc.width).saturating_sub(TITLE_BASE.chars().count() + 2); + format!("{} Esc to close ", "─".repeat(repeat_num)) } else { String::new() }; diff --git a/src/canvas/widgets/temp_table.rs b/src/canvas/widgets/temp_table.rs index 32b35733..69085fca 100644 --- a/src/canvas/widgets/temp_table.rs +++ b/src/canvas/widgets/temp_table.rs @@ -40,9 +40,8 @@ impl TempTableWidget for Painter { if let Some(temp_widget_state) = app_state.temp_state.widget_states.get_mut(&widget_id) { let temp_sensor_data: &mut [Vec] = &mut app_state.canvas_data.temp_sensor_data; - let num_rows = max(0, i64::from(draw_loc.height) - self.table_height_offset) as u64; let start_position = get_start_position( - num_rows, + draw_loc.height.saturating_sub(self.table_height_offset) as u64, &temp_widget_state.scroll_state.scroll_direction, &mut temp_widget_state.scroll_state.previous_scroll_position, temp_widget_state.scroll_state.current_scroll_position, @@ -66,16 +65,12 @@ impl TempTableWidget for Painter { let title = if app_state.is_expanded { const TITLE_BASE: &str = " Temperatures ── Esc to go back "; - let repeat_num = max( - 0, - draw_loc.width as i32 - TITLE_BASE.chars().count() as i32 - 2, - ); - let result_title = format!( + format!( " Temperatures ─{}─ Esc to go back ", - "─".repeat(repeat_num as usize) - ); - - result_title + "─".repeat( + usize::from(draw_loc.width).saturating_sub(TITLE_BASE.chars().count() + 2) + ) + ) } else if app_state.app_config_fields.use_basic_mode { String::new() } else {