Fix clippy suggestions

Author: sharkdp

numbat-cli/src/config.rs

@@ -34,17 +34,12 @@ pub enum ExchangeRateFetchingPolicy {
 
 #[derive(Serialize, Deserialize)]
 #[serde(rename_all = "kebab-case", default, deny_unknown_fields)]
+#[derive(Default)]
 pub struct ExchangeRateConfig {
     pub fetching_policy: ExchangeRateFetchingPolicy,
 }
 
-impl Default for ExchangeRateConfig {
-    fn default() -> Self {
-        Self {
-            fetching_policy: ExchangeRateFetchingPolicy::default(),
-        }
-    }
-}
+
 
 #[derive(Serialize, Deserialize)]
 #[serde(rename_all = "kebab-case", default, deny_unknown_fields)]

numbat-cli/src/main.rs

@@ -26,7 +26,7 @@ use rustyline::{
     Validator,
 };
 use rustyline::{EventHandler, Highlighter, KeyCode, KeyEvent, Modifiers};
-use toml;
+
 
 use std::io::IsTerminal;
 use std::path::PathBuf;

numbat/src/module_importer.rs

@@ -63,23 +63,22 @@ impl ModuleImporter for FileSystemImporter {
                 .follow_links(true)
                 .follow_root_links(false)
                 .into_iter()
+                .flatten()
             {
-                if let Ok(entry) = entry {
-                    let path = entry.path();
-                    if path.is_file() && path.extension() == Some(OsStr::new("nbt")) {
-                        if let Ok(relative_path) = path.strip_prefix(root_path) {
-                            let components: Vec<String> = relative_path
-                                .components()
-                                .map(|c| {
-                                    c.as_os_str()
-                                        .to_string_lossy()
-                                        .trim_end_matches(".nbt")
-                                        .to_string()
-                                })
-                                .collect();
-
-                            modules.push(ModulePath(components));
-                        }
+                let path = entry.path();
+                if path.is_file() && path.extension() == Some(OsStr::new("nbt")) {
+                    if let Ok(relative_path) = path.strip_prefix(root_path) {
+                        let components: Vec<String> = relative_path
+                            .components()
+                            .map(|c| {
+                                c.as_os_str()
+                                    .to_string_lossy()
+                                    .trim_end_matches(".nbt")
+                                    .to_string()
+                            })
+                            .collect();
+
+                        modules.push(ModulePath(components));
                     }
                 }
             }

numbat/src/parser.rs

@@ -1137,11 +1137,7 @@ impl<'a> Parser<'a> {
         } else if let Some(inner) = self.match_any(&[TokenKind::True, TokenKind::False]) {
             Ok(Expression::Boolean(
                 inner.span,
-                if matches!(inner.kind, TokenKind::True) {
-                    true
-                } else {
-                    false
-                },
+                matches!(inner.kind, TokenKind::True),
             ))
         } else if let Some(token) = self.match_exact(TokenKind::StringFixed) {
             Ok(Expression::String(

numbat/src/value.rs

@@ -63,7 +63,7 @@ impl PrettyPrint for Value {
             Value::DateTime(dt, offset) => {
                 let l: chrono::DateTime<chrono::FixedOffset> =
                     chrono::DateTime::from_naive_utc_and_offset(dt.naive_utc(), *offset);
-                crate::markup::string(format!("{}", l.to_rfc2822()))
+                crate::markup::string(l.to_rfc2822())
             }
         }
     }