clippy

src/ntfs/read.rs

@@ -295,7 +295,7 @@ where
             |v| v != 0,
         );
 
-        let precision = stop_time.precision.or_else(|| {
+        let precision = stop_time.precision.or({
             if datetime_estimated {
                 Some(StopTimePrecision::Estimated)
             } else {

src/ntfs/write.rs

@@ -267,7 +267,7 @@ fn build_price_v1(id: &str, ticket: &Ticket, price: &TicketPrice) -> Result<Pric
         .round_dp(0)
         .to_u32()
         .ok_or_else(|| anyhow!("Cannot convert price {:?} into a u32", cents_price))?;
-    let comment = ticket.comment.clone().unwrap_or_else(String::new);
+    let comment = ticket.comment.clone().unwrap_or_default();
     let price_v1 = PriceV1 {
         id: id.to_string(),
         start_date: price.ticket_validity_start,

src/objects.rs

@@ -796,7 +796,7 @@ impl FromStr for Time {
         };
         let hours: u32 = hours.parse()?;
         let minutes: u32 = minutes.parse()?;
-        let last_second_number = seconds.find('.').unwrap_or_else(|| seconds.len());
+        let last_second_number = seconds.find('.').unwrap_or(seconds.len());
         let seconds: u32 = seconds[..last_second_number].parse()?;
         if minutes > 59 || seconds > 59 {
             return Err(TimeError::WrongValue);

src/serde_utils.rs

@@ -130,7 +130,7 @@ where
 {
     use serde::Deserialize;
     let option = Option::<String>::deserialize(de)?;
-    Ok(option.map(|s| s.replace("/", "")))
+    Ok(option.map(|s| s.replace('/', "")))
 }
 
 /// deserialize type
@@ -267,7 +267,7 @@ where
     use serde::ser::Error;
     let currency_code = iso4217::alpha3(currency_code)
         .ok_or_else(|| S::Error::custom("The String is not a valid currency code (ISO-4217)"))?;
-    serializer.serialize_str(&currency_code.alpha3.to_string())
+    serializer.serialize_str(currency_code.alpha3)
 }
 
 /// serialize geometry to wkt

src/vptranslator.rs

@@ -324,7 +324,7 @@ mod tests {
     fn empty_vp() {
         let res = translate(&get_dates_from_bitset(
             Date::from_ymd(2012, 7, 16),
-            &"0000000".to_string(),
+            "0000000",
         ));
 
         assert_eq!(0b000_0000, get_week_from_weekday(res.operating_days));