refactor: derive Eq, Hash and Copy for View

To use as a key in a hashmap, a type must implement `Eq` and `Hash`.
Additionally, `Copy` is required as well to be used in the .entry()
method of Hashmap.

Derive `Eq`, `Copy` and `Hash` on View to use it as a key in Hashmaps.

Signed-off-by: Ivin Joel Abraham <[email protected]>
Signed-off-by: Ivin Joel Abraham <[email protected]>

Author: ivinjabraham

src/handler/details_actions.rs

@@ -64,7 +64,7 @@ pub fn handle_patchset_details<B: Backend>(
             model.popup = Some(popup);
         }
         KeyCode::Esc | KeyCode::Char('q') => {
-            let ps_da_clone = patchset_details_and_actions.last_screen.clone();
+            let ps_da_clone = patchset_details_and_actions.last_screen;
             model.set_current_screen(ps_da_clone);
             model.reset_details_actions();
         }

src/model/mod.rs

@@ -255,7 +255,7 @@ impl Model {
                     reviewed_by,
                     tested_by,
                     acked_by,
-                    last_screen: self.current_screen.clone(),
+                    last_screen: self.current_screen,
                     lore_api_client: self.lore_api_client.clone(),
                     patchset_path,
                 });

src/model/screens/mod.rs

@@ -4,7 +4,7 @@ pub mod edit_config;
 pub mod latest;
 pub mod mail_list;
 
-#[derive(Debug, Clone, PartialEq)]
+#[derive(Copy, Debug, Clone, PartialEq, Eq, Hash)]
 pub enum View {
     MailingListSelection,
     BookmarkedPatchsets,