diff options
-rw-r--r-- | crates/ide/src/annotations.rs | 16 | ||||
-rw-r--r-- | crates/ide/src/file_structure.rs | 228 | ||||
-rw-r--r-- | crates/ide/src/lib.rs | 4 | ||||
-rw-r--r-- | crates/ide/src/references.rs | 26 | ||||
-rw-r--r-- | crates/ide/src/references/rename.rs | 3 | ||||
-rw-r--r-- | crates/ide_db/src/defs.rs | 2 | ||||
-rw-r--r-- | crates/ide_db/src/search.rs | 16 | ||||
-rw-r--r-- | crates/rust-analyzer/src/handlers.rs | 22 | ||||
-rw-r--r-- | crates/rust-analyzer/src/to_proto.rs | 10 |
9 files changed, 254 insertions, 73 deletions
diff --git a/crates/ide/src/annotations.rs b/crates/ide/src/annotations.rs index 2e8e82b70..8e0a8fd8d 100644 --- a/crates/ide/src/annotations.rs +++ b/crates/ide/src/annotations.rs | |||
@@ -11,7 +11,7 @@ use crate::{ | |||
11 | goto_implementation::goto_implementation, | 11 | goto_implementation::goto_implementation, |
12 | references::find_all_refs, | 12 | references::find_all_refs, |
13 | runnables::{runnables, Runnable}, | 13 | runnables::{runnables, Runnable}, |
14 | NavigationTarget, RunnableKind, | 14 | NavigationTarget, RunnableKind, StructureNodeKind, |
15 | }; | 15 | }; |
16 | 16 | ||
17 | // Feature: Annotations | 17 | // Feature: Annotations |
@@ -80,15 +80,17 @@ pub(crate) fn annotations( | |||
80 | .filter(|node| { | 80 | .filter(|node| { |
81 | matches!( | 81 | matches!( |
82 | node.kind, | 82 | node.kind, |
83 | SymbolKind::Trait | 83 | StructureNodeKind::SymbolKind(SymbolKind::Trait) |
84 | | SymbolKind::Struct | 84 | | StructureNodeKind::SymbolKind(SymbolKind::Struct) |
85 | | SymbolKind::Enum | 85 | | StructureNodeKind::SymbolKind(SymbolKind::Enum) |
86 | | SymbolKind::Union | 86 | | StructureNodeKind::SymbolKind(SymbolKind::Union) |
87 | | SymbolKind::Const | 87 | | StructureNodeKind::SymbolKind(SymbolKind::Const) |
88 | ) | 88 | ) |
89 | }) | 89 | }) |
90 | .for_each(|node| { | 90 | .for_each(|node| { |
91 | if config.annotate_impls && node.kind != SymbolKind::Const { | 91 | if config.annotate_impls |
92 | && node.kind != StructureNodeKind::SymbolKind(SymbolKind::Const) | ||
93 | { | ||
92 | annotations.push(Annotation { | 94 | annotations.push(Annotation { |
93 | range: node.node_range, | 95 | range: node.node_range, |
94 | kind: AnnotationKind::HasImpls { | 96 | kind: AnnotationKind::HasImpls { |
diff --git a/crates/ide/src/file_structure.rs b/crates/ide/src/file_structure.rs index 26793bdb4..9f879a66e 100644 --- a/crates/ide/src/file_structure.rs +++ b/crates/ide/src/file_structure.rs | |||
@@ -1,7 +1,8 @@ | |||
1 | use ide_db::SymbolKind; | 1 | use ide_db::SymbolKind; |
2 | use syntax::{ | 2 | use syntax::{ |
3 | ast::{self, AttrsOwner, GenericParamsOwner, NameOwner}, | 3 | ast::{self, AttrsOwner, GenericParamsOwner, NameOwner}, |
4 | match_ast, AstNode, SourceFile, SyntaxNode, TextRange, WalkEvent, | 4 | match_ast, AstNode, AstToken, NodeOrToken, SourceFile, SyntaxNode, SyntaxToken, TextRange, |
5 | WalkEvent, | ||
5 | }; | 6 | }; |
6 | 7 | ||
7 | #[derive(Debug, Clone)] | 8 | #[derive(Debug, Clone)] |
@@ -10,11 +11,17 @@ pub struct StructureNode { | |||
10 | pub label: String, | 11 | pub label: String, |
11 | pub navigation_range: TextRange, | 12 | pub navigation_range: TextRange, |
12 | pub node_range: TextRange, | 13 | pub node_range: TextRange, |
13 | pub kind: SymbolKind, | 14 | pub kind: StructureNodeKind, |
14 | pub detail: Option<String>, | 15 | pub detail: Option<String>, |
15 | pub deprecated: bool, | 16 | pub deprecated: bool, |
16 | } | 17 | } |
17 | 18 | ||
19 | #[derive(Debug, Clone, Copy, PartialEq, Eq, Hash, PartialOrd, Ord)] | ||
20 | pub enum StructureNodeKind { | ||
21 | SymbolKind(SymbolKind), | ||
22 | Region, | ||
23 | } | ||
24 | |||
18 | // Feature: File Structure | 25 | // Feature: File Structure |
19 | // | 26 | // |
20 | // Provides a tree of the symbols defined in the file. Can be used to | 27 | // Provides a tree of the symbols defined in the file. Can be used to |
@@ -32,34 +39,46 @@ pub(crate) fn file_structure(file: &SourceFile) -> Vec<StructureNode> { | |||
32 | let mut res = Vec::new(); | 39 | let mut res = Vec::new(); |
33 | let mut stack = Vec::new(); | 40 | let mut stack = Vec::new(); |
34 | 41 | ||
35 | for event in file.syntax().preorder() { | 42 | for event in file.syntax().preorder_with_tokens() { |
36 | match event { | 43 | match event { |
37 | WalkEvent::Enter(node) => { | 44 | WalkEvent::Enter(NodeOrToken::Node(node)) => { |
38 | if let Some(mut symbol) = structure_node(&node) { | 45 | if let Some(mut symbol) = structure_node(&node) { |
39 | symbol.parent = stack.last().copied(); | 46 | symbol.parent = stack.last().copied(); |
40 | stack.push(res.len()); | 47 | stack.push(res.len()); |
41 | res.push(symbol); | 48 | res.push(symbol); |
42 | } | 49 | } |
43 | } | 50 | } |
44 | WalkEvent::Leave(node) => { | 51 | WalkEvent::Leave(NodeOrToken::Node(node)) => { |
45 | if structure_node(&node).is_some() { | 52 | if structure_node(&node).is_some() { |
46 | stack.pop().unwrap(); | 53 | stack.pop().unwrap(); |
47 | } | 54 | } |
48 | } | 55 | } |
56 | WalkEvent::Enter(NodeOrToken::Token(token)) => { | ||
57 | if let Some(mut symbol) = structure_token(token) { | ||
58 | symbol.parent = stack.last().copied(); | ||
59 | stack.push(res.len()); | ||
60 | res.push(symbol); | ||
61 | } | ||
62 | } | ||
63 | WalkEvent::Leave(NodeOrToken::Token(token)) => { | ||
64 | if structure_token(token).is_some() { | ||
65 | stack.pop().unwrap(); | ||
66 | } | ||
67 | } | ||
49 | } | 68 | } |
50 | } | 69 | } |
51 | res | 70 | res |
52 | } | 71 | } |
53 | 72 | ||
54 | fn structure_node(node: &SyntaxNode) -> Option<StructureNode> { | 73 | fn structure_node(node: &SyntaxNode) -> Option<StructureNode> { |
55 | fn decl<N: NameOwner + AttrsOwner>(node: N, kind: SymbolKind) -> Option<StructureNode> { | 74 | fn decl<N: NameOwner + AttrsOwner>(node: N, kind: StructureNodeKind) -> Option<StructureNode> { |
56 | decl_with_detail(&node, None, kind) | 75 | decl_with_detail(&node, None, kind) |
57 | } | 76 | } |
58 | 77 | ||
59 | fn decl_with_type_ref<N: NameOwner + AttrsOwner>( | 78 | fn decl_with_type_ref<N: NameOwner + AttrsOwner>( |
60 | node: &N, | 79 | node: &N, |
61 | type_ref: Option<ast::Type>, | 80 | type_ref: Option<ast::Type>, |
62 | kind: SymbolKind, | 81 | kind: StructureNodeKind, |
63 | ) -> Option<StructureNode> { | 82 | ) -> Option<StructureNode> { |
64 | let detail = type_ref.map(|type_ref| { | 83 | let detail = type_ref.map(|type_ref| { |
65 | let mut detail = String::new(); | 84 | let mut detail = String::new(); |
@@ -72,7 +91,7 @@ fn structure_node(node: &SyntaxNode) -> Option<StructureNode> { | |||
72 | fn decl_with_detail<N: NameOwner + AttrsOwner>( | 91 | fn decl_with_detail<N: NameOwner + AttrsOwner>( |
73 | node: &N, | 92 | node: &N, |
74 | detail: Option<String>, | 93 | detail: Option<String>, |
75 | kind: SymbolKind, | 94 | kind: StructureNodeKind, |
76 | ) -> Option<StructureNode> { | 95 | ) -> Option<StructureNode> { |
77 | let name = node.name()?; | 96 | let name = node.name()?; |
78 | 97 | ||
@@ -120,18 +139,18 @@ fn structure_node(node: &SyntaxNode) -> Option<StructureNode> { | |||
120 | collapse_ws(ret_type.syntax(), &mut detail); | 139 | collapse_ws(ret_type.syntax(), &mut detail); |
121 | } | 140 | } |
122 | 141 | ||
123 | decl_with_detail(&it, Some(detail), SymbolKind::Function) | 142 | decl_with_detail(&it, Some(detail), StructureNodeKind::SymbolKind(SymbolKind::Function)) |
124 | }, | 143 | }, |
125 | ast::Struct(it) => decl(it, SymbolKind::Struct), | 144 | ast::Struct(it) => decl(it, StructureNodeKind::SymbolKind(SymbolKind::Struct)), |
126 | ast::Union(it) => decl(it, SymbolKind::Union), | 145 | ast::Union(it) => decl(it, StructureNodeKind::SymbolKind(SymbolKind::Union)), |
127 | ast::Enum(it) => decl(it, SymbolKind::Enum), | 146 | ast::Enum(it) => decl(it, StructureNodeKind::SymbolKind(SymbolKind::Enum)), |
128 | ast::Variant(it) => decl(it, SymbolKind::Variant), | 147 | ast::Variant(it) => decl(it, StructureNodeKind::SymbolKind(SymbolKind::Variant)), |
129 | ast::Trait(it) => decl(it, SymbolKind::Trait), | 148 | ast::Trait(it) => decl(it, StructureNodeKind::SymbolKind(SymbolKind::Trait)), |
130 | ast::Module(it) => decl(it, SymbolKind::Module), | 149 | ast::Module(it) => decl(it, StructureNodeKind::SymbolKind(SymbolKind::Module)), |
131 | ast::TypeAlias(it) => decl_with_type_ref(&it, it.ty(), SymbolKind::TypeAlias), | 150 | ast::TypeAlias(it) => decl_with_type_ref(&it, it.ty(), StructureNodeKind::SymbolKind(SymbolKind::TypeAlias)), |
132 | ast::RecordField(it) => decl_with_type_ref(&it, it.ty(), SymbolKind::Field), | 151 | ast::RecordField(it) => decl_with_type_ref(&it, it.ty(), StructureNodeKind::SymbolKind(SymbolKind::Field)), |
133 | ast::Const(it) => decl_with_type_ref(&it, it.ty(), SymbolKind::Const), | 152 | ast::Const(it) => decl_with_type_ref(&it, it.ty(), StructureNodeKind::SymbolKind(SymbolKind::Const)), |
134 | ast::Static(it) => decl_with_type_ref(&it, it.ty(), SymbolKind::Static), | 153 | ast::Static(it) => decl_with_type_ref(&it, it.ty(), StructureNodeKind::SymbolKind(SymbolKind::Static)), |
135 | ast::Impl(it) => { | 154 | ast::Impl(it) => { |
136 | let target_type = it.self_ty()?; | 155 | let target_type = it.self_ty()?; |
137 | let target_trait = it.trait_(); | 156 | let target_trait = it.trait_(); |
@@ -147,18 +166,38 @@ fn structure_node(node: &SyntaxNode) -> Option<StructureNode> { | |||
147 | label, | 166 | label, |
148 | navigation_range: target_type.syntax().text_range(), | 167 | navigation_range: target_type.syntax().text_range(), |
149 | node_range: it.syntax().text_range(), | 168 | node_range: it.syntax().text_range(), |
150 | kind: SymbolKind::Impl, | 169 | kind: StructureNodeKind::SymbolKind(SymbolKind::Impl), |
151 | detail: None, | 170 | detail: None, |
152 | deprecated: false, | 171 | deprecated: false, |
153 | }; | 172 | }; |
154 | Some(node) | 173 | Some(node) |
155 | }, | 174 | }, |
156 | ast::MacroRules(it) => decl(it, SymbolKind::Macro), | 175 | ast::MacroRules(it) => decl(it, StructureNodeKind::SymbolKind(SymbolKind::Macro)), |
157 | _ => None, | 176 | _ => None, |
158 | } | 177 | } |
159 | } | 178 | } |
160 | } | 179 | } |
161 | 180 | ||
181 | fn structure_token(token: SyntaxToken) -> Option<StructureNode> { | ||
182 | if let Some(comment) = ast::Comment::cast(token) { | ||
183 | let text = comment.text().trim(); | ||
184 | |||
185 | if let Some(region_name) = text.strip_prefix("// region:").map(str::trim) { | ||
186 | return Some(StructureNode { | ||
187 | parent: None, | ||
188 | label: region_name.to_string(), | ||
189 | navigation_range: comment.syntax().text_range(), | ||
190 | node_range: comment.syntax().text_range(), | ||
191 | kind: StructureNodeKind::Region, | ||
192 | detail: None, | ||
193 | deprecated: false, | ||
194 | }); | ||
195 | } | ||
196 | } | ||
197 | |||
198 | None | ||
199 | } | ||
200 | |||
162 | #[cfg(test)] | 201 | #[cfg(test)] |
163 | mod tests { | 202 | mod tests { |
164 | use expect_test::{expect, Expect}; | 203 | use expect_test::{expect, Expect}; |
@@ -217,6 +256,16 @@ fn obsolete() {} | |||
217 | 256 | ||
218 | #[deprecated(note = "for awhile")] | 257 | #[deprecated(note = "for awhile")] |
219 | fn very_obsolete() {} | 258 | fn very_obsolete() {} |
259 | |||
260 | // region: Some region name | ||
261 | // endregion | ||
262 | |||
263 | // region: dontpanic | ||
264 | mod m { | ||
265 | fn f() {} | ||
266 | // endregion | ||
267 | fn g() {} | ||
268 | } | ||
220 | "#, | 269 | "#, |
221 | expect![[r#" | 270 | expect![[r#" |
222 | [ | 271 | [ |
@@ -225,7 +274,9 @@ fn very_obsolete() {} | |||
225 | label: "Foo", | 274 | label: "Foo", |
226 | navigation_range: 8..11, | 275 | navigation_range: 8..11, |
227 | node_range: 1..26, | 276 | node_range: 1..26, |
228 | kind: Struct, | 277 | kind: SymbolKind( |
278 | Struct, | ||
279 | ), | ||
229 | detail: None, | 280 | detail: None, |
230 | deprecated: false, | 281 | deprecated: false, |
231 | }, | 282 | }, |
@@ -236,7 +287,9 @@ fn very_obsolete() {} | |||
236 | label: "x", | 287 | label: "x", |
237 | navigation_range: 18..19, | 288 | navigation_range: 18..19, |
238 | node_range: 18..24, | 289 | node_range: 18..24, |
239 | kind: Field, | 290 | kind: SymbolKind( |
291 | Field, | ||
292 | ), | ||
240 | detail: Some( | 293 | detail: Some( |
241 | "i32", | 294 | "i32", |
242 | ), | 295 | ), |
@@ -247,7 +300,9 @@ fn very_obsolete() {} | |||
247 | label: "m", | 300 | label: "m", |
248 | navigation_range: 32..33, | 301 | navigation_range: 32..33, |
249 | node_range: 28..158, | 302 | node_range: 28..158, |
250 | kind: Module, | 303 | kind: SymbolKind( |
304 | Module, | ||
305 | ), | ||
251 | detail: None, | 306 | detail: None, |
252 | deprecated: false, | 307 | deprecated: false, |
253 | }, | 308 | }, |
@@ -258,7 +313,9 @@ fn very_obsolete() {} | |||
258 | label: "bar1", | 313 | label: "bar1", |
259 | navigation_range: 43..47, | 314 | navigation_range: 43..47, |
260 | node_range: 40..52, | 315 | node_range: 40..52, |
261 | kind: Function, | 316 | kind: SymbolKind( |
317 | Function, | ||
318 | ), | ||
262 | detail: Some( | 319 | detail: Some( |
263 | "fn()", | 320 | "fn()", |
264 | ), | 321 | ), |
@@ -271,7 +328,9 @@ fn very_obsolete() {} | |||
271 | label: "bar2", | 328 | label: "bar2", |
272 | navigation_range: 60..64, | 329 | navigation_range: 60..64, |
273 | node_range: 57..81, | 330 | node_range: 57..81, |
274 | kind: Function, | 331 | kind: SymbolKind( |
332 | Function, | ||
333 | ), | ||
275 | detail: Some( | 334 | detail: Some( |
276 | "fn<T>(t: T) -> T", | 335 | "fn<T>(t: T) -> T", |
277 | ), | 336 | ), |
@@ -284,7 +343,9 @@ fn very_obsolete() {} | |||
284 | label: "bar3", | 343 | label: "bar3", |
285 | navigation_range: 89..93, | 344 | navigation_range: 89..93, |
286 | node_range: 86..156, | 345 | node_range: 86..156, |
287 | kind: Function, | 346 | kind: SymbolKind( |
347 | Function, | ||
348 | ), | ||
288 | detail: Some( | 349 | detail: Some( |
289 | "fn<A, B>(a: A, b: B) -> Vec< u32 >", | 350 | "fn<A, B>(a: A, b: B) -> Vec< u32 >", |
290 | ), | 351 | ), |
@@ -295,7 +356,9 @@ fn very_obsolete() {} | |||
295 | label: "E", | 356 | label: "E", |
296 | navigation_range: 165..166, | 357 | navigation_range: 165..166, |
297 | node_range: 160..180, | 358 | node_range: 160..180, |
298 | kind: Enum, | 359 | kind: SymbolKind( |
360 | Enum, | ||
361 | ), | ||
299 | detail: None, | 362 | detail: None, |
300 | deprecated: false, | 363 | deprecated: false, |
301 | }, | 364 | }, |
@@ -306,7 +369,9 @@ fn very_obsolete() {} | |||
306 | label: "X", | 369 | label: "X", |
307 | navigation_range: 169..170, | 370 | navigation_range: 169..170, |
308 | node_range: 169..170, | 371 | node_range: 169..170, |
309 | kind: Variant, | 372 | kind: SymbolKind( |
373 | Variant, | ||
374 | ), | ||
310 | detail: None, | 375 | detail: None, |
311 | deprecated: false, | 376 | deprecated: false, |
312 | }, | 377 | }, |
@@ -317,7 +382,9 @@ fn very_obsolete() {} | |||
317 | label: "Y", | 382 | label: "Y", |
318 | navigation_range: 172..173, | 383 | navigation_range: 172..173, |
319 | node_range: 172..178, | 384 | node_range: 172..178, |
320 | kind: Variant, | 385 | kind: SymbolKind( |
386 | Variant, | ||
387 | ), | ||
321 | detail: None, | 388 | detail: None, |
322 | deprecated: false, | 389 | deprecated: false, |
323 | }, | 390 | }, |
@@ -326,7 +393,9 @@ fn very_obsolete() {} | |||
326 | label: "T", | 393 | label: "T", |
327 | navigation_range: 186..187, | 394 | navigation_range: 186..187, |
328 | node_range: 181..193, | 395 | node_range: 181..193, |
329 | kind: TypeAlias, | 396 | kind: SymbolKind( |
397 | TypeAlias, | ||
398 | ), | ||
330 | detail: Some( | 399 | detail: Some( |
331 | "()", | 400 | "()", |
332 | ), | 401 | ), |
@@ -337,7 +406,9 @@ fn very_obsolete() {} | |||
337 | label: "S", | 406 | label: "S", |
338 | navigation_range: 201..202, | 407 | navigation_range: 201..202, |
339 | node_range: 194..213, | 408 | node_range: 194..213, |
340 | kind: Static, | 409 | kind: SymbolKind( |
410 | Static, | ||
411 | ), | ||
341 | detail: Some( | 412 | detail: Some( |
342 | "i32", | 413 | "i32", |
343 | ), | 414 | ), |
@@ -348,7 +419,9 @@ fn very_obsolete() {} | |||
348 | label: "C", | 419 | label: "C", |
349 | navigation_range: 220..221, | 420 | navigation_range: 220..221, |
350 | node_range: 214..232, | 421 | node_range: 214..232, |
351 | kind: Const, | 422 | kind: SymbolKind( |
423 | Const, | ||
424 | ), | ||
352 | detail: Some( | 425 | detail: Some( |
353 | "i32", | 426 | "i32", |
354 | ), | 427 | ), |
@@ -359,7 +432,9 @@ fn very_obsolete() {} | |||
359 | label: "impl E", | 432 | label: "impl E", |
360 | navigation_range: 239..240, | 433 | navigation_range: 239..240, |
361 | node_range: 234..243, | 434 | node_range: 234..243, |
362 | kind: Impl, | 435 | kind: SymbolKind( |
436 | Impl, | ||
437 | ), | ||
363 | detail: None, | 438 | detail: None, |
364 | deprecated: false, | 439 | deprecated: false, |
365 | }, | 440 | }, |
@@ -368,7 +443,9 @@ fn very_obsolete() {} | |||
368 | label: "impl fmt::Debug for E", | 443 | label: "impl fmt::Debug for E", |
369 | navigation_range: 265..266, | 444 | navigation_range: 265..266, |
370 | node_range: 245..269, | 445 | node_range: 245..269, |
371 | kind: Impl, | 446 | kind: SymbolKind( |
447 | Impl, | ||
448 | ), | ||
372 | detail: None, | 449 | detail: None, |
373 | deprecated: false, | 450 | deprecated: false, |
374 | }, | 451 | }, |
@@ -377,7 +454,9 @@ fn very_obsolete() {} | |||
377 | label: "mc", | 454 | label: "mc", |
378 | navigation_range: 284..286, | 455 | navigation_range: 284..286, |
379 | node_range: 271..303, | 456 | node_range: 271..303, |
380 | kind: Macro, | 457 | kind: SymbolKind( |
458 | Macro, | ||
459 | ), | ||
381 | detail: None, | 460 | detail: None, |
382 | deprecated: false, | 461 | deprecated: false, |
383 | }, | 462 | }, |
@@ -386,7 +465,9 @@ fn very_obsolete() {} | |||
386 | label: "mcexp", | 465 | label: "mcexp", |
387 | navigation_range: 334..339, | 466 | navigation_range: 334..339, |
388 | node_range: 305..356, | 467 | node_range: 305..356, |
389 | kind: Macro, | 468 | kind: SymbolKind( |
469 | Macro, | ||
470 | ), | ||
390 | detail: None, | 471 | detail: None, |
391 | deprecated: false, | 472 | deprecated: false, |
392 | }, | 473 | }, |
@@ -395,7 +476,9 @@ fn very_obsolete() {} | |||
395 | label: "mcexp", | 476 | label: "mcexp", |
396 | navigation_range: 387..392, | 477 | navigation_range: 387..392, |
397 | node_range: 358..409, | 478 | node_range: 358..409, |
398 | kind: Macro, | 479 | kind: SymbolKind( |
480 | Macro, | ||
481 | ), | ||
399 | detail: None, | 482 | detail: None, |
400 | deprecated: false, | 483 | deprecated: false, |
401 | }, | 484 | }, |
@@ -404,7 +487,9 @@ fn very_obsolete() {} | |||
404 | label: "obsolete", | 487 | label: "obsolete", |
405 | navigation_range: 428..436, | 488 | navigation_range: 428..436, |
406 | node_range: 411..441, | 489 | node_range: 411..441, |
407 | kind: Function, | 490 | kind: SymbolKind( |
491 | Function, | ||
492 | ), | ||
408 | detail: Some( | 493 | detail: Some( |
409 | "fn()", | 494 | "fn()", |
410 | ), | 495 | ), |
@@ -415,12 +500,75 @@ fn very_obsolete() {} | |||
415 | label: "very_obsolete", | 500 | label: "very_obsolete", |
416 | navigation_range: 481..494, | 501 | navigation_range: 481..494, |
417 | node_range: 443..499, | 502 | node_range: 443..499, |
418 | kind: Function, | 503 | kind: SymbolKind( |
504 | Function, | ||
505 | ), | ||
419 | detail: Some( | 506 | detail: Some( |
420 | "fn()", | 507 | "fn()", |
421 | ), | 508 | ), |
422 | deprecated: true, | 509 | deprecated: true, |
423 | }, | 510 | }, |
511 | StructureNode { | ||
512 | parent: None, | ||
513 | label: "Some region name", | ||
514 | navigation_range: 501..528, | ||
515 | node_range: 501..528, | ||
516 | kind: Region, | ||
517 | detail: None, | ||
518 | deprecated: false, | ||
519 | }, | ||
520 | StructureNode { | ||
521 | parent: None, | ||
522 | label: "m", | ||
523 | navigation_range: 568..569, | ||
524 | node_range: 543..606, | ||
525 | kind: SymbolKind( | ||
526 | Module, | ||
527 | ), | ||
528 | detail: None, | ||
529 | deprecated: false, | ||
530 | }, | ||
531 | StructureNode { | ||
532 | parent: Some( | ||
533 | 20, | ||
534 | ), | ||
535 | label: "dontpanic", | ||
536 | navigation_range: 543..563, | ||
537 | node_range: 543..563, | ||
538 | kind: Region, | ||
539 | detail: None, | ||
540 | deprecated: false, | ||
541 | }, | ||
542 | StructureNode { | ||
543 | parent: Some( | ||
544 | 20, | ||
545 | ), | ||
546 | label: "f", | ||
547 | navigation_range: 575..576, | ||
548 | node_range: 572..581, | ||
549 | kind: SymbolKind( | ||
550 | Function, | ||
551 | ), | ||
552 | detail: Some( | ||
553 | "fn()", | ||
554 | ), | ||
555 | deprecated: false, | ||
556 | }, | ||
557 | StructureNode { | ||
558 | parent: Some( | ||
559 | 20, | ||
560 | ), | ||
561 | label: "g", | ||
562 | navigation_range: 598..599, | ||
563 | node_range: 582..604, | ||
564 | kind: SymbolKind( | ||
565 | Function, | ||
566 | ), | ||
567 | detail: Some( | ||
568 | "fn()", | ||
569 | ), | ||
570 | deprecated: false, | ||
571 | }, | ||
424 | ] | 572 | ] |
425 | "#]], | 573 | "#]], |
426 | ); | 574 | ); |
diff --git a/crates/ide/src/lib.rs b/crates/ide/src/lib.rs index a8b169e87..662da5a96 100644 --- a/crates/ide/src/lib.rs +++ b/crates/ide/src/lib.rs | |||
@@ -71,7 +71,7 @@ pub use crate::{ | |||
71 | diagnostics::{Diagnostic, DiagnosticsConfig, Fix, Severity}, | 71 | diagnostics::{Diagnostic, DiagnosticsConfig, Fix, Severity}, |
72 | display::navigation_target::NavigationTarget, | 72 | display::navigation_target::NavigationTarget, |
73 | expand_macro::ExpandedMacro, | 73 | expand_macro::ExpandedMacro, |
74 | file_structure::StructureNode, | 74 | file_structure::{StructureNode, StructureNodeKind}, |
75 | folding_ranges::{Fold, FoldKind}, | 75 | folding_ranges::{Fold, FoldKind}, |
76 | hover::{HoverAction, HoverConfig, HoverGotoTypeData, HoverResult}, | 76 | hover::{HoverAction, HoverConfig, HoverGotoTypeData, HoverResult}, |
77 | inlay_hints::{InlayHint, InlayHintsConfig, InlayKind}, | 77 | inlay_hints::{InlayHint, InlayHintsConfig, InlayKind}, |
@@ -101,7 +101,7 @@ pub use ide_db::{ | |||
101 | search::{ReferenceAccess, SearchScope}, | 101 | search::{ReferenceAccess, SearchScope}, |
102 | source_change::{FileSystemEdit, SourceChange}, | 102 | source_change::{FileSystemEdit, SourceChange}, |
103 | symbol_index::Query, | 103 | symbol_index::Query, |
104 | RootDatabase, | 104 | RootDatabase, SymbolKind, |
105 | }; | 105 | }; |
106 | pub use ide_ssr::SsrError; | 106 | pub use ide_ssr::SsrError; |
107 | pub use syntax::{TextRange, TextSize}; | 107 | pub use syntax::{TextRange, TextSize}; |
diff --git a/crates/ide/src/references.rs b/crates/ide/src/references.rs index fef70533d..ec7c7686d 100644 --- a/crates/ide/src/references.rs +++ b/crates/ide/src/references.rs | |||
@@ -29,7 +29,7 @@ use crate::{display::TryToNav, FilePosition, NavigationTarget}; | |||
29 | 29 | ||
30 | #[derive(Debug, Clone)] | 30 | #[derive(Debug, Clone)] |
31 | pub struct ReferenceSearchResult { | 31 | pub struct ReferenceSearchResult { |
32 | pub declaration: Declaration, | 32 | pub declaration: Option<Declaration>, |
33 | pub references: FxHashMap<FileId, Vec<(TextRange, Option<ReferenceAccess>)>>, | 33 | pub references: FxHashMap<FileId, Vec<(TextRange, Option<ReferenceAccess>)>>, |
34 | } | 34 | } |
35 | 35 | ||
@@ -91,10 +91,10 @@ pub(crate) fn find_all_refs( | |||
91 | _ => {} | 91 | _ => {} |
92 | } | 92 | } |
93 | } | 93 | } |
94 | let nav = def.try_to_nav(sema.db)?; | 94 | let declaration = def.try_to_nav(sema.db).map(|nav| { |
95 | let decl_range = nav.focus_or_full_range(); | 95 | let decl_range = nav.focus_or_full_range(); |
96 | 96 | Declaration { nav, access: decl_access(&def, &syntax, decl_range) } | |
97 | let declaration = Declaration { nav, access: decl_access(&def, &syntax, decl_range) }; | 97 | }); |
98 | let references = usages | 98 | let references = usages |
99 | .into_iter() | 99 | .into_iter() |
100 | .map(|(file_id, refs)| { | 100 | .map(|(file_id, refs)| { |
@@ -1004,8 +1004,7 @@ impl Foo { | |||
1004 | let refs = analysis.find_all_refs(pos, search_scope).unwrap().unwrap(); | 1004 | let refs = analysis.find_all_refs(pos, search_scope).unwrap().unwrap(); |
1005 | 1005 | ||
1006 | let mut actual = String::new(); | 1006 | let mut actual = String::new(); |
1007 | { | 1007 | if let Some(decl) = refs.declaration { |
1008 | let decl = refs.declaration; | ||
1009 | format_to!(actual, "{}", decl.nav.debug_render()); | 1008 | format_to!(actual, "{}", decl.nav.debug_render()); |
1010 | if let Some(access) = decl.access { | 1009 | if let Some(access) = decl.access { |
1011 | format_to!(actual, " {:?}", access) | 1010 | format_to!(actual, " {:?}", access) |
@@ -1258,4 +1257,17 @@ fn main() { | |||
1258 | "#]], | 1257 | "#]], |
1259 | ); | 1258 | ); |
1260 | } | 1259 | } |
1260 | |||
1261 | #[test] | ||
1262 | fn test_primitives() { | ||
1263 | check( | ||
1264 | r#" | ||
1265 | fn foo(_: bool) -> bo$0ol { true } | ||
1266 | "#, | ||
1267 | expect![[r#" | ||
1268 | FileId(0) 10..14 | ||
1269 | FileId(0) 19..23 | ||
1270 | "#]], | ||
1271 | ); | ||
1272 | } | ||
1261 | } | 1273 | } |
diff --git a/crates/ide/src/references/rename.rs b/crates/ide/src/references/rename.rs index 1e378279d..5340b638a 100644 --- a/crates/ide/src/references/rename.rs +++ b/crates/ide/src/references/rename.rs | |||
@@ -510,7 +510,8 @@ fn source_edit_from_def( | |||
510 | def: Definition, | 510 | def: Definition, |
511 | new_name: &str, | 511 | new_name: &str, |
512 | ) -> RenameResult<(FileId, TextEdit)> { | 512 | ) -> RenameResult<(FileId, TextEdit)> { |
513 | let nav = def.try_to_nav(sema.db).unwrap(); | 513 | let nav = |
514 | def.try_to_nav(sema.db).ok_or_else(|| format_err!("No references found at position"))?; | ||
514 | 515 | ||
515 | let mut replacement_text = String::new(); | 516 | let mut replacement_text = String::new(); |
516 | let mut repl_range = nav.focus_or_full_range(); | 517 | let mut repl_range = nav.focus_or_full_range(); |
diff --git a/crates/ide_db/src/defs.rs b/crates/ide_db/src/defs.rs index ff612b7d0..f86e5ce93 100644 --- a/crates/ide_db/src/defs.rs +++ b/crates/ide_db/src/defs.rs | |||
@@ -70,7 +70,7 @@ impl Definition { | |||
70 | hir::ModuleDef::Static(it) => it.name(db)?, | 70 | hir::ModuleDef::Static(it) => it.name(db)?, |
71 | hir::ModuleDef::Trait(it) => it.name(db), | 71 | hir::ModuleDef::Trait(it) => it.name(db), |
72 | hir::ModuleDef::TypeAlias(it) => it.name(db), | 72 | hir::ModuleDef::TypeAlias(it) => it.name(db), |
73 | hir::ModuleDef::BuiltinType(_) => return None, | 73 | hir::ModuleDef::BuiltinType(it) => it.name(), |
74 | }, | 74 | }, |
75 | Definition::SelfType(_) => return None, | 75 | Definition::SelfType(_) => return None, |
76 | Definition::Local(it) => it.name(db)?, | 76 | Definition::Local(it) => it.name(db)?, |
diff --git a/crates/ide_db/src/search.rs b/crates/ide_db/src/search.rs index fa18703e1..d00a8b6e7 100644 --- a/crates/ide_db/src/search.rs +++ b/crates/ide_db/src/search.rs | |||
@@ -6,7 +6,7 @@ | |||
6 | 6 | ||
7 | use std::{convert::TryInto, mem}; | 7 | use std::{convert::TryInto, mem}; |
8 | 8 | ||
9 | use base_db::{FileId, FileRange, SourceDatabaseExt}; | 9 | use base_db::{FileId, FileRange, SourceDatabase, SourceDatabaseExt}; |
10 | use hir::{DefWithBody, HasSource, Module, ModuleSource, Semantics, Visibility}; | 10 | use hir::{DefWithBody, HasSource, Module, ModuleSource, Semantics, Visibility}; |
11 | use once_cell::unsync::Lazy; | 11 | use once_cell::unsync::Lazy; |
12 | use rustc_hash::FxHashMap; | 12 | use rustc_hash::FxHashMap; |
@@ -138,6 +138,20 @@ impl IntoIterator for SearchScope { | |||
138 | impl Definition { | 138 | impl Definition { |
139 | fn search_scope(&self, db: &RootDatabase) -> SearchScope { | 139 | fn search_scope(&self, db: &RootDatabase) -> SearchScope { |
140 | let _p = profile::span("search_scope"); | 140 | let _p = profile::span("search_scope"); |
141 | |||
142 | if let Definition::ModuleDef(hir::ModuleDef::BuiltinType(_)) = self { | ||
143 | let mut res = FxHashMap::default(); | ||
144 | |||
145 | let graph = db.crate_graph(); | ||
146 | for krate in graph.iter() { | ||
147 | let root_file = graph[krate].root_file_id; | ||
148 | let source_root_id = db.file_source_root(root_file); | ||
149 | let source_root = db.source_root(source_root_id); | ||
150 | res.extend(source_root.iter().map(|id| (id, None))); | ||
151 | } | ||
152 | return SearchScope::new(res); | ||
153 | } | ||
154 | |||
141 | let module = match self.module(db) { | 155 | let module = match self.module(db) { |
142 | Some(it) => it, | 156 | Some(it) => it, |
143 | None => return SearchScope::empty(), | 157 | None => return SearchScope::empty(), |
diff --git a/crates/rust-analyzer/src/handlers.rs b/crates/rust-analyzer/src/handlers.rs index 706a39dab..6fb7da79c 100644 --- a/crates/rust-analyzer/src/handlers.rs +++ b/crates/rust-analyzer/src/handlers.rs | |||
@@ -289,7 +289,7 @@ pub(crate) fn handle_document_symbol( | |||
289 | let doc_symbol = lsp_types::DocumentSymbol { | 289 | let doc_symbol = lsp_types::DocumentSymbol { |
290 | name: symbol.label, | 290 | name: symbol.label, |
291 | detail: symbol.detail, | 291 | detail: symbol.detail, |
292 | kind: to_proto::symbol_kind(symbol.kind), | 292 | kind: to_proto::structure_node_kind(symbol.kind), |
293 | tags: Some(tags), | 293 | tags: Some(tags), |
294 | deprecated: Some(symbol.deprecated), | 294 | deprecated: Some(symbol.deprecated), |
295 | range: to_proto::range(&line_index, symbol.node_range), | 295 | range: to_proto::range(&line_index, symbol.node_range), |
@@ -846,9 +846,9 @@ pub(crate) fn handle_references( | |||
846 | }; | 846 | }; |
847 | 847 | ||
848 | let decl = if params.context.include_declaration { | 848 | let decl = if params.context.include_declaration { |
849 | Some(FileRange { | 849 | refs.declaration.map(|decl| FileRange { |
850 | file_id: refs.declaration.nav.file_id, | 850 | file_id: decl.nav.file_id, |
851 | range: refs.declaration.nav.focus_or_full_range(), | 851 | range: decl.nav.focus_or_full_range(), |
852 | }) | 852 | }) |
853 | } else { | 853 | } else { |
854 | None | 854 | None |
@@ -1153,14 +1153,12 @@ pub(crate) fn handle_document_highlight( | |||
1153 | Some(refs) => refs, | 1153 | Some(refs) => refs, |
1154 | }; | 1154 | }; |
1155 | 1155 | ||
1156 | let decl = if refs.declaration.nav.file_id == position.file_id { | 1156 | let decl = refs.declaration.filter(|decl| decl.nav.file_id == position.file_id).map(|decl| { |
1157 | Some(DocumentHighlight { | 1157 | DocumentHighlight { |
1158 | range: to_proto::range(&line_index, refs.declaration.nav.focus_or_full_range()), | 1158 | range: to_proto::range(&line_index, decl.nav.focus_or_full_range()), |
1159 | kind: refs.declaration.access.map(to_proto::document_highlight_kind), | 1159 | kind: decl.access.map(to_proto::document_highlight_kind), |
1160 | }) | 1160 | } |
1161 | } else { | 1161 | }); |
1162 | None | ||
1163 | }; | ||
1164 | 1162 | ||
1165 | let file_refs = refs.references.get(&position.file_id).map_or(&[][..], Vec::as_slice); | 1163 | let file_refs = refs.references.get(&position.file_id).map_or(&[][..], Vec::as_slice); |
1166 | let mut res = Vec::with_capacity(file_refs.len() + 1); | 1164 | let mut res = Vec::with_capacity(file_refs.len() + 1); |
diff --git a/crates/rust-analyzer/src/to_proto.rs b/crates/rust-analyzer/src/to_proto.rs index d415ed4d3..70eaae5e8 100644 --- a/crates/rust-analyzer/src/to_proto.rs +++ b/crates/rust-analyzer/src/to_proto.rs | |||
@@ -9,9 +9,8 @@ use ide::{ | |||
9 | CompletionRelevance, Documentation, FileId, FileRange, FileSystemEdit, Fold, FoldKind, | 9 | CompletionRelevance, Documentation, FileId, FileRange, FileSystemEdit, Fold, FoldKind, |
10 | Highlight, HlMod, HlPunct, HlRange, HlTag, Indel, InlayHint, InlayKind, InsertTextFormat, | 10 | Highlight, HlMod, HlPunct, HlRange, HlTag, Indel, InlayHint, InlayKind, InsertTextFormat, |
11 | Markup, NavigationTarget, ReferenceAccess, RenameError, Runnable, Severity, SourceChange, | 11 | Markup, NavigationTarget, ReferenceAccess, RenameError, Runnable, Severity, SourceChange, |
12 | TextEdit, TextRange, TextSize, | 12 | StructureNodeKind, SymbolKind, TextEdit, TextRange, TextSize, |
13 | }; | 13 | }; |
14 | use ide_db::SymbolKind; | ||
15 | use itertools::Itertools; | 14 | use itertools::Itertools; |
16 | use serde_json::to_value; | 15 | use serde_json::to_value; |
17 | 16 | ||
@@ -63,6 +62,13 @@ pub(crate) fn symbol_kind(symbol_kind: SymbolKind) -> lsp_types::SymbolKind { | |||
63 | } | 62 | } |
64 | } | 63 | } |
65 | 64 | ||
65 | pub(crate) fn structure_node_kind(kind: StructureNodeKind) -> lsp_types::SymbolKind { | ||
66 | match kind { | ||
67 | StructureNodeKind::SymbolKind(symbol) => symbol_kind(symbol), | ||
68 | StructureNodeKind::Region => lsp_types::SymbolKind::Namespace, | ||
69 | } | ||
70 | } | ||
71 | |||
66 | pub(crate) fn document_highlight_kind( | 72 | pub(crate) fn document_highlight_kind( |
67 | reference_access: ReferenceAccess, | 73 | reference_access: ReferenceAccess, |
68 | ) -> lsp_types::DocumentHighlightKind { | 74 | ) -> lsp_types::DocumentHighlightKind { |