From 8f3a15137155dc374e6b2822c9155c07d05d5e28 Mon Sep 17 00:00:00 2001
From: Jack Wills <32690432+mrjackwills@users.noreply.github.com>
Date: Sun, 6 Nov 2022 01:34:13 +0000
Subject: [PATCH] chore: aggressive linting, with rust v1.65.0

---
 src/app_data/container_state.rs | 19 ++++---------------
 src/app_data/mod.rs             | 10 +++++-----
 2 files changed, 9 insertions(+), 20 deletions(-)

diff --git a/src/app_data/container_state.rs b/src/app_data/container_state.rs
index 5f7ed01..69f1430 100644
--- a/src/app_data/container_state.rs
+++ b/src/app_data/container_state.rs
@@ -93,16 +93,11 @@ impl<T> StatefulList<T> {
 
     pub fn previous(&mut self) {
         if !self.items.is_empty() {
-            let i = match self.state.selected() {
-                Some(i) => {
-                    if i == 0 {
+            let i = self.state.selected().map_or(0, |i| if i == 0 {
                         0
                     } else {
                         i - 1
-                    }
-                }
-                None => 0,
-            };
+                    });
             self.state.select(Some(i));
         }
     }
@@ -403,18 +398,12 @@ impl ContainerItem {
 
     /// Find the max value in the cpu stats VecDeque
     fn max_cpu_stats(&self) -> CpuStats {
-        match self.cpu_stats.iter().max() {
-            Some(value) => *value,
-            None => CpuStats::default(),
-        }
+        self.cpu_stats.iter().max().map_or_else(CpuStats::default, |value| *value)
     }
 
     /// Find the max value in the mem stats VecDeque
     fn max_mem_stats(&self) -> ByteStats {
-        match self.mem_stats.iter().max() {
-            Some(value) => *value,
-            None => ByteStats::default(),
-        }
+        self.mem_stats.iter().max().map_or_else(ByteStats::default, |value| *value)
     }
 
     /// Convert cpu stats into a vec for the charts function
diff --git a/src/app_data/mod.rs b/src/app_data/mod.rs
index cf7887a..e429d1c 100644
--- a/src/app_data/mod.rs
+++ b/src/app_data/mod.rs
@@ -258,7 +258,7 @@ impl AppData {
     /// 2) "logs - container_name" when no logs found, again 32 chars max
     pub fn get_log_title(&self) -> String {
         self.get_selected_log_index()
-            .map_or("".to_owned(), |index| {
+            .map_or(String::new(), |index| {
                 let logs_len = self.containers.items[index].logs.get_state_title();
                 let mut name = self.containers.items[index].name.clone();
                 name.truncate(32);
@@ -454,8 +454,8 @@ impl AppData {
 
         for i in all_containers {
             if let Some(id) = i.id.as_ref() {
-                let name = i.names.as_mut().map_or("".to_owned(), |names| {
-                    names.first_mut().map_or("".to_owned(), |f| {
+                let name = i.names.as_mut().map_or(String::new(), |names| {
+                    names.first_mut().map_or(String::new(), |f| {
                         if f.starts_with('/') {
                             f.remove(0);
                         }
@@ -464,12 +464,12 @@ impl AppData {
                 });
 
                 let state = State::from(i.state.as_ref().map_or("dead".to_owned(), trim_owned));
-                let status = i.status.as_ref().map_or("".to_owned(), trim_owned);
+                let status = i.status.as_ref().map_or(String::new(), trim_owned);
 
                 let image = i
                     .image
                     .as_ref()
-                    .map_or("".to_owned(), std::clone::Clone::clone);
+                    .map_or(String::new(), std::clone::Clone::clone);
 
                 let id = ContainerId::from(id);
                 // If container info already in containers Vec, then just update details