From d45637782139c74c25597e6ed7c18e7b2fc96b3c Mon Sep 17 00:00:00 2001 From: Michael Davis Date: Tue, 4 Feb 2025 08:52:09 -0500 Subject: [PATCH] minor: Remove double BufReader wrapper in DAP client `reader` is already a `BufReader` so there's no need to wrap it in another `BufReader`. This is a typo/mistake made possible by the type erasure (a `Box>>` is also a boxed reader). --- helix-dap/src/client.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helix-dap/src/client.rs b/helix-dap/src/client.rs index e0e4d56b5..6aa656e17 100644 --- a/helix-dap/src/client.rs +++ b/helix-dap/src/client.rs @@ -132,7 +132,7 @@ impl Client { let stderr = BufReader::new(process.stderr.take().expect("Failed to open stderr")); Self::streams( - Box::new(BufReader::new(reader)), + Box::new(reader), Box::new(writer), Some(Box::new(stderr)), id,