clippy

src/analysis/event_consumer.rs

@@ -1251,10 +1251,7 @@ impl RefComponent for Cookware<ScalableValue> {
 }
 
 fn find_temperature<'a>(text: &'a str, re: &Regex) -> Option<(&'a str, Quantity<Value>, &'a str)> {
-    let Some(caps) = re.captures(text) else {
-        return None;
-    };
-
+    let caps = re.captures(text)?;
     let value = caps[1].replace(',', ".").parse::<f64>().ok()?;
     let unit = caps.get(3).unwrap().range();
     let unit_text = text[unit].to_string();

src/convert/mod.rs

@@ -281,7 +281,7 @@ pub struct Unit {
     pub ratio: f64,
     /// Difference offset to the conversion ratio
     pub difference: f64,
-    /// The [PhysicalQuantity] this unit belongs to
+    /// The [`PhysicalQuantity`] this unit belongs to
     pub physical_quantity: PhysicalQuantity,
     /// The unit [System] this unit belongs to, if any
     pub system: Option<System>,

src/ingredient_list.rs

@@ -245,7 +245,7 @@ impl IntoIterator for IngredientList {
 pub struct CategorizedIngredientList {
     /// One ingredient list per category
     ///
-    /// Because this is a [BTreeMap], the categories are sorted by name
+    /// Because this is a [`BTreeMap`], the categories are sorted by name
     pub categories: BTreeMap<String, IngredientList>,
     /// Ingredients with no category assigned
     pub other: IngredientList,

src/metadata.rs

@@ -450,7 +450,7 @@ impl NameAndUrl {
     fn new(name: Option<&str>, url: Option<Url>) -> Self {
         let name = name
             .map(|n| n.trim())
-            .filter(|n| n.len() > 0)
+            .filter(|n| !n.is_empty())
             .map(String::from);
         Self { name, url }
     }

src/quantity.rs

@@ -758,7 +758,11 @@ impl FractionLookupTable {
     const DENOMS: &'static [u8] = &[2, 3, 4, 8, 10, 16];
 
     pub fn new() -> Self {
-        debug_assert!(!Self::DENOMS.is_empty());
+        #[allow(clippy::const_is_empty)]
+        {
+            // I really want to be sure clippy
+            debug_assert!(!Self::DENOMS.is_empty());
+        }
         debug_assert!(Self::DENOMS.windows(2).all(|w| w[0] < w[1]));
         let mut table = Vec::new();