Fixed some clippy warnings in libpaperback.

Author: trypsynth

lib/src/bridge.rs

@@ -1127,7 +1127,7 @@ fn session_get_status_info(session: &DocumentSession, position: i64) -> ffi::Ffi
 	}
 }
 
-fn session_stats(session: &DocumentSession) -> ffi::FfiDocumentStats {
+const fn session_stats(session: &DocumentSession) -> ffi::FfiDocumentStats {
 	document_stats_to_ffi(session.stats())
 }
 

lib/src/parser.rs

@@ -170,10 +170,10 @@ where
 		if ext.is_empty() {
 			continue;
 		}
-		if !first {
-			result.push(';');
-		} else {
+		if first {
 			first = false;
+		} else {
+			result.push(';');
 		}
 		result.push_str("*.");
 		result.push_str(ext);

lib/src/session.rs

@@ -89,19 +89,14 @@ pub struct LinkActivationResult {
 	pub url: String,
 }
 
-#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+#[derive(Debug, Clone, Copy, PartialEq, Eq, Default)]
 pub enum LinkAction {
 	Internal,
 	External,
+	#[default]
 	NotFound,
 }
 
-impl Default for LinkAction {
-	fn default() -> Self {
-		Self::NotFound
-	}
-}
-
 impl LinkActivationResult {
 	const fn not_found() -> Self {
 		Self { found: false, action: LinkAction::NotFound, offset: 0, url: String::new() }
@@ -657,8 +652,7 @@ impl DocumentSession {
 		}
 		self.handle
 			.get_marker_position_by_index(MarkerType::PageBreak, page_index)
-			.map(|offset| i64::try_from(offset).unwrap_or(-1))
-			.unwrap_or(-1)
+			.map_or(-1, |offset| i64::try_from(offset).unwrap_or(-1))
 	}
 
 	/// Returns the text between two positions (start inclusive, end exclusive).