From e7cae3d12f9a9fda8ee14e497380f5209ce09409 Mon Sep 17 00:00:00 2001 From: Andrew Phillips Date: Tue, 2 Sep 2025 12:40:28 -0300 Subject: [PATCH] refactor: Simplify filter plugin return values and apply io::copy directly Co-authored-by: aider (openai/andrew/openrouter/deepseek/deepseek-chat-v3.1) --- src/filter_plugin/skip.rs | 3 +-- src/filter_plugin/strip_ansi.rs | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/src/filter_plugin/skip.rs b/src/filter_plugin/skip.rs index 052c65c..9bb897a 100644 --- a/src/filter_plugin/skip.rs +++ b/src/filter_plugin/skip.rs @@ -30,8 +30,7 @@ impl FilterPlugin for SkipBytesFilter { } // Copy the remaining data using io::copy for efficiency - std::io::copy(reader, writer)?; - Ok(()) + std::io::copy(reader, writer) } } diff --git a/src/filter_plugin/strip_ansi.rs b/src/filter_plugin/strip_ansi.rs index 500f577..0e70f15 100644 --- a/src/filter_plugin/strip_ansi.rs +++ b/src/filter_plugin/strip_ansi.rs @@ -14,7 +14,6 @@ impl FilterPlugin for StripAnsiFilter { fn filter(&mut self, reader: &mut R, writer: &mut W) -> Result<()> { let mut ansi_writer = Writer::new(writer); std::io::copy(reader, &mut ansi_writer)?; - ansi_writer.flush()?; - Ok(()) + ansi_writer.flush() } }