Merge #2349

2349: Ban println in lsp_server r=matklad a=matklad



Co-authored-by: Aleksey Kladov <[email protected]>

2 files changed, 23 insertions, 4 deletions

diff --git a/crates/ra_lsp_server/src/lib.rs b/crates/ra_lsp_server/src/lib.rs
index 0e5dbbbd5..9c36402b0 100644
--- a/crates/ra_lsp_server/src/lib.rs
+++ b/crates/ra_lsp_server/src/lib.rs
@@ -1,6 +1,26 @@
-//! FIXME: write short doc here
-
+//! Implementation of the LSP for rust-analyzer.
+//!
+//! This crate takes Rust-specific analysis results from ra_ide_api and
+//! translates into LSP types.
+//!
+//! It also is the root of all state. `world` module defines the bulk of the
+//! state, and `main_loop` module defines the rules for modifying it.
 #![recursion_limit = "512"]
+
+#[allow(unused)]
+macro_rules! println {
+    ($($tt:tt)*) => {
+        compile_error!("stdout is locked, use eprintln")
+    };
+}
+
+#[allow(unused)]
+macro_rules! print {
+    ($($tt:tt)*) => {
+        compile_error!("stdout is locked, use eprint")
+    };
+}
+
 mod caps;
 mod cargo_target_spec;
 mod conv;
diff --git a/crates/ra_lsp_server/src/main.rs b/crates/ra_lsp_server/src/main.rs
index 7d9a1d054..e13c8ca14 100644
--- a/crates/ra_lsp_server/src/main.rs
+++ b/crates/ra_lsp_server/src/main.rs
@@ -1,8 +1,7 @@
-//! FIXME: write short doc here
+//! `ra_lsp_server` binary
 
 use flexi_logger::{Duplicate, Logger};
 use lsp_server::Connection;
-
 use ra_lsp_server::{show_message, Result, ServerConfig};
 use ra_prof;