about summary refs log tree commit diff
diff options
context:
space:
mode:
authorAmos Wenger <amoswenger@gmail.com>2022-07-21 13:37:41 +0200
committerAmos Wenger <amoswenger@gmail.com>2022-07-21 13:37:41 +0200
commitbbaf4daca02991f9b6ee90051248034a461bf91a (patch)
treeb7c56359152cd67684d1160cede516935c9bbdb3
parentfdddd832240efc0f1966c90aab75ad24a5d12fac (diff)
downloadrust-bbaf4daca02991f9b6ee90051248034a461bf91a.tar.gz
rust-bbaf4daca02991f9b6ee90051248034a461bf91a.zip
Pass tidy checks
-rw-r--r--crates/proc-macro-srv/build.rs6
-rw-r--r--crates/proc-macro-srv/src/abis/abi_sysroot/ra_server.rs7
-rw-r--r--crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/symbol.rs2
-rw-r--r--crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/token_stream.rs2
4 files changed, 11 insertions, 6 deletions
diff --git a/crates/proc-macro-srv/build.rs b/crates/proc-macro-srv/build.rs
index e3e4b9d87ba..a8c732f3154 100644
--- a/crates/proc-macro-srv/build.rs
+++ b/crates/proc-macro-srv/build.rs
@@ -1,9 +1,9 @@
+//! Determine rustc version `proc-macro-srv` (and thus the sysroot ABI) is
+//! build with and make it accessible at runtime for ABI selection.
+
 use std::{env, fs::File, io::Write, path::PathBuf, process::Command};
 
 fn main() {
-    // Determine rustc version `proc-macro-srv` (and thus the sysroot ABI) is
-    // build with and make it accessible at runtime for ABI selection.
-
     let mut path = PathBuf::from(env::var_os("OUT_DIR").unwrap());
     path.push("rustc_version.rs");
     let mut f = File::create(&path).unwrap();
diff --git a/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server.rs b/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server.rs
index d46f8dc88ad..69f1e13fb06 100644
--- a/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server.rs
+++ b/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server.rs
@@ -82,7 +82,7 @@ impl server::FreeFunctions for RustAnalyzer {
         &mut self,
         s: &str,
     ) -> Result<bridge::Literal<Self::Span, Self::Symbol>, ()> {
-        // TODO: keep track of LitKind and Suffix
+        // FIXME: keep track of LitKind and Suffix
         let symbol = SYMBOL_INTERNER.lock().unwrap().intern(s);
         Ok(bridge::Literal {
             kind: bridge::LitKind::Err,
@@ -391,8 +391,9 @@ impl server::MultiSpan for RustAnalyzer {
 }
 
 impl server::Symbol for RustAnalyzer {
-    fn normalize_and_validate_ident(&mut self, _string: &str) -> Result<Self::Symbol, ()> {
-        todo!()
+    fn normalize_and_validate_ident(&mut self, string: &str) -> Result<Self::Symbol, ()> {
+        // FIXME: nfc-normalize and validate idents
+        Ok(<Self as server::Server>::intern_symbol(string))
     }
 }
 
diff --git a/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/symbol.rs b/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/symbol.rs
index 5bfc62a301b..d16c2d22440 100644
--- a/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/symbol.rs
+++ b/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/symbol.rs
@@ -1,3 +1,5 @@
+//! Symbol interner for proc-macro-srv
+
 use once_cell::sync::Lazy;
 use std::{collections::HashMap, sync::Mutex};
 use tt::SmolStr;
diff --git a/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/token_stream.rs b/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/token_stream.rs
index c8bf2ecdd3d..113bb52c1af 100644
--- a/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/token_stream.rs
+++ b/crates/proc-macro-srv/src/abis/abi_sysroot/ra_server/token_stream.rs
@@ -1,3 +1,5 @@
+//! TokenStream implementation used by sysroot ABI
+
 use tt::TokenTree;
 
 #[derive(Debug, Default, Clone)]