chore: remove unwanted comments, extra lines and putting cli srv on default

src/tools/rust-analyzer/.github/workflows/ci.yaml

@@ -70,7 +70,7 @@ jobs:
 
       - name: Test
         run: cargo test --features sysroot-abi -p proc-macro-srv -p proc-macro-srv-cli -p proc-macro-api -- --quiet
-      
+
       - name: Check salsa dependency
         run: "! (cargo tree -p proc-macro-srv-cli | grep -q salsa)"
 

src/tools/rust-analyzer/Cargo.lock

@@ -1856,7 +1856,6 @@ dependencies = [
 name = "proc-macro-srv"
 version = "0.0.0"
 dependencies = [
- "crossbeam-channel",
  "expect-test",
  "intern",
  "libc",

src/tools/rust-analyzer/crates/proc-macro-srv-cli/src/main.rs

@@ -31,7 +31,7 @@ fn main() -> std::io::Result<()> {
             clap::Arg::new("format")
                 .long("format")
                 .action(clap::ArgAction::Set)
-                .default_value("bidirectional-postcard-prototype")
+                .default_value("json-legacy")
                 .value_parser(clap::builder::EnumValueParser::<ProtocolFormat>::new()),
             clap::Arg::new("version")
                 .long("version")
@@ -71,7 +71,7 @@ impl ValueEnum for ProtocolFormat {
                 Some(clap::builder::PossibleValue::new("postcard-legacy"))
             }
             ProtocolFormat::BidirectionalPostcardPrototype => {
-                Some(clap::builder::PossibleValue::new("postcard-new"))
+                Some(clap::builder::PossibleValue::new("bidirectional-postcard-prototype"))
             }
         }
     }

src/tools/rust-analyzer/crates/proc-macro-srv/Cargo.toml

@@ -22,7 +22,6 @@ paths.workspace = true
 # span = {workspace = true, default-features = false} does not work
 span = { path = "../span", version = "0.0.0", default-features = false}
 intern.workspace = true
-crossbeam-channel.workspace = true
 
 ra-ap-rustc_lexer.workspace = true
 

src/tools/rust-analyzer/crates/proc-macro-srv/src/server_impl/token_id.rs

@@ -141,7 +141,6 @@ impl server::Span for SpanIdServer {
     /// See PR:
     /// https://github.com/rust-lang/rust/pull/55780
     fn source_text(&mut self, _span: Self::Span) -> Option<String> {
-        // FIXME requires db, needs special handling wrt fixup spans
         None
     }