meta.cfg.clone(),
meta.cfg,
meta.env,
- Default::default(),
+ Ok(Vec::new()),
false,
origin,
);
default_cfg.clone(),
default_cfg,
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Vec::new(),
+ Ok(Vec::new()),
false,
CrateOrigin::Lang(LangCrateOrigin::Core),
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- proc_macro,
+ Ok(proc_macro),
true,
CrateOrigin::CratesIo { repo: None },
);
System(String),
}
+pub type ProcMacroLoadResult = Result<Vec<ProcMacro>, String>;
+
#[derive(Debug, Clone)]
pub struct ProcMacro {
pub name: SmolStr,
pub potential_cfg_options: CfgOptions,
pub env: Env,
pub dependencies: Vec<Dependency>,
- pub proc_macro: Vec<ProcMacro>,
+ pub proc_macro: ProcMacroLoadResult,
pub origin: CrateOrigin,
pub is_proc_macro: bool,
}
impl CrateGraph {
pub fn add_crate_root(
&mut self,
- file_id: FileId,
+ root_file_id: FileId,
edition: Edition,
display_name: Option<CrateDisplayName>,
version: Option<String>,
cfg_options: CfgOptions,
potential_cfg_options: CfgOptions,
env: Env,
- proc_macro: Vec<ProcMacro>,
+ proc_macro: ProcMacroLoadResult,
is_proc_macro: bool,
origin: CrateOrigin,
) -> CrateId {
let data = CrateData {
- root_file_id: file_id,
+ root_file_id,
edition,
version,
display_name,
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
CfgOptions::default(),
CfgOptions::default(),
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
input::{
CrateData, CrateDisplayName, CrateGraph, CrateId, CrateName, CrateOrigin, Dependency,
Edition, Env, LangCrateOrigin, ProcMacro, ProcMacroExpander, ProcMacroExpansionError,
- ProcMacroId, ProcMacroKind, SourceRoot, SourceRootId,
+ ProcMacroId, ProcMacroKind, ProcMacroLoadResult, SourceRoot, SourceRootId,
},
};
pub use salsa::{self, Cancelled};
}
let cfg_options = &krate.cfg_options;
- let proc_macros = krate
- .proc_macro
- .iter()
- .enumerate()
- .map(|(idx, it)| {
- // FIXME: a hacky way to create a Name from string.
- let name = tt::Ident { text: it.name.clone(), id: tt::TokenId::unspecified() };
+ let (proc_macros, proc_macro_err) = match &krate.proc_macro {
+ Ok(proc_macros) => {
(
- name.as_name(),
- ProcMacroExpander::new(def_map.krate, base_db::ProcMacroId(idx as u32)),
+ proc_macros
+ .iter()
+ .enumerate()
+ .map(|(idx, it)| {
+ // FIXME: a hacky way to create a Name from string.
+ let name =
+ tt::Ident { text: it.name.clone(), id: tt::TokenId::unspecified() };
+ (
+ name.as_name(),
+ ProcMacroExpander::new(def_map.krate, base_db::ProcMacroId(idx as u32)),
+ )
+ })
+ .collect(),
+ None,
)
- })
- .collect();
+ }
+ Err(e) => (Vec::new(), Some(e.clone())),
+ };
let is_proc_macro = krate.is_proc_macro;
let mut collector = DefCollector {
mod_dirs: FxHashMap::default(),
cfg_options,
proc_macros,
+ proc_macro_err,
from_glob_import: Default::default(),
skip_attrs: Default::default(),
derive_helpers_in_scope: Default::default(),
/// empty when proc. macro support is disabled (in which case we still do name resolution for
/// them).
proc_macros: Vec<(Name, ProcMacroExpander)>,
+ proc_macro_err: Option<String>,
is_proc_macro: bool,
from_glob_import: PerNsGlobImports,
/// If we fail to resolve an attribute on a `ModItem`, we fall back to ignoring the attribute.
self.def_map.diagnostics.push(DefDiagnostic::unresolved_proc_macro(
directive.module_id,
loc.kind,
+ self.proc_macro_err.clone(),
));
return recollect_without(self);
let diag = match err {
hir_expand::ExpandError::UnresolvedProcMacro => {
// Missing proc macros are non-fatal, so they are handled specially.
- DefDiagnostic::unresolved_proc_macro(module_id, loc.kind.clone())
+ DefDiagnostic::unresolved_proc_macro(
+ module_id,
+ loc.kind.clone(),
+ self.proc_macro_err.clone(),
+ )
}
_ => DefDiagnostic::macro_error(module_id, loc.kind.clone(), err.to_string()),
};
mod_dirs: FxHashMap::default(),
cfg_options: &CfgOptions::default(),
proc_macros: Default::default(),
+ proc_macro_err: None,
from_glob_import: Default::default(),
skip_attrs: Default::default(),
derive_helpers_in_scope: Default::default(),
UnconfiguredCode { ast: AstId<ast::Item>, cfg: CfgExpr, opts: CfgOptions },
- UnresolvedProcMacro { ast: MacroCallKind },
+ UnresolvedProcMacro { ast: MacroCallKind, proc_macro_err: Option<String> },
UnresolvedMacroCall { ast: MacroCallKind, path: ModPath },
Self { in_module: container, kind: DefDiagnosticKind::UnconfiguredCode { ast, cfg, opts } }
}
- pub(super) fn unresolved_proc_macro(container: LocalModuleId, ast: MacroCallKind) -> Self {
- Self { in_module: container, kind: DefDiagnosticKind::UnresolvedProcMacro { ast } }
+ pub(super) fn unresolved_proc_macro(
+ container: LocalModuleId,
+ ast: MacroCallKind,
+ proc_macro_err: Option<String>,
+ ) -> Self {
+ Self {
+ in_module: container,
+ kind: DefDiagnosticKind::UnresolvedProcMacro { ast, proc_macro_err },
+ }
}
pub(super) fn macro_error(
match self.proc_macro_id {
Some(id) => {
let krate_graph = db.crate_graph();
- let proc_macro = match krate_graph[self.krate].proc_macro.get(id.0 as usize) {
+ let proc_macros = match &krate_graph[self.krate].proc_macro {
+ Ok(proc_macros) => proc_macros,
+ Err(e) => {
+ return ExpandResult::only_err(ExpandError::Other(
+ e.clone().into_boxed_str(),
+ ))
+ }
+ };
+ let proc_macro = match proc_macros.get(id.0 as usize) {
Some(proc_macro) => proc_macro,
None => {
return ExpandResult::only_err(ExpandError::Other(
pub precise_location: Option<TextRange>,
pub macro_name: Option<String>,
pub kind: MacroKind,
+ pub proc_macro_err: Option<String>,
}
#[derive(Debug, Clone, Eq, PartialEq)]
);
}
- DefDiagnosticKind::UnresolvedProcMacro { ast } => {
+ DefDiagnosticKind::UnresolvedProcMacro { ast, proc_macro_err } => {
let (node, precise_location, macro_name, kind) = match ast {
MacroCallKind::FnLike { ast_id, .. } => {
let node = ast_id.to_node(db.upcast());
)
}
};
- acc.push(UnresolvedProcMacro { node, precise_location, macro_name, kind }.into());
+ acc.push(
+ UnresolvedProcMacro {
+ node,
+ precise_location,
+ macro_name,
+ kind,
+ proc_macro_err: proc_macro_err.clone(),
+ }
+ .into(),
+ );
}
DefDiagnosticKind::UnresolvedMacroCall { ast, path } => {
precise_location: None,
macro_name: None,
kind: MacroKind::ProcMacro,
+ proc_macro_err: None,
}
.into(),
),
Some(name) => format!("proc macro `{}` not expanded", name),
None => "proc macro not expanded".to_string(),
};
- let (message, severity) = if config_enabled {
- (message, Severity::Error)
- } else {
- let message = match d.kind {
- hir::MacroKind::Attr if proc_macros_enabled => {
- format!("{message}{}", " (attribute macro expansion is disabled)")
+ let severity = if config_enabled { Severity::Error } else { Severity::WeakWarning };
+ let message = format!(
+ "{message}: {}",
+ if config_enabled {
+ match &d.proc_macro_err {
+ Some(e) => e,
+ None => "proc macro not found",
}
- _ => {
- format!("{message}{}", " (proc-macro expansion is disabled)")
+ } else {
+ match d.kind {
+ hir::MacroKind::Attr if proc_macros_enabled => {
+ "attribute macro expansion is disabled"
+ }
+ _ => "proc-macro expansion is disabled",
}
- };
- (message, Severity::WeakWarning)
- };
+ },
+ );
Diagnostic::new("unresolved-proc-macro", message, display_range).severity(severity)
}
cfg_options.clone(),
cfg_options,
Env::default(),
- Default::default(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
Ok(ProcMacroServer { process: Arc::new(Mutex::new(process)) })
}
- pub fn load_dylib(
- &self,
- dylib: MacroDylib,
- ) -> Result<Result<Vec<ProcMacro>, String>, ServerError> {
+ pub fn load_dylib(&self, dylib: MacroDylib) -> Result<Vec<ProcMacro>, ServerError> {
let _p = profile::span("ProcMacroClient::by_dylib_path");
let macros =
self.process.lock().unwrap_or_else(|e| e.into_inner()).find_proc_macros(&dylib.path)?;
- let res = macros.map(|macros| {
- macros
+ match macros {
+ Ok(macros) => Ok(macros
.into_iter()
.map(|(name, kind)| ProcMacro {
process: self.process.clone(),
kind,
dylib_path: dylib.path.clone(),
})
- .collect()
- });
-
- Ok(res)
+ .collect()),
+ Err(message) => Err(ServerError { message, io: None }),
+ }
}
}
}
fn to_crate_graph(project_workspace: ProjectWorkspace) -> CrateGraph {
- project_workspace.to_crate_graph(&mut |_, _| Vec::new(), &mut {
+ project_workspace.to_crate_graph(&mut |_, _| Ok(Vec::new()), &mut {
let mut counter = 0;
move |_path| {
counter += 1;
use anyhow::{format_err, Context, Result};
use base_db::{
CrateDisplayName, CrateGraph, CrateId, CrateName, CrateOrigin, Dependency, Edition, Env,
- FileId, LangCrateOrigin, ProcMacro,
+ FileId, LangCrateOrigin, ProcMacroLoadResult,
};
use cfg::{CfgDiff, CfgOptions};
use paths::{AbsPath, AbsPathBuf};
pub fn to_crate_graph(
&self,
- load_proc_macro: &mut dyn FnMut(&str, &AbsPath) -> Vec<ProcMacro>,
+ load_proc_macro: &mut dyn FnMut(&str, &AbsPath) -> ProcMacroLoadResult,
load: &mut dyn FnMut(&AbsPath) -> Option<FileId>,
) -> CrateGraph {
let _p = profile::span("ProjectWorkspace::to_crate_graph");
fn project_json_to_crate_graph(
rustc_cfg: Vec<CfgFlag>,
- load_proc_macro: &mut dyn FnMut(&str, &AbsPath) -> Vec<ProcMacro>,
+ load_proc_macro: &mut dyn FnMut(&str, &AbsPath) -> ProcMacroLoadResult,
load: &mut dyn FnMut(&AbsPath) -> Option<FileId>,
project: &ProjectJson,
sysroot: &Option<Sysroot>,
})
.map(|(crate_id, krate, file_id)| {
let env = krate.env.clone().into_iter().collect();
- let proc_macro = krate.proc_macro_dylib_path.clone().map(|it| {
- load_proc_macro(
+ let proc_macro = match krate.proc_macro_dylib_path.clone() {
+ Some(it) => load_proc_macro(
krate.display_name.as_ref().map(|it| it.canonical_name()).unwrap_or(""),
&it,
- )
- });
+ ),
+ None => Ok(Vec::new()),
+ };
let target_cfgs = match krate.target.as_deref() {
Some(target) => {
cfg_options.clone(),
cfg_options,
env,
- proc_macro.unwrap_or_default(),
+ proc_macro,
krate.is_proc_macro,
if krate.display_name.is_some() {
CrateOrigin::CratesIo { repo: krate.repository.clone() }
fn cargo_to_crate_graph(
rustc_cfg: Vec<CfgFlag>,
override_cfg: &CfgOverrides,
- load_proc_macro: &mut dyn FnMut(&str, &AbsPath) -> Vec<ProcMacro>,
+ load_proc_macro: &mut dyn FnMut(&str, &AbsPath) -> ProcMacroLoadResult,
load: &mut dyn FnMut(&AbsPath) -> Option<FileId>,
cargo: &CargoWorkspace,
build_scripts: &WorkspaceBuildScripts,
cfg_options.clone(),
cfg_options.clone(),
Env::default(),
- Vec::new(),
+ Ok(Vec::new()),
false,
CrateOrigin::CratesIo { repo: None },
);
crate_graph: &mut CrateGraph,
cfg_options: &CfgOptions,
override_cfg: &CfgOverrides,
- load_proc_macro: &mut dyn FnMut(&str, &AbsPath) -> Vec<ProcMacro>,
+ load_proc_macro: &mut dyn FnMut(&str, &AbsPath) -> ProcMacroLoadResult,
pkg_to_lib_crate: &mut FxHashMap<la_arena::Idx<crate::PackageData>, CrateId>,
public_deps: &SysrootPublicDeps,
cargo: &CargoWorkspace,
pkg: &PackageData,
build_data: Option<&BuildScriptOutput>,
cfg_options: &CfgOptions,
- load_proc_macro: &mut dyn FnMut(&AbsPath) -> Vec<ProcMacro>,
+ load_proc_macro: &mut dyn FnMut(&AbsPath) -> ProcMacroLoadResult,
file_id: FileId,
cargo_name: &str,
is_proc_macro: bool,
}
}
- let proc_macro = build_data
- .as_ref()
- .and_then(|it| it.proc_macro_dylib_path.as_ref())
- .map(|it| load_proc_macro(it))
- .unwrap_or_default();
+ let proc_macro = match build_data.as_ref().and_then(|it| it.proc_macro_dylib_path.as_ref()) {
+ Some(it) => load_proc_macro(it),
+ None => Ok(Vec::new()),
+ };
let display_name = CrateDisplayName::from_canonical_name(cargo_name.to_string());
let mut potential_cfg_options = cfg_options.clone();
let file_id = load(&sysroot[krate].root)?;
let env = Env::default();
- let proc_macro = vec![];
let display_name = CrateDisplayName::from_canonical_name(sysroot[krate].name.clone());
let crate_id = crate_graph.add_crate_root(
file_id,
cfg_options.clone(),
cfg_options.clone(),
env,
- proc_macro,
+ Ok(Vec::new()),
false,
CrateOrigin::Lang(LangCrateOrigin::from(&*sysroot[krate].name)),
);
use ide::Change;
use ide_db::base_db::{
CrateGraph, Env, ProcMacro, ProcMacroExpander, ProcMacroExpansionError, ProcMacroKind,
- SourceRoot, VfsPath,
+ ProcMacroLoadResult, SourceRoot, VfsPath,
};
use proc_macro_api::{MacroDylib, ProcMacroServer};
use project_model::{ProjectWorkspace, WorkspaceBuildScripts};
/// Load the proc-macros for the given lib path, replacing all expanders whose names are in `dummy_replace`
/// with an identity dummy expander.
pub(crate) fn load_proc_macro(
- client: Option<&ProcMacroServer>,
+ server: Option<&ProcMacroServer>,
path: &AbsPath,
dummy_replace: &[Box<str>],
-) -> Vec<ProcMacro> {
- let dylib = match MacroDylib::new(path.to_path_buf()) {
- Ok(it) => it,
- Err(err) => {
- // FIXME: that's not really right -- we store this error in a
- // persistent status.
- tracing::warn!("failed to load proc macro: {}", err);
- return Vec::new();
+) -> ProcMacroLoadResult {
+ let res: Result<_, String> = (|| {
+ let dylib = MacroDylib::new(path.to_path_buf())
+ .map_err(|io| format!("Proc-macro dylib loading failed: {io}"))?;
+ Ok(if let Some(it) = server {
+ let vec = it.load_dylib(dylib).map_err(|e| format!("{e}"))?;
+ vec.into_iter()
+ .map(|expander| expander_to_proc_macro(expander, dummy_replace))
+ .collect()
+ } else {
+ Vec::new()
+ })
+ })();
+ return match res {
+ Ok(proc_macros) => {
+ tracing::info!(
+ "Loaded proc-macros for {}: {:?}",
+ path.display(),
+ proc_macros.iter().map(|it| it.name.clone()).collect::<Vec<_>>()
+ );
+ Ok(proc_macros)
+ }
+ Err(e) => {
+ tracing::warn!("proc-macro loading for {} failed: {e}", path.display());
+ Err(e)
}
};
- let proc_macros: Vec<_> = client
- .map(|it| it.load_dylib(dylib))
- .into_iter()
- .flat_map(|it| match it {
- Ok(Ok(macros)) => macros,
- Err(err) => {
- tracing::error!("proc macro server crashed: {}", err);
- Vec::new()
- }
- Ok(Err(err)) => {
- // FIXME: that's not really right -- we store this error in a
- // persistent status.
- tracing::warn!("failed to load proc macro: {}", err);
- Vec::new()
- }
- })
- .map(|expander| expander_to_proc_macro(expander, dummy_replace))
- .collect();
- tracing::info!(
- "Loaded proc-macros for {}: {:?}",
- path.display(),
- proc_macros.iter().map(|it| it.name.clone()).collect::<Vec<_>>()
- );
- return proc_macros;
-
fn expander_to_proc_macro(
expander: proc_macro_api::ProcMacro,
dummy_replace: &[Box<str>],