diff options
Diffstat (limited to 'crates/ra_ide_api/src')
-rw-r--r-- | crates/ra_ide_api/src/call_info.rs | 9 | ||||
-rw-r--r-- | crates/ra_ide_api/src/expand_macro.rs | 23 | ||||
-rw-r--r-- | crates/ra_ide_api/src/references/classify.rs | 2 |
3 files changed, 27 insertions, 7 deletions
diff --git a/crates/ra_ide_api/src/call_info.rs b/crates/ra_ide_api/src/call_info.rs index d0283e410..d559dc4d0 100644 --- a/crates/ra_ide_api/src/call_info.rs +++ b/crates/ra_ide_api/src/call_info.rs | |||
@@ -18,12 +18,9 @@ pub(crate) fn call_info(db: &RootDatabase, position: FilePosition) -> Option<Cal | |||
18 | // Find the calling expression and it's NameRef | 18 | // Find the calling expression and it's NameRef |
19 | let calling_node = FnCallNode::with_node(&syntax, position.offset)?; | 19 | let calling_node = FnCallNode::with_node(&syntax, position.offset)?; |
20 | let name_ref = calling_node.name_ref()?; | 20 | let name_ref = calling_node.name_ref()?; |
21 | let name_ref = hir::Source::new(position.file_id.into(), name_ref.syntax()); | ||
21 | 22 | ||
22 | let analyzer = hir::SourceAnalyzer::new( | 23 | let analyzer = hir::SourceAnalyzer::new(db, name_ref, None); |
23 | db, | ||
24 | hir::Source::new(position.file_id.into(), name_ref.syntax()), | ||
25 | None, | ||
26 | ); | ||
27 | let (mut call_info, has_self) = match &calling_node { | 24 | let (mut call_info, has_self) = match &calling_node { |
28 | FnCallNode::CallExpr(expr) => { | 25 | FnCallNode::CallExpr(expr) => { |
29 | //FIXME: Type::as_callable is broken | 26 | //FIXME: Type::as_callable is broken |
@@ -44,7 +41,7 @@ pub(crate) fn call_info(db: &RootDatabase, position: FilePosition) -> Option<Cal | |||
44 | (CallInfo::with_fn(db, function), function.has_self_param(db)) | 41 | (CallInfo::with_fn(db, function), function.has_self_param(db)) |
45 | } | 42 | } |
46 | FnCallNode::MacroCallExpr(expr) => { | 43 | FnCallNode::MacroCallExpr(expr) => { |
47 | let macro_def = analyzer.resolve_macro_call(db, &expr)?; | 44 | let macro_def = analyzer.resolve_macro_call(db, name_ref.with_value(&expr))?; |
48 | (CallInfo::with_macro(db, macro_def)?, false) | 45 | (CallInfo::with_macro(db, macro_def)?, false) |
49 | } | 46 | } |
50 | }; | 47 | }; |
diff --git a/crates/ra_ide_api/src/expand_macro.rs b/crates/ra_ide_api/src/expand_macro.rs index 0b540b8cd..abc602244 100644 --- a/crates/ra_ide_api/src/expand_macro.rs +++ b/crates/ra_ide_api/src/expand_macro.rs | |||
@@ -269,4 +269,27 @@ fn some_thing() -> u32 { | |||
269 | assert_eq!(res.name, "foo"); | 269 | assert_eq!(res.name, "foo"); |
270 | assert_snapshot!(res.expansion, @r###"bar!()"###); | 270 | assert_snapshot!(res.expansion, @r###"bar!()"###); |
271 | } | 271 | } |
272 | |||
273 | #[test] | ||
274 | fn macro_expand_with_dollar_crate() { | ||
275 | let res = check_expand_macro( | ||
276 | r#" | ||
277 | //- /lib.rs | ||
278 | #[macro_export] | ||
279 | macro_rules! bar { | ||
280 | () => {0}; | ||
281 | } | ||
282 | macro_rules! foo { | ||
283 | () => {$crate::bar!()}; | ||
284 | } | ||
285 | |||
286 | fn main() { | ||
287 | let res = fo<|>o!(); | ||
288 | } | ||
289 | "#, | ||
290 | ); | ||
291 | |||
292 | assert_eq!(res.name, "foo"); | ||
293 | assert_snapshot!(res.expansion, @r###"0"###); | ||
294 | } | ||
272 | } | 295 | } |
diff --git a/crates/ra_ide_api/src/references/classify.rs b/crates/ra_ide_api/src/references/classify.rs index cab06dea9..227737ad2 100644 --- a/crates/ra_ide_api/src/references/classify.rs +++ b/crates/ra_ide_api/src/references/classify.rs | |||
@@ -152,7 +152,7 @@ pub(crate) fn classify_name_ref( | |||
152 | 152 | ||
153 | if let Some(macro_call) = parent.ancestors().find_map(ast::MacroCall::cast) { | 153 | if let Some(macro_call) = parent.ancestors().find_map(ast::MacroCall::cast) { |
154 | tested_by!(goto_definition_works_for_macros); | 154 | tested_by!(goto_definition_works_for_macros); |
155 | if let Some(macro_def) = analyzer.resolve_macro_call(db, ¯o_call) { | 155 | if let Some(macro_def) = analyzer.resolve_macro_call(db, name_ref.with_value(¯o_call)) { |
156 | let kind = NameKind::Macro(macro_def); | 156 | let kind = NameKind::Macro(macro_def); |
157 | return Some(NameDefinition { kind, container, visibility }); | 157 | return Some(NameDefinition { kind, container, visibility }); |
158 | } | 158 | } |