extension: Use hardcoded slices instead of Vecs when creating an `E…

…xtension`

src/commands.rs

@@ -128,7 +128,7 @@ pub fn run(args: Opts, question_policy: QuestionPolicy) -> crate::Result<()> {
                     } else if inp_ext
                         .compression_formats
                         .iter()
-                        .zip(&out_ext.compression_formats)
+                        .zip(out_ext.compression_formats.iter())
                         .all(|(inp, out)| inp == out)
                     {
                         let new_ext = Extension::new(
@@ -348,7 +348,7 @@ fn decompress_file(
     // in-memory decompression/copying first.
     //
     // Any other Zip decompression done can take up the whole RAM and freeze ouch.
-    if formats.len() == 1 && *formats[0].compression_formats.as_slice() == [Zip] {
+    if formats.len() == 1 && *formats[0].compression_formats == [Zip] {
         utils::create_dir_if_non_existent(output_dir)?;
         let zip_archive = zip::ZipArchive::new(reader)?;
         let _files = crate::archive::zip::unpack_archive(zip_archive, output_dir, question_policy)?;

src/extension.rs

@@ -8,16 +8,15 @@ use self::CompressionFormat::*;
 #[derive(Debug, Clone, PartialEq, Eq)]
 pub struct Extension {
     /// One extension like "tgz" can be made of multiple CompressionFormats ([Tar, Gz])
-    pub compression_formats: Vec<CompressionFormat>,
+    pub compression_formats: &'static [CompressionFormat],
     /// The input text for this extension, like "tgz", "tar" or "xz"
     pub display_text: String,
 }
 
 impl Extension {
     /// # Panics:
     ///   Will panic if `formats` is empty
-    pub fn new(formats: impl Into<Vec<CompressionFormat>>, text: impl Into<String>) -> Self {
-        let formats = formats.into();
+    pub fn new(formats: &'static [CompressionFormat], text: impl Into<String>) -> Self {
         assert!(!formats.is_empty());
         Self { compression_formats: formats, display_text: text.into() }
     }
@@ -111,18 +110,18 @@ pub fn separate_known_extensions_from_name(mut path: &Path) -> (&Path, Vec<Exten
     // While there is known extensions at the tail, grab them
     while let Some(extension) = path.extension().and_then(OsStr::to_str) {
         extensions.push(match extension {
-            "tar" => Extension::new([Tar], extension),
-            "tgz" => Extension::new([Tar, Gzip], extension),
-            "tbz" | "tbz2" => Extension::new([Tar, Bzip], extension),
-            "tlz4" => Extension::new([Tar, Lz4], extension),
-            "txz" | "tlz" | "tlzma" => Extension::new([Tar, Lzma], extension),
-            "tzst" => Extension::new([Tar, Zstd], ".tzst"),
-            "zip" => Extension::new([Zip], extension),
-            "bz" | "bz2" => Extension::new([Bzip], extension),
-            "gz" => Extension::new([Gzip], extension),
-            "lz4" => Extension::new([Lz4], extension),
-            "xz" | "lzma" | "lz" => Extension::new([Lzma], extension),
-            "zst" => Extension::new([Zstd], extension),
+            "tar" => Extension::new(&[Tar], extension),
+            "tgz" => Extension::new(&[Tar, Gzip], extension),
+            "tbz" | "tbz2" => Extension::new(&[Tar, Bzip], extension),
+            "tlz4" => Extension::new(&[Tar, Lz4], extension),
+            "txz" | "tlz" | "tlzma" => Extension::new(&[Tar, Lzma], extension),
+            "tzst" => Extension::new(&[Tar, Zstd], extension),
+            "zip" => Extension::new(&[Zip], extension),
+            "bz" | "bz2" => Extension::new(&[Bzip], extension),
+            "gz" => Extension::new(&[Gzip], extension),
+            "lz4" => Extension::new(&[Lz4], extension),
+            "xz" | "lzma" | "lz" => Extension::new(&[Lzma], extension),
+            "zst" => Extension::new(&[Zstd], extension),
             _ => break,
         });