fix: fixed by clippy

Author: Fatpandac

src/popups/branchlist.rs

@@ -588,7 +588,7 @@ impl BranchListPopup {
 			StatusType::WorkingDir,
 			None,
 		)
-		.unwrap();
+		.expect("Could not get status");
 
 		let selected_branch = &self.branches[self.selection as usize];
 		if status.is_empty() {
@@ -601,7 +601,7 @@ impl BranchListPopup {
 			} else {
 				checkout_remote_branch(
 					&self.repo.borrow(),
-					&selected_branch,
+					selected_branch,
 				)?;
 				self.local = true;
 				self.update_branches()?;
@@ -610,7 +610,7 @@ impl BranchListPopup {
 		} else {
 			self.queue.push(InternalEvent::CheckoutOption(
 				selected_branch.clone(),
-				self.local.clone(),
+				self.local,
 			));
 		}
 

src/popups/checkout_option.rs

@@ -61,7 +61,7 @@ impl CheckoutOptionPopup {
 				self.theme.text(true, false),
 			),
 			Span::styled(
-				self.branch.as_ref().unwrap().name.clone(),
+				self.branch.as_ref().expect("No branch").name.clone(),
 				self.theme.commit_hash(false),
 			),
 		]));
@@ -99,12 +99,12 @@ impl CheckoutOptionPopup {
 		if self.local {
 			checkout_branch(
 				&self.repo,
-				&self.branch.as_ref().unwrap().name,
-			)?
+				&self.branch.as_ref().expect("No branch").name,
+			)?;
 		} else {
 			checkout_remote_branch(
 				&self.repo,
-				&self.branch.as_ref().unwrap(),
+				self.branch.as_ref().expect("No branch"),
 			)?;
 		}
 

src/strings.rs

@@ -438,14 +438,14 @@ pub fn ellipsis_trim_start(s: &str, width: usize) -> Cow<str> {
 	}
 }
 
-#[derive(PartialEq, Clone, Copy)]
+#[derive(PartialEq, Eq, Clone, Copy)]
 pub enum CheckoutOptions {
 	StashAndReapply,
 	Unchange,
 	Discard,
 }
 
-pub fn checkout_option_to_string(
+pub const fn checkout_option_to_string(
 	kind: CheckoutOptions,
 ) -> (&'static str, &'static str) {
 	const CHECKOUT_OPTION_STASH_AND_REAPPLY: &str =