Unnamed repository; edit this file 'description' to name the repository.
rename handler's to be context specific
bit-aloo 4 months ago
parent 91b5c3c · commit 76ea982
-rw-r--r--crates/proc-macro-srv-cli/src/main_loop.rs6
-rw-r--r--crates/proc-macro-srv/src/dylib.rs6
-rw-r--r--crates/proc-macro-srv/src/dylib/proc_macros.rs4
-rw-r--r--crates/proc-macro-srv/src/lib.rs12
-rw-r--r--crates/proc-macro-srv/src/server_impl/rust_analyzer_span.rs4
-rw-r--r--crates/proc-macro-srv/src/server_impl/token_id.rs4
6 files changed, 18 insertions, 18 deletions
diff --git a/crates/proc-macro-srv-cli/src/main_loop.rs b/crates/proc-macro-srv-cli/src/main_loop.rs
index bd2fd2df79..d0f3a2b15d 100644
--- a/crates/proc-macro-srv-cli/src/main_loop.rs
+++ b/crates/proc-macro-srv-cli/src/main_loop.rs
@@ -169,12 +169,12 @@ fn handle_expand_id<W: std::io::Write, C: Codec>(
send_response::<_, C>(stdout, bidirectional::Response::ExpandMacro(res))
}
-struct BidirectionalProxy {
+struct ProcMacroClientHandle {
subreq_tx: mpsc::Sender<bidirectional::SubRequest>,
subresp_rx: mpsc::Receiver<bidirectional::SubResponse>,
}
-impl proc_macro_srv::BidirectionalHandler for BidirectionalProxy {
+impl proc_macro_srv::ProcMacroClientInterface for ProcMacroClientHandle {
fn source_text(&mut self, file_id: u32, start: u32, end: u32) -> Option<String> {
self.subreq_tx.send(bidirectional::SubRequest::SourceText { file_id, start, end }).ok()?;
@@ -228,7 +228,7 @@ fn handle_expand_ra<W: io::Write, R: io::BufRead, C: Codec>(
std::thread::scope(|s| {
s.spawn(|| {
- let callback = BidirectionalProxy { subreq_tx, subresp_rx };
+ let callback = ProcMacroClientHandle { subreq_tx, subresp_rx };
let res = srv
.expand(
diff --git a/crates/proc-macro-srv/src/dylib.rs b/crates/proc-macro-srv/src/dylib.rs
index 082a1d77b5..d34f37b16a 100644
--- a/crates/proc-macro-srv/src/dylib.rs
+++ b/crates/proc-macro-srv/src/dylib.rs
@@ -12,8 +12,8 @@ use object::Object;
use paths::{Utf8Path, Utf8PathBuf};
use crate::{
- PanicMessage, ProcMacroKind, ProcMacroSrvSpan, SubCallback, dylib::proc_macros::ProcMacros,
- token_stream::TokenStream,
+ PanicMessage, ProcMacroClientHandle, ProcMacroKind, ProcMacroSrvSpan,
+ dylib::proc_macros::ProcMacros, token_stream::TokenStream,
};
pub(crate) struct Expander {
@@ -45,7 +45,7 @@ impl Expander {
def_site: S,
call_site: S,
mixed_site: S,
- callback: Option<SubCallback>,
+ callback: Option<ProcMacroClientHandle>,
) -> Result<TokenStream<S>, PanicMessage>
where
<S::Server as bridge::server::Types>::TokenStream: Default,
diff --git a/crates/proc-macro-srv/src/dylib/proc_macros.rs b/crates/proc-macro-srv/src/dylib/proc_macros.rs
index 6f6bd086de..ddbb0128f5 100644
--- a/crates/proc-macro-srv/src/dylib/proc_macros.rs
+++ b/crates/proc-macro-srv/src/dylib/proc_macros.rs
@@ -1,5 +1,5 @@
//! Proc macro ABI
-use crate::{ProcMacroKind, ProcMacroSrvSpan, SubCallback, token_stream::TokenStream};
+use crate::{ProcMacroClientHandle, ProcMacroKind, ProcMacroSrvSpan, token_stream::TokenStream};
use proc_macro::bridge;
#[repr(transparent)]
@@ -20,7 +20,7 @@ impl ProcMacros {
def_site: S,
call_site: S,
mixed_site: S,
- callback: Option<SubCallback>,
+ callback: Option<ProcMacroClientHandle>,
) -> Result<TokenStream<S>, crate::PanicMessage> {
let parsed_attributes = attribute.unwrap_or_default();
diff --git a/crates/proc-macro-srv/src/lib.rs b/crates/proc-macro-srv/src/lib.rs
index 5f0273d7bc..9fb81afdee 100644
--- a/crates/proc-macro-srv/src/lib.rs
+++ b/crates/proc-macro-srv/src/lib.rs
@@ -91,9 +91,9 @@ impl<'env> ProcMacroSrv<'env> {
}
}
-pub type SubCallback = Box<dyn BidirectionalHandler + Send>;
+pub type ProcMacroClientHandle = Box<dyn ProcMacroClientInterface + Send>;
-pub trait BidirectionalHandler {
+pub trait ProcMacroClientInterface {
fn source_text(&mut self, file_id: u32, start: u32, end: u32) -> Option<String>;
}
@@ -111,7 +111,7 @@ impl ProcMacroSrv<'_> {
def_site: S,
call_site: S,
mixed_site: S,
- callback: Option<SubCallback>,
+ callback: Option<ProcMacroClientHandle>,
) -> Result<token_stream::TokenStream<S>, PanicMessage> {
let snapped_env = self.env;
let expander = self.expander(lib.as_ref()).map_err(|err| PanicMessage {
@@ -183,7 +183,7 @@ pub trait ProcMacroSrvSpan: Copy + Send + Sync {
call_site: Self,
def_site: Self,
mixed_site: Self,
- callback: Option<SubCallback>,
+ callback: Option<ProcMacroClientHandle>,
) -> Self::Server;
}
@@ -194,7 +194,7 @@ impl ProcMacroSrvSpan for SpanId {
call_site: Self,
def_site: Self,
mixed_site: Self,
- callback: Option<SubCallback>,
+ callback: Option<ProcMacroClientHandle>,
) -> Self::Server {
Self::Server {
call_site,
@@ -213,7 +213,7 @@ impl ProcMacroSrvSpan for Span {
call_site: Self,
def_site: Self,
mixed_site: Self,
- callback: Option<SubCallback>,
+ callback: Option<ProcMacroClientHandle>,
) -> Self::Server {
Self::Server {
call_site,
diff --git a/crates/proc-macro-srv/src/server_impl/rust_analyzer_span.rs b/crates/proc-macro-srv/src/server_impl/rust_analyzer_span.rs
index ee834e37c1..1b496950df 100644
--- a/crates/proc-macro-srv/src/server_impl/rust_analyzer_span.rs
+++ b/crates/proc-macro-srv/src/server_impl/rust_analyzer_span.rs
@@ -14,7 +14,7 @@ use proc_macro::bridge::server;
use span::{FIXUP_ERASED_FILE_AST_ID_MARKER, Span, TextRange, TextSize};
use crate::{
- SubCallback,
+ ProcMacroClientHandle,
bridge::{Diagnostic, ExpnGlobals, Literal, TokenTree},
server_impl::literal_from_str,
};
@@ -29,7 +29,7 @@ pub struct RaSpanServer {
pub call_site: Span,
pub def_site: Span,
pub mixed_site: Span,
- pub callback: Option<SubCallback>,
+ pub callback: Option<ProcMacroClientHandle>,
}
impl server::Types for RaSpanServer {
diff --git a/crates/proc-macro-srv/src/server_impl/token_id.rs b/crates/proc-macro-srv/src/server_impl/token_id.rs
index 9db7597d84..646dde7952 100644
--- a/crates/proc-macro-srv/src/server_impl/token_id.rs
+++ b/crates/proc-macro-srv/src/server_impl/token_id.rs
@@ -9,7 +9,7 @@ use intern::Symbol;
use proc_macro::bridge::server;
use crate::{
- SubCallback,
+ ProcMacroClientHandle,
bridge::{Diagnostic, ExpnGlobals, Literal, TokenTree},
server_impl::literal_from_str,
};
@@ -35,7 +35,7 @@ pub struct SpanIdServer {
pub call_site: Span,
pub def_site: Span,
pub mixed_site: Span,
- pub callback: Option<SubCallback>,
+ pub callback: Option<ProcMacroClientHandle>,
}
impl server::Types for SpanIdServer {