diff options
author | Aleksey Kladov <[email protected]> | 2018-11-01 10:15:11 +0000 |
---|---|---|
committer | Aleksey Kladov <[email protected]> | 2018-11-01 10:15:11 +0000 |
commit | 2050812cad7905c33870c36362593360f7216310 (patch) | |
tree | e329f4ca8a75790442dc5aa67bfa6e2599aedfe3 | |
parent | 2b210d98b8807e0dfd6762a2974388fca7ee1ee9 (diff) |
remove SyntaxPtrDatabase
-rw-r--r-- | crates/ra_analysis/src/db.rs | 14 | ||||
-rw-r--r-- | crates/ra_analysis/src/descriptors/mod.rs | 4 | ||||
-rw-r--r-- | crates/ra_analysis/src/imp.rs | 1 | ||||
-rw-r--r-- | crates/ra_analysis/src/syntax_ptr.rs | 12 |
4 files changed, 12 insertions, 19 deletions
diff --git a/crates/ra_analysis/src/db.rs b/crates/ra_analysis/src/db.rs index 9ab273ad7..94fdd36a4 100644 --- a/crates/ra_analysis/src/db.rs +++ b/crates/ra_analysis/src/db.rs | |||
@@ -1,7 +1,7 @@ | |||
1 | use std::sync::Arc; | 1 | use std::sync::Arc; |
2 | 2 | ||
3 | use ra_editor::LineIndex; | 3 | use ra_editor::LineIndex; |
4 | use ra_syntax::File; | 4 | use ra_syntax::{File, SyntaxNode}; |
5 | use salsa; | 5 | use salsa; |
6 | 6 | ||
7 | use crate::{ | 7 | use crate::{ |
@@ -11,7 +11,7 @@ use crate::{ | |||
11 | SubmodulesQuery, | 11 | SubmodulesQuery, |
12 | }, | 12 | }, |
13 | symbol_index::SymbolIndex, | 13 | symbol_index::SymbolIndex, |
14 | syntax_ptr::{ResolveSyntaxPtrQuery, SyntaxPtrDatabase}, | 14 | syntax_ptr::SyntaxPtr, |
15 | Cancelable, Canceled, FileId, | 15 | Cancelable, Canceled, FileId, |
16 | }; | 16 | }; |
17 | 17 | ||
@@ -62,6 +62,7 @@ salsa::database_storage! { | |||
62 | fn file_syntax() for FileSyntaxQuery; | 62 | fn file_syntax() for FileSyntaxQuery; |
63 | fn file_lines() for FileLinesQuery; | 63 | fn file_lines() for FileLinesQuery; |
64 | fn file_symbols() for FileSymbolsQuery; | 64 | fn file_symbols() for FileSymbolsQuery; |
65 | fn resolve_syntax_ptr() for ResolveSyntaxPtrQuery; | ||
65 | } | 66 | } |
66 | impl DescriptorDatabase { | 67 | impl DescriptorDatabase { |
67 | fn module_tree() for ModuleTreeQuery; | 68 | fn module_tree() for ModuleTreeQuery; |
@@ -70,9 +71,6 @@ salsa::database_storage! { | |||
70 | fn fn_syntax() for FnSyntaxQuery; | 71 | fn fn_syntax() for FnSyntaxQuery; |
71 | fn fn_scopes() for FnScopesQuery; | 72 | fn fn_scopes() for FnScopesQuery; |
72 | } | 73 | } |
73 | impl SyntaxPtrDatabase { | ||
74 | fn resolve_syntax_ptr() for ResolveSyntaxPtrQuery; | ||
75 | } | ||
76 | } | 74 | } |
77 | } | 75 | } |
78 | 76 | ||
@@ -87,6 +85,12 @@ salsa::query_group! { | |||
87 | fn file_symbols(file_id: FileId) -> Cancelable<Arc<SymbolIndex>> { | 85 | fn file_symbols(file_id: FileId) -> Cancelable<Arc<SymbolIndex>> { |
88 | type FileSymbolsQuery; | 86 | type FileSymbolsQuery; |
89 | } | 87 | } |
88 | fn resolve_syntax_ptr(ptr: SyntaxPtr) -> SyntaxNode { | ||
89 | type ResolveSyntaxPtrQuery; | ||
90 | // Don't retain syntax trees in memory | ||
91 | storage volatile; | ||
92 | use fn crate::syntax_ptr::resolve_syntax_ptr; | ||
93 | } | ||
90 | } | 94 | } |
91 | } | 95 | } |
92 | 96 | ||
diff --git a/crates/ra_analysis/src/descriptors/mod.rs b/crates/ra_analysis/src/descriptors/mod.rs index c28764336..eaeef54c1 100644 --- a/crates/ra_analysis/src/descriptors/mod.rs +++ b/crates/ra_analysis/src/descriptors/mod.rs | |||
@@ -13,12 +13,12 @@ use crate::{ | |||
13 | descriptors::function::{resolve_local_name, FnId, FnScopes}, | 13 | descriptors::function::{resolve_local_name, FnId, FnScopes}, |
14 | descriptors::module::{ModuleId, ModuleScope, ModuleTree}, | 14 | descriptors::module::{ModuleId, ModuleScope, ModuleTree}, |
15 | input::SourceRootId, | 15 | input::SourceRootId, |
16 | syntax_ptr::{LocalSyntaxPtr, SyntaxPtrDatabase}, | 16 | syntax_ptr::LocalSyntaxPtr, |
17 | Cancelable, FileId, | 17 | Cancelable, FileId, |
18 | }; | 18 | }; |
19 | 19 | ||
20 | salsa::query_group! { | 20 | salsa::query_group! { |
21 | pub(crate) trait DescriptorDatabase: SyntaxDatabase + SyntaxPtrDatabase { | 21 | pub(crate) trait DescriptorDatabase: SyntaxDatabase { |
22 | fn module_tree(source_root_id: SourceRootId) -> Cancelable<Arc<ModuleTree>> { | 22 | fn module_tree(source_root_id: SourceRootId) -> Cancelable<Arc<ModuleTree>> { |
23 | type ModuleTreeQuery; | 23 | type ModuleTreeQuery; |
24 | use fn module::imp::module_tree; | 24 | use fn module::imp::module_tree; |
diff --git a/crates/ra_analysis/src/imp.rs b/crates/ra_analysis/src/imp.rs index 44077b507..0744ea9c8 100644 --- a/crates/ra_analysis/src/imp.rs +++ b/crates/ra_analysis/src/imp.rs | |||
@@ -25,7 +25,6 @@ use crate::{ | |||
25 | }, | 25 | }, |
26 | input::{FilesDatabase, SourceRoot, SourceRootId, WORKSPACE}, | 26 | input::{FilesDatabase, SourceRoot, SourceRootId, WORKSPACE}, |
27 | symbol_index::SymbolIndex, | 27 | symbol_index::SymbolIndex, |
28 | syntax_ptr::SyntaxPtrDatabase, | ||
29 | AnalysisChange, Cancelable, CrateGraph, CrateId, Diagnostic, FileId, FileResolver, | 28 | AnalysisChange, Cancelable, CrateGraph, CrateId, Diagnostic, FileId, FileResolver, |
30 | FileSystemEdit, Position, Query, SourceChange, SourceFileEdit, | 29 | FileSystemEdit, Position, Query, SourceChange, SourceFileEdit, |
31 | }; | 30 | }; |
diff --git a/crates/ra_analysis/src/syntax_ptr.rs b/crates/ra_analysis/src/syntax_ptr.rs index 7b89ecbc8..4db1529c2 100644 --- a/crates/ra_analysis/src/syntax_ptr.rs +++ b/crates/ra_analysis/src/syntax_ptr.rs | |||
@@ -3,17 +3,7 @@ use ra_syntax::{File, SyntaxKind, SyntaxNode, SyntaxNodeRef, TextRange}; | |||
3 | use crate::db::SyntaxDatabase; | 3 | use crate::db::SyntaxDatabase; |
4 | use crate::FileId; | 4 | use crate::FileId; |
5 | 5 | ||
6 | salsa::query_group! { | 6 | pub(crate) fn resolve_syntax_ptr(db: &impl SyntaxDatabase, ptr: SyntaxPtr) -> SyntaxNode { |
7 | pub(crate) trait SyntaxPtrDatabase: SyntaxDatabase { | ||
8 | fn resolve_syntax_ptr(ptr: SyntaxPtr) -> SyntaxNode { | ||
9 | type ResolveSyntaxPtrQuery; | ||
10 | // Don't retain syntax trees in memory | ||
11 | storage volatile; | ||
12 | } | ||
13 | } | ||
14 | } | ||
15 | |||
16 | fn resolve_syntax_ptr(db: &impl SyntaxDatabase, ptr: SyntaxPtr) -> SyntaxNode { | ||
17 | let syntax = db.file_syntax(ptr.file_id); | 7 | let syntax = db.file_syntax(ptr.file_id); |
18 | ptr.local.resolve(&syntax) | 8 | ptr.local.resolve(&syntax) |
19 | } | 9 | } |