diff options
Diffstat (limited to 'crates')
-rw-r--r-- | crates/ra_analysis/src/imp.rs | 32 | ||||
-rw-r--r-- | crates/ra_analysis/src/lib.rs | 3 | ||||
-rw-r--r-- | crates/ra_analysis/tests/tests.rs | 42 | ||||
-rw-r--r-- | crates/ra_editor/src/scope/fn_scope.rs | 1 | ||||
-rw-r--r-- | crates/ra_lsp_server/src/caps.rs | 2 | ||||
-rw-r--r-- | crates/ra_lsp_server/src/main_loop/handlers.rs | 16 | ||||
-rw-r--r-- | crates/ra_lsp_server/src/main_loop/mod.rs | 1 | ||||
-rw-r--r-- | crates/ra_lsp_server/src/req.rs | 2 |
8 files changed, 96 insertions, 3 deletions
diff --git a/crates/ra_analysis/src/imp.rs b/crates/ra_analysis/src/imp.rs index f1403cb5d..a67b1717a 100644 --- a/crates/ra_analysis/src/imp.rs +++ b/crates/ra_analysis/src/imp.rs | |||
@@ -257,6 +257,38 @@ impl AnalysisImpl { | |||
257 | vec![] | 257 | vec![] |
258 | } | 258 | } |
259 | 259 | ||
260 | pub fn find_all_refs(&self, file_id: FileId, offset: TextUnit, _token: &JobToken) -> Vec<(FileId, TextRange)> { | ||
261 | let root = self.root(file_id); | ||
262 | let file = root.syntax(file_id); | ||
263 | let syntax = file.syntax(); | ||
264 | |||
265 | let mut ret = vec![]; | ||
266 | |||
267 | // Find the symbol we are looking for | ||
268 | if let Some(name_ref) = find_node_at_offset::<ast::NameRef>(syntax, offset) { | ||
269 | |||
270 | // We are only handing local references for now | ||
271 | if let Some(resolved) = resolve_local_name(&file, offset, name_ref) { | ||
272 | |||
273 | ret.push((file_id, resolved.1)); | ||
274 | |||
275 | if let Some(fn_def) = find_node_at_offset::<ast::FnDef>(syntax, offset) { | ||
276 | |||
277 | let refs : Vec<_> = fn_def.syntax().descendants() | ||
278 | .filter_map(ast::NameRef::cast) | ||
279 | .filter(|n: &ast::NameRef| resolve_local_name(&file, n.syntax().range().start(), *n) == Some(resolved.clone())) | ||
280 | .collect(); | ||
281 | |||
282 | for r in refs { | ||
283 | ret.push((file_id, r.syntax().range())); | ||
284 | } | ||
285 | } | ||
286 | } | ||
287 | } | ||
288 | |||
289 | ret | ||
290 | } | ||
291 | |||
260 | pub fn diagnostics(&self, file_id: FileId) -> Vec<Diagnostic> { | 292 | pub fn diagnostics(&self, file_id: FileId) -> Vec<Diagnostic> { |
261 | let root = self.root(file_id); | 293 | let root = self.root(file_id); |
262 | let module_tree = root.module_tree(); | 294 | let module_tree = root.module_tree(); |
diff --git a/crates/ra_analysis/src/lib.rs b/crates/ra_analysis/src/lib.rs index 2eeacaabe..46cc0722b 100644 --- a/crates/ra_analysis/src/lib.rs +++ b/crates/ra_analysis/src/lib.rs | |||
@@ -217,6 +217,9 @@ impl Analysis { | |||
217 | self.imp | 217 | self.imp |
218 | .approximately_resolve_symbol(file_id, offset, token) | 218 | .approximately_resolve_symbol(file_id, offset, token) |
219 | } | 219 | } |
220 | pub fn find_all_refs(&self, file_id: FileId, offset: TextUnit, token: &JobToken) -> Vec<(FileId, TextRange)> { | ||
221 | self.imp.find_all_refs(file_id, offset, token) | ||
222 | } | ||
220 | pub fn parent_module(&self, file_id: FileId) -> Vec<(FileId, FileSymbol)> { | 223 | pub fn parent_module(&self, file_id: FileId) -> Vec<(FileId, FileSymbol)> { |
221 | self.imp.parent_module(file_id) | 224 | self.imp.parent_module(file_id) |
222 | } | 225 | } |
diff --git a/crates/ra_analysis/tests/tests.rs b/crates/ra_analysis/tests/tests.rs index e0c637d65..0c2c69ea0 100644 --- a/crates/ra_analysis/tests/tests.rs +++ b/crates/ra_analysis/tests/tests.rs | |||
@@ -10,6 +10,8 @@ use std::sync::Arc; | |||
10 | use ra_analysis::{ | 10 | use ra_analysis::{ |
11 | Analysis, AnalysisHost, CrateGraph, CrateId, FileId, FileResolver, FnDescriptor, JobHandle, | 11 | Analysis, AnalysisHost, CrateGraph, CrateId, FileId, FileResolver, FnDescriptor, JobHandle, |
12 | }; | 12 | }; |
13 | use ra_syntax::TextRange; | ||
14 | |||
13 | use relative_path::{RelativePath, RelativePathBuf}; | 15 | use relative_path::{RelativePath, RelativePathBuf}; |
14 | use rustc_hash::FxHashMap; | 16 | use rustc_hash::FxHashMap; |
15 | use test_utils::{assert_eq_dbg, extract_offset}; | 17 | use test_utils::{assert_eq_dbg, extract_offset}; |
@@ -225,3 +227,43 @@ fn bar() { | |||
225 | assert_eq!(desc.ret_type, None); | 227 | assert_eq!(desc.ret_type, None); |
226 | assert_eq!(param, Some(1)); | 228 | assert_eq!(param, Some(1)); |
227 | } | 229 | } |
230 | |||
231 | fn get_all_refs(text: &str) -> Vec<(FileId, TextRange)> { | ||
232 | let (offset, code) = extract_offset(text); | ||
233 | let code = code.as_str(); | ||
234 | |||
235 | let (_handle, token) = JobHandle::new(); | ||
236 | let snap = analysis(&[("/lib.rs", code)]); | ||
237 | |||
238 | snap.find_all_refs(FileId(1), offset, &token) | ||
239 | } | ||
240 | |||
241 | #[test] | ||
242 | fn test_find_all_refs_for_local() { | ||
243 | let code = r#" | ||
244 | fn main() { | ||
245 | let mut i = 1; | ||
246 | let j = 1; | ||
247 | i = i<|> + j; | ||
248 | |||
249 | { | ||
250 | i = 0; | ||
251 | } | ||
252 | |||
253 | i = 5; | ||
254 | }"#; | ||
255 | |||
256 | let refs = get_all_refs(code); | ||
257 | assert_eq!(refs.len(), 5); | ||
258 | } | ||
259 | |||
260 | #[test] | ||
261 | fn test_find_all_refs_for_param_inside() { | ||
262 | let code = r#" | ||
263 | fn foo(i : u32) -> u32 { | ||
264 | i<|> | ||
265 | }"#; | ||
266 | |||
267 | let refs = get_all_refs(code); | ||
268 | assert_eq!(refs.len(), 2); | ||
269 | } \ No newline at end of file | ||
diff --git a/crates/ra_editor/src/scope/fn_scope.rs b/crates/ra_editor/src/scope/fn_scope.rs index 9088e5a60..f10bdf657 100644 --- a/crates/ra_editor/src/scope/fn_scope.rs +++ b/crates/ra_editor/src/scope/fn_scope.rs | |||
@@ -270,7 +270,6 @@ pub fn resolve_local_name<'a>( | |||
270 | .filter(|entry| shadowed.insert(entry.name())) | 270 | .filter(|entry| shadowed.insert(entry.name())) |
271 | .filter(|entry| entry.name() == name_ref.text()) | 271 | .filter(|entry| entry.name() == name_ref.text()) |
272 | .nth(0); | 272 | .nth(0); |
273 | eprintln!("ret = {:?}", ret); | ||
274 | ret | 273 | ret |
275 | } | 274 | } |
276 | 275 | ||
diff --git a/crates/ra_lsp_server/src/caps.rs b/crates/ra_lsp_server/src/caps.rs index 1dd495791..84c43bbec 100644 --- a/crates/ra_lsp_server/src/caps.rs +++ b/crates/ra_lsp_server/src/caps.rs | |||
@@ -27,7 +27,7 @@ pub fn server_capabilities() -> ServerCapabilities { | |||
27 | definition_provider: Some(true), | 27 | definition_provider: Some(true), |
28 | type_definition_provider: None, | 28 | type_definition_provider: None, |
29 | implementation_provider: None, | 29 | implementation_provider: None, |
30 | references_provider: None, | 30 | references_provider: Some(true), |
31 | document_highlight_provider: None, | 31 | document_highlight_provider: None, |
32 | document_symbol_provider: Some(true), | 32 | document_symbol_provider: Some(true), |
33 | workspace_symbol_provider: Some(true), | 33 | workspace_symbol_provider: Some(true), |
diff --git a/crates/ra_lsp_server/src/main_loop/handlers.rs b/crates/ra_lsp_server/src/main_loop/handlers.rs index c25b63852..9b8d40eaa 100644 --- a/crates/ra_lsp_server/src/main_loop/handlers.rs +++ b/crates/ra_lsp_server/src/main_loop/handlers.rs | |||
@@ -460,6 +460,22 @@ pub fn handle_signature_help( | |||
460 | } | 460 | } |
461 | } | 461 | } |
462 | 462 | ||
463 | pub fn handle_references( | ||
464 | world: ServerWorld, | ||
465 | params: req::ReferenceParams, | ||
466 | token: JobToken, | ||
467 | ) -> Result<Option<Vec<Location>>> { | ||
468 | let file_id = params.text_document.try_conv_with(&world)?; | ||
469 | let line_index = world.analysis().file_line_index(file_id); | ||
470 | let offset = params.position.conv_with(&line_index); | ||
471 | |||
472 | let refs = world.analysis().find_all_refs(file_id, offset, &token); | ||
473 | |||
474 | Ok(Some(refs.into_iter() | ||
475 | .filter_map(|r| to_location(r.0, r.1, &world, &line_index).ok()) | ||
476 | .collect())) | ||
477 | } | ||
478 | |||
463 | pub fn handle_code_action( | 479 | pub fn handle_code_action( |
464 | world: ServerWorld, | 480 | world: ServerWorld, |
465 | params: req::CodeActionParams, | 481 | params: req::CodeActionParams, |
diff --git a/crates/ra_lsp_server/src/main_loop/mod.rs b/crates/ra_lsp_server/src/main_loop/mod.rs index a11baf4aa..7efec8a7a 100644 --- a/crates/ra_lsp_server/src/main_loop/mod.rs +++ b/crates/ra_lsp_server/src/main_loop/mod.rs | |||
@@ -248,6 +248,7 @@ fn on_request( | |||
248 | .on::<req::CodeActionRequest>(handlers::handle_code_action)? | 248 | .on::<req::CodeActionRequest>(handlers::handle_code_action)? |
249 | .on::<req::FoldingRangeRequest>(handlers::handle_folding_range)? | 249 | .on::<req::FoldingRangeRequest>(handlers::handle_folding_range)? |
250 | .on::<req::SignatureHelpRequest>(handlers::handle_signature_help)? | 250 | .on::<req::SignatureHelpRequest>(handlers::handle_signature_help)? |
251 | .on::<req::References>(handlers::handle_references)? | ||
251 | .finish(); | 252 | .finish(); |
252 | match req { | 253 | match req { |
253 | Ok((id, handle)) => { | 254 | Ok((id, handle)) => { |
diff --git a/crates/ra_lsp_server/src/req.rs b/crates/ra_lsp_server/src/req.rs index b76bfbcbc..6cd04d84c 100644 --- a/crates/ra_lsp_server/src/req.rs +++ b/crates/ra_lsp_server/src/req.rs | |||
@@ -7,7 +7,7 @@ pub use languageserver_types::{ | |||
7 | CompletionResponse, DocumentOnTypeFormattingParams, DocumentSymbolParams, | 7 | CompletionResponse, DocumentOnTypeFormattingParams, DocumentSymbolParams, |
8 | DocumentSymbolResponse, ExecuteCommandParams, Hover, InitializeResult, | 8 | DocumentSymbolResponse, ExecuteCommandParams, Hover, InitializeResult, |
9 | PublishDiagnosticsParams, SignatureHelp, TextDocumentEdit, TextDocumentPositionParams, | 9 | PublishDiagnosticsParams, SignatureHelp, TextDocumentEdit, TextDocumentPositionParams, |
10 | TextEdit, WorkspaceSymbolParams, | 10 | TextEdit, WorkspaceSymbolParams, ReferenceParams, |
11 | }; | 11 | }; |
12 | 12 | ||
13 | pub enum SyntaxTree {} | 13 | pub enum SyntaxTree {} |