From 8026742a562a794e93ac8b4da4e9248ed66ccc2e Mon Sep 17 00:00:00 2001 From: "Andrew Phillips (aider)" Date: Tue, 6 May 2025 11:50:32 -0300 Subject: [PATCH] refactor: remove redundant copy method from CompressionEngine implementations --- src/compression/gzip.rs | 6 ------ src/compression/lz4.rs | 6 ------ src/compression/none.rs | 6 ------ src/compression/program.rs | 6 ------ 4 files changed, 24 deletions(-) diff --git a/src/compression/gzip.rs b/src/compression/gzip.rs index e4f9016..395dc34 100644 --- a/src/compression/gzip.rs +++ b/src/compression/gzip.rs @@ -41,12 +41,6 @@ impl CompressionEngine for CompressionEngineGZip { Ok(Box::new(AutoFinishGzEncoder::new(gzip_write))) } - fn copy(&self, file_path: PathBuf, writer: &mut dyn Write) -> Result<()> { - let mut reader = self.open(file_path)?; - io::copy(&mut reader, writer)?; - writer.flush()?; - Ok(()) - } } pub struct AutoFinishGzEncoder { diff --git a/src/compression/lz4.rs b/src/compression/lz4.rs index bd1d2ce..f696ceb 100644 --- a/src/compression/lz4.rs +++ b/src/compression/lz4.rs @@ -40,11 +40,5 @@ impl CompressionEngine for CompressionEngineLZ4 { Ok(Box::new(lz4_write)) } - fn copy(&self, file_path: PathBuf, writer: &mut dyn Write) -> Result<()> { - let mut reader = self.open(file_path)?; - io::copy(&mut reader, writer)?; - writer.flush()?; - Ok(()) - } } diff --git a/src/compression/none.rs b/src/compression/none.rs index e0e01a8..359353b 100644 --- a/src/compression/none.rs +++ b/src/compression/none.rs @@ -32,10 +32,4 @@ impl CompressionEngine for CompressionEngineNone { Ok(Box::new(File::create(file_path)?)) } - fn copy(&self, file_path: PathBuf, writer: &mut dyn Write) -> Result<()> { - let mut reader = self.open(file_path)?; - io::copy(&mut reader, writer)?; - writer.flush()?; - Ok(()) - } } diff --git a/src/compression/program.rs b/src/compression/program.rs index 7bd2f6a..a23daa3 100644 --- a/src/compression/program.rs +++ b/src/compression/program.rs @@ -79,12 +79,6 @@ impl CompressionEngine for CompressionEngineProgram { Ok(Box::new(process.stdin.unwrap())) } - fn copy(&self, file_path: PathBuf, writer: &mut dyn Write) -> Result<()> { - let mut reader = self.open(file_path)?; - io::copy(&mut reader, writer)?; - writer.flush()?; - Ok(()) - } } fn get_program_path(program: &str) -> Result {