dir: cargo clippy --fix -- -W clippy::pedantic

Signed-off-by: Daniel Maslowski <[email protected]>

Author: orangecms

src/dir/gen2.rs

@@ -315,9 +315,9 @@ impl Directory {
     /// Calculate a new minimum possible offset for relocation to save space.
     ///
     /// As taken from `me_cleaner`:
-    /// The two bytes in the middle of addr_base are added to spi_base to
-    /// compute the final start of the LUT. However, addr_base is not
-    /// modifiable, so act only on spi_base.
+    /// The two bytes in the middle of `addr_base` are added to `spi_base` to
+    /// compute the final start of the LUT. However, `addr_base` is not
+    /// modifiable, so act only on `spi_base`.
     pub fn calc_new_offset(&self, min_offset: u32) -> Result<u32, String> {
         if let Some((offset, m)) = self.get_huffman_mod() {
             let b = (m.header.addr_base & 0x00ff_ffff) >> 8;
@@ -410,7 +410,7 @@ impl Directory {
                     //
                 } else {
                     *c -= offset_diff;
-                };
+                }
             }
             return Ok(());
         }
@@ -473,7 +473,7 @@ impl Removables for Directory {
                     let last_chunk = first_chunk + c;
                     info!("Huffman compressed {n} @ {o:08x} ({s} bytes)");
                     let a = if retention_list.contains(&n) {
-                        for o in &all_chunks[first_chunk..last_chunk + 1] {
+                        for o in &all_chunks[first_chunk..=last_chunk] {
                             if o.start != 0 {
                                 unremovable_chunks.push(o.clone());
                             }

src/dir/gen3.rs

@@ -41,7 +41,7 @@ impl CPDHeader {
         match std::str::from_utf8(&n) {
             // some names are shorter than 4 bytes and padded with 0x0
             Ok(n) => n.trim_end_matches('\0').to_string(),
-            Err(_) => format!("{:02x?}", n),
+            Err(_) => format!("{n:02x?}"),
         }
     }
 }
@@ -112,10 +112,7 @@ pub struct CodePartitionDirectory {
 }
 
 fn stringify_vec(v: Vec<u8>) -> String {
-    v.iter()
-        .map(|b| format!("{b:02x}"))
-        .collect::<Vec<String>>()
-        .join("")
+    v.iter().map(|b| format!("{b:02x}")).collect::<String>()
 }
 
 impl Display for CodePartitionDirectory {
@@ -135,7 +132,7 @@ impl Display for CodePartitionDirectory {
                 };
                 format!("{m}\n{kh}{me}")
             }
-            Err(e) => e.to_string(),
+            Err(e) => e.clone(),
         };
         let l3 = "  file name        offset    end       size      kind".to_string();
         write!(f, "{l1}\n{l2}\n{l3}\n")?;
@@ -187,7 +184,7 @@ impl CodePartitionDirectory {
             entries,
             offset,
             size,
-            name: name.to_string(),
+            name: name.clone(),
         };
 
         Ok(cpd)