clippy fix

Author: gtfierro

cli/src/main.rs

@@ -9,7 +9,6 @@ use ontoenv::util::write_dataset_to_file;
 use ontoenv::ToUriString;
 use oxigraph::io::{JsonLdProfileSet, RdfFormat};
 use oxigraph::model::NamedNode;
-use serde_json;
 use std::collections::{BTreeMap, BTreeSet};
 use std::env::current_dir;
 use std::path::PathBuf;
@@ -617,7 +616,7 @@ fn main() -> Result<()> {
                 // Recompute status details similar to env.status()
                 let ontoenv_dir = current_dir()?.join(".ontoenv");
                 let last_updated = if ontoenv_dir.exists() {
-                    Some(std::fs::metadata(&ontoenv_dir)?.modified()?.into())
+                    Some(std::fs::metadata(&ontoenv_dir)?.modified()?)
                         as Option<std::time::SystemTime>
                 } else {
                     None
@@ -847,16 +846,14 @@ fn main() -> Result<()> {
                     }))
                     .collect();
                 println!("{}", serde_json::to_string_pretty(&out)?);
+            } else if problems.is_empty() {
+                println!("No issues found.");
             } else {
-                if problems.is_empty() {
-                    println!("No issues found.");
-                } else {
-                    println!("Found {} issues:", problems.len());
-                    for problem in problems {
-                        println!("- {}", problem.message);
-                        for location in problem.locations {
-                            println!("  - {location}");
-                        }
+                println!("Found {} issues:", problems.len());
+                for problem in problems {
+                    println!("- {}", problem.message);
+                    for location in problem.locations {
+                        println!("  - {location}");
                     }
                 }
             }

lib/src/io.rs

@@ -282,7 +282,7 @@ impl PersistentGraphIO {
         let mut writer = StreamingWriter::new(&self.store_path, opts);
 
         let mut iter = self.store.quads_for_pattern(None, None, None, None);
-        while let Some(q) = iter.next() {
+        for q in iter {
             let q = q?;
             // Dataset id: reuse graph name string; Graph name: same string
             let gname_str = match q.graph_name {

lib/src/options.rs

@@ -58,8 +58,10 @@ impl From<bool> for RefreshStrategy {
 
 /// Represents the cache usage policy captured in the configuration.
 #[derive(Debug, Copy, Clone, Eq, PartialEq)]
+#[derive(Default)]
 pub enum CacheMode {
     Enabled,
+    #[default]
     Disabled,
 }
 
@@ -69,11 +71,6 @@ impl CacheMode {
     }
 }
 
-impl Default for CacheMode {
-    fn default() -> Self {
-        CacheMode::Disabled
-    }
-}
 
 impl From<bool> for CacheMode {
     fn from(value: bool) -> Self {

rdf5d/src/bin/r5tu.rs

@@ -303,18 +303,7 @@ fn main() -> Result<(), Box<dyn std::error::Error>> {
         Commands::Stat(args) => {
             let file = args.file;
             let f = match {
-                #[cfg(feature = "mmap")]
-                {
-                    if args.no_mmap {
-                        R5tuFile::open(&file)
-                    } else {
-                        R5tuFile::open_mmap(&file)
-                    }
-                }
-                #[cfg(not(feature = "mmap"))]
-                {
-                    R5tuFile::open(&file)
-                }
+                R5tuFile::open(&file)
             } {
                 Ok(f) => f,
                 Err(e) => {