From c6033c831dfac958d2365501d164a390b511de8c Mon Sep 17 00:00:00 2001 From: Neil Date: Sat, 15 Mar 2025 12:31:56 -0700 Subject: [PATCH] refactor: Simplify by removing ? operator `out.flush()` already returns a `io::Result<()>`, so there is no need for `?` operator and `Ok(())` --- .../crates/proc-macro-api/src/legacy_protocol/json.rs | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/tools/rust-analyzer/crates/proc-macro-api/src/legacy_protocol/json.rs b/src/tools/rust-analyzer/crates/proc-macro-api/src/legacy_protocol/json.rs index ec89f6a9e65d..30e35f8f1434 100644 --- a/src/tools/rust-analyzer/crates/proc-macro-api/src/legacy_protocol/json.rs +++ b/src/tools/rust-analyzer/crates/proc-macro-api/src/legacy_protocol/json.rs @@ -30,6 +30,5 @@ pub fn write_json(out: &mut impl Write, msg: &str) -> io::Result<()> { tracing::debug!("> {}", msg); out.write_all(msg.as_bytes())?; out.write_all(b"\n")?; - out.flush()?; - Ok(()) + out.flush() }