aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--crates/ra_parser/src/grammar/expressions.rs70
-rw-r--r--crates/ra_parser/src/grammar/items.rs84
2 files changed, 70 insertions, 84 deletions
diff --git a/crates/ra_parser/src/grammar/expressions.rs b/crates/ra_parser/src/grammar/expressions.rs
index 5a3b9f589..ccbc905ab 100644
--- a/crates/ra_parser/src/grammar/expressions.rs
+++ b/crates/ra_parser/src/grammar/expressions.rs
@@ -62,47 +62,41 @@ pub(crate) fn expr_block_contents(p: &mut Parser) {
62 continue; 62 continue;
63 } 63 }
64 64
65 match items::maybe_item(p, items::ItemFlavor::Mod) { 65 let m = match items::maybe_item(p, m, items::ItemFlavor::Mod) {
66 items::MaybeItem::Item(kind) => { 66 Ok(()) => continue,
67 m.complete(p, kind); 67 Err(m) => m,
68 } 68 };
69 items::MaybeItem::Modifiers => { 69
70 // test pub_expr
71 // fn foo() { pub 92; } //FIXME
72 if has_attrs {
73 m.abandon(p);
74 p.error("expected a let statement or an item after attributes in block");
75 } else {
76 let is_blocklike = expressions::expr_stmt(p) == BlockLike::Block;
77 if p.at(R_CURLY) {
70 m.abandon(p); 78 m.abandon(p);
71 p.error("expected an item"); 79 } else {
72 } 80 // test no_semi_after_block
73 // test pub_expr 81 // fn foo() {
74 // fn foo() { pub 92; } //FIXME 82 // if true {}
75 items::MaybeItem::None => { 83 // loop {}
76 if has_attrs { 84 // match () {}
77 m.abandon(p); 85 // while true {}
78 p.error("expected a let statement or an item after attributes in block"); 86 // for _ in () {}
87 // {}
88 // {}
89 // macro_rules! test {
90 // () => {}
91 // }
92 // test!{}
93 // }
94 if is_blocklike {
95 p.eat(SEMI);
79 } else { 96 } else {
80 let is_blocklike = expressions::expr_stmt(p) == BlockLike::Block; 97 p.expect(SEMI);
81 if p.at(R_CURLY) {
82 m.abandon(p);
83 } else {
84 // test no_semi_after_block
85 // fn foo() {
86 // if true {}
87 // loop {}
88 // match () {}
89 // while true {}
90 // for _ in () {}
91 // {}
92 // {}
93 // macro_rules! test {
94 // () => {}
95 // }
96 // test!{}
97 // }
98 if is_blocklike {
99 p.eat(SEMI);
100 } else {
101 p.expect(SEMI);
102 }
103 m.complete(p, EXPR_STMT);
104 }
105 } 98 }
99 m.complete(p, EXPR_STMT);
106 } 100 }
107 } 101 }
108 } 102 }
diff --git a/crates/ra_parser/src/grammar/items.rs b/crates/ra_parser/src/grammar/items.rs
index a057c8167..94b93a02b 100644
--- a/crates/ra_parser/src/grammar/items.rs
+++ b/crates/ra_parser/src/grammar/items.rs
@@ -37,52 +37,40 @@ pub(super) const ITEM_RECOVERY_SET: TokenSet = token_set![
37pub(super) fn item_or_macro(p: &mut Parser, stop_on_r_curly: bool, flavor: ItemFlavor) { 37pub(super) fn item_or_macro(p: &mut Parser, stop_on_r_curly: bool, flavor: ItemFlavor) {
38 let m = p.start(); 38 let m = p.start();
39 attributes::outer_attributes(p); 39 attributes::outer_attributes(p);
40 match maybe_item(p, flavor) { 40 let m = match maybe_item(p, m, flavor) {
41 MaybeItem::Item(kind) => { 41 Ok(()) => return,
42 m.complete(p, kind); 42 Err(m) => m,
43 } 43 };
44 MaybeItem::None => { 44 if paths::is_path_start(p) {
45 if paths::is_path_start(p) { 45 match macro_call(p) {
46 match macro_call(p) { 46 BlockLike::Block => (),
47 BlockLike::Block => (), 47 BlockLike::NotBlock => {
48 BlockLike::NotBlock => { 48 p.expect(SEMI);
49 p.expect(SEMI);
50 }
51 }
52 m.complete(p, MACRO_CALL);
53 } else {
54 m.abandon(p);
55 if p.at(L_CURLY) {
56 error_block(p, "expected an item");
57 } else if p.at(R_CURLY) && !stop_on_r_curly {
58 let e = p.start();
59 p.error("unmatched `}`");
60 p.bump();
61 e.complete(p, ERROR);
62 } else if !p.at(EOF) && !p.at(R_CURLY) {
63 p.err_and_bump("expected an item");
64 } else {
65 p.error("expected an item");
66 }
67 } 49 }
68 } 50 }
69 MaybeItem::Modifiers => { 51 m.complete(p, MACRO_CALL);
70 p.error("expected fn, trait or impl"); 52 } else {
71 m.complete(p, ERROR); 53 m.abandon(p);
54 if p.at(L_CURLY) {
55 error_block(p, "expected an item");
56 } else if p.at(R_CURLY) && !stop_on_r_curly {
57 let e = p.start();
58 p.error("unmatched `}`");
59 p.bump();
60 e.complete(p, ERROR);
61 } else if !p.at(EOF) && !p.at(R_CURLY) {
62 p.err_and_bump("expected an item");
63 } else {
64 p.error("expected an item");
72 } 65 }
73 } 66 }
74} 67}
75 68
76pub(super) enum MaybeItem { 69pub(super) fn maybe_item(p: &mut Parser, m: Marker, flavor: ItemFlavor) -> Result<(), Marker> {
77 None,
78 Item(SyntaxKind),
79 Modifiers,
80}
81
82pub(super) fn maybe_item(p: &mut Parser, flavor: ItemFlavor) -> MaybeItem {
83 opt_visibility(p); 70 opt_visibility(p);
84 if let Some(kind) = items_without_modifiers(p) { 71 if let Some(kind) = items_without_modifiers(p) {
85 return MaybeItem::Item(kind); 72 m.complete(p, kind);
73 return Ok(());
86 } 74 }
87 75
88 let mut has_mods = false; 76 let mut has_mods = false;
@@ -115,7 +103,7 @@ pub(super) fn maybe_item(p: &mut Parser, flavor: ItemFlavor) -> MaybeItem {
115 } 103 }
116 104
117 // items 105 // items
118 let kind = match p.current() { 106 match p.current() {
119 // test async_fn 107 // test async_fn
120 // async fn foo() {} 108 // async fn foo() {}
121 109
@@ -135,7 +123,7 @@ pub(super) fn maybe_item(p: &mut Parser, flavor: ItemFlavor) -> MaybeItem {
135 // unsafe fn foo() {} 123 // unsafe fn foo() {}
136 FN_KW => { 124 FN_KW => {
137 fn_def(p, flavor); 125 fn_def(p, flavor);
138 FN_DEF 126 m.complete(p, FN_DEF);
139 } 127 }
140 128
141 // test unsafe_trait 129 // test unsafe_trait
@@ -148,7 +136,7 @@ pub(super) fn maybe_item(p: &mut Parser, flavor: ItemFlavor) -> MaybeItem {
148 // unsafe auto trait T {} 136 // unsafe auto trait T {}
149 TRAIT_KW => { 137 TRAIT_KW => {
150 traits::trait_def(p); 138 traits::trait_def(p);
151 TRAIT_DEF 139 m.complete(p, TRAIT_DEF);
152 } 140 }
153 141
154 // test unsafe_impl 142 // test unsafe_impl
@@ -161,14 +149,18 @@ pub(super) fn maybe_item(p: &mut Parser, flavor: ItemFlavor) -> MaybeItem {
161 // unsafe default impl Foo {} 149 // unsafe default impl Foo {}
162 IMPL_KW => { 150 IMPL_KW => {
163 traits::impl_block(p); 151 traits::impl_block(p);
164 IMPL_BLOCK 152 m.complete(p, IMPL_BLOCK);
165 } 153 }
166 _ => { 154 _ => {
167 return if has_mods { MaybeItem::Modifiers } else { MaybeItem::None }; 155 if !has_mods {
156 return Err(m);
157 } else {
158 p.error("expected fn, trait or impl");
159 m.complete(p, ERROR);
160 }
168 } 161 }
169 }; 162 }
170 163 Ok(())
171 MaybeItem::Item(kind)
172} 164}
173 165
174fn items_without_modifiers(p: &mut Parser) -> Option<SyntaxKind> { 166fn items_without_modifiers(p: &mut Parser) -> Option<SyntaxKind> {