perf: don't read 7z file to mem when not necessary

src/commands/list.rs

@@ -14,7 +14,7 @@ use crate::{
     QuestionAction, QuestionPolicy, BUFFER_CAPACITY,
 };
 
-/// File at input_file_path is opened for reading, example: "archive.tar.gz"
+/// File at archive_path is opened for reading, example: "archive.tar.gz"
 /// formats contains each format necessary for decompression, example: [Gz, Tar] (in decompression order)
 pub fn list_archive_contents(
     archive_path: &Path,
@@ -25,7 +25,7 @@ pub fn list_archive_contents(
 ) -> crate::Result<()> {
     let reader = fs::File::open(archive_path)?;
 
-    // Zip archives are special, because they require io::Seek, so it requires it's logic separated
+    // Zip archives are special, because they require io::Seek, so it requires its logic separated
     // from decoder chaining.
     //
     // This is the only case where we can read and unpack it directly, without having to do
@@ -128,12 +128,14 @@ pub fn list_archive_contents(
                 if !user_wants_to_continue(archive_path, question_policy, QuestionAction::Decompression)? {
                     return Ok(());
                 }
-            }
-
-            let mut vec = vec![];
-            io::copy(&mut reader, &mut vec)?;
+                let mut vec = vec![];
+                io::copy(&mut reader, &mut vec)?;
 
-            Box::new(archive::sevenz::list_archive(io::Cursor::new(vec), password)?)
+                Box::new(archive::sevenz::list_archive(io::Cursor::new(vec), password)?)
+            } else {
+                // If it's the only format, we can read the archive directly.
+                Box::new(archive::sevenz::list_archive(fs::File::open(archive_path)?, password)?)
+            }
         }
         Gzip | Bzip | Bzip3 | Lz4 | Lzma | Xz | Lzip | Snappy | Zstd | Brotli => {
             unreachable!("Not an archive, should be validated before calling this function.");