diff options
author | Aleksey Kladov <[email protected]> | 2020-07-31 20:58:36 +0100 |
---|---|---|
committer | Aleksey Kladov <[email protected]> | 2020-07-31 21:04:12 +0100 |
commit | b9c6aa9ec9e491160a6ad7c5ec66151bd67b0ecd (patch) | |
tree | 0ab0318ef1bd2bea1834fbe2ec0c899b5dd6189b | |
parent | 675e86becfab2615528152487db49bce1b43bd60 (diff) |
Unify naming of tuple fields
-rw-r--r-- | crates/ra_assists/src/handlers/early_return.rs | 4 | ||||
-rw-r--r-- | crates/ra_hir_def/src/body/lower.rs | 6 | ||||
-rw-r--r-- | crates/ra_syntax/src/ast/generated/nodes.rs | 8 | ||||
-rw-r--r-- | crates/ra_syntax/src/ast/node_ext.rs | 2 | ||||
-rw-r--r-- | xtask/src/codegen/rust.ungram | 8 |
5 files changed, 14 insertions, 14 deletions
diff --git a/crates/ra_assists/src/handlers/early_return.rs b/crates/ra_assists/src/handlers/early_return.rs index 3650289fd..69852b611 100644 --- a/crates/ra_assists/src/handlers/early_return.rs +++ b/crates/ra_assists/src/handlers/early_return.rs | |||
@@ -51,11 +51,11 @@ pub(crate) fn convert_to_guarded_return(acc: &mut Assists, ctx: &AssistContext) | |||
51 | // Check if there is an IfLet that we can handle. | 51 | // Check if there is an IfLet that we can handle. |
52 | let if_let_pat = match cond.pat() { | 52 | let if_let_pat = match cond.pat() { |
53 | None => None, // No IfLet, supported. | 53 | None => None, // No IfLet, supported. |
54 | Some(ast::Pat::TupleStructPat(pat)) if pat.args().count() == 1 => { | 54 | Some(ast::Pat::TupleStructPat(pat)) if pat.fields().count() == 1 => { |
55 | let path = pat.path()?; | 55 | let path = pat.path()?; |
56 | match path.qualifier() { | 56 | match path.qualifier() { |
57 | None => { | 57 | None => { |
58 | let bound_ident = pat.args().next().unwrap(); | 58 | let bound_ident = pat.fields().next().unwrap(); |
59 | Some((path, bound_ident)) | 59 | Some((path, bound_ident)) |
60 | } | 60 | } |
61 | Some(_) => return None, | 61 | Some(_) => return None, |
diff --git a/crates/ra_hir_def/src/body/lower.rs b/crates/ra_hir_def/src/body/lower.rs index 0d0365370..9d6562c38 100644 --- a/crates/ra_hir_def/src/body/lower.rs +++ b/crates/ra_hir_def/src/body/lower.rs | |||
@@ -496,7 +496,7 @@ impl ExprCollector<'_> { | |||
496 | self.alloc_expr(Expr::BinaryOp { lhs, rhs, op }, syntax_ptr) | 496 | self.alloc_expr(Expr::BinaryOp { lhs, rhs, op }, syntax_ptr) |
497 | } | 497 | } |
498 | ast::Expr::TupleExpr(e) => { | 498 | ast::Expr::TupleExpr(e) => { |
499 | let exprs = e.exprs().map(|expr| self.collect_expr(expr)).collect(); | 499 | let exprs = e.fields().map(|expr| self.collect_expr(expr)).collect(); |
500 | self.alloc_expr(Expr::Tuple { exprs }, syntax_ptr) | 500 | self.alloc_expr(Expr::Tuple { exprs }, syntax_ptr) |
501 | } | 501 | } |
502 | ast::Expr::BoxExpr(e) => { | 502 | ast::Expr::BoxExpr(e) => { |
@@ -762,7 +762,7 @@ impl ExprCollector<'_> { | |||
762 | } | 762 | } |
763 | ast::Pat::TupleStructPat(p) => { | 763 | ast::Pat::TupleStructPat(p) => { |
764 | let path = p.path().and_then(|path| self.expander.parse_path(path)); | 764 | let path = p.path().and_then(|path| self.expander.parse_path(path)); |
765 | let (args, ellipsis) = self.collect_tuple_pat(p.args()); | 765 | let (args, ellipsis) = self.collect_tuple_pat(p.fields()); |
766 | Pat::TupleStruct { path, args, ellipsis } | 766 | Pat::TupleStruct { path, args, ellipsis } |
767 | } | 767 | } |
768 | ast::Pat::RefPat(p) => { | 768 | ast::Pat::RefPat(p) => { |
@@ -780,7 +780,7 @@ impl ExprCollector<'_> { | |||
780 | } | 780 | } |
781 | ast::Pat::ParenPat(p) => return self.collect_pat_opt(p.pat()), | 781 | ast::Pat::ParenPat(p) => return self.collect_pat_opt(p.pat()), |
782 | ast::Pat::TuplePat(p) => { | 782 | ast::Pat::TuplePat(p) => { |
783 | let (args, ellipsis) = self.collect_tuple_pat(p.args()); | 783 | let (args, ellipsis) = self.collect_tuple_pat(p.fields()); |
784 | Pat::Tuple { args, ellipsis } | 784 | Pat::Tuple { args, ellipsis } |
785 | } | 785 | } |
786 | ast::Pat::WildcardPat(_) => Pat::Wild, | 786 | ast::Pat::WildcardPat(_) => Pat::Wild, |
diff --git a/crates/ra_syntax/src/ast/generated/nodes.rs b/crates/ra_syntax/src/ast/generated/nodes.rs index 6cb637b1d..5f51c7536 100644 --- a/crates/ra_syntax/src/ast/generated/nodes.rs +++ b/crates/ra_syntax/src/ast/generated/nodes.rs | |||
@@ -893,7 +893,7 @@ pub struct TupleExpr { | |||
893 | impl ast::AttrsOwner for TupleExpr {} | 893 | impl ast::AttrsOwner for TupleExpr {} |
894 | impl TupleExpr { | 894 | impl TupleExpr { |
895 | pub fn l_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T!['(']) } | 895 | pub fn l_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T!['(']) } |
896 | pub fn exprs(&self) -> AstChildren<Expr> { support::children(&self.syntax) } | 896 | pub fn fields(&self) -> AstChildren<Expr> { support::children(&self.syntax) } |
897 | pub fn r_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T![')']) } | 897 | pub fn r_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T![')']) } |
898 | } | 898 | } |
899 | #[derive(Debug, Clone, PartialEq, Eq, Hash)] | 899 | #[derive(Debug, Clone, PartialEq, Eq, Hash)] |
@@ -1210,7 +1210,7 @@ pub struct SlicePat { | |||
1210 | } | 1210 | } |
1211 | impl SlicePat { | 1211 | impl SlicePat { |
1212 | pub fn l_brack_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T!['[']) } | 1212 | pub fn l_brack_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T!['[']) } |
1213 | pub fn args(&self) -> AstChildren<Pat> { support::children(&self.syntax) } | 1213 | pub fn pats(&self) -> AstChildren<Pat> { support::children(&self.syntax) } |
1214 | pub fn r_brack_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T![']']) } | 1214 | pub fn r_brack_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T![']']) } |
1215 | } | 1215 | } |
1216 | #[derive(Debug, Clone, PartialEq, Eq, Hash)] | 1216 | #[derive(Debug, Clone, PartialEq, Eq, Hash)] |
@@ -1219,7 +1219,7 @@ pub struct TuplePat { | |||
1219 | } | 1219 | } |
1220 | impl TuplePat { | 1220 | impl TuplePat { |
1221 | pub fn l_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T!['(']) } | 1221 | pub fn l_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T!['(']) } |
1222 | pub fn args(&self) -> AstChildren<Pat> { support::children(&self.syntax) } | 1222 | pub fn fields(&self) -> AstChildren<Pat> { support::children(&self.syntax) } |
1223 | pub fn r_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T![')']) } | 1223 | pub fn r_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T![')']) } |
1224 | } | 1224 | } |
1225 | #[derive(Debug, Clone, PartialEq, Eq, Hash)] | 1225 | #[derive(Debug, Clone, PartialEq, Eq, Hash)] |
@@ -1229,7 +1229,7 @@ pub struct TupleStructPat { | |||
1229 | impl TupleStructPat { | 1229 | impl TupleStructPat { |
1230 | pub fn path(&self) -> Option<Path> { support::child(&self.syntax) } | 1230 | pub fn path(&self) -> Option<Path> { support::child(&self.syntax) } |
1231 | pub fn l_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T!['(']) } | 1231 | pub fn l_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T!['(']) } |
1232 | pub fn args(&self) -> AstChildren<Pat> { support::children(&self.syntax) } | 1232 | pub fn fields(&self) -> AstChildren<Pat> { support::children(&self.syntax) } |
1233 | pub fn r_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T![')']) } | 1233 | pub fn r_paren_token(&self) -> Option<SyntaxToken> { support::token(&self.syntax, T![')']) } |
1234 | } | 1234 | } |
1235 | #[derive(Debug, Clone, PartialEq, Eq, Hash)] | 1235 | #[derive(Debug, Clone, PartialEq, Eq, Hash)] |
diff --git a/crates/ra_syntax/src/ast/node_ext.rs b/crates/ra_syntax/src/ast/node_ext.rs index 2ffb83819..733e97877 100644 --- a/crates/ra_syntax/src/ast/node_ext.rs +++ b/crates/ra_syntax/src/ast/node_ext.rs | |||
@@ -290,7 +290,7 @@ pub struct SlicePatComponents { | |||
290 | 290 | ||
291 | impl ast::SlicePat { | 291 | impl ast::SlicePat { |
292 | pub fn components(&self) -> SlicePatComponents { | 292 | pub fn components(&self) -> SlicePatComponents { |
293 | let mut args = self.args().peekable(); | 293 | let mut args = self.pats().peekable(); |
294 | let prefix = args | 294 | let prefix = args |
295 | .peeking_take_while(|p| match p { | 295 | .peeking_take_while(|p| match p { |
296 | ast::Pat::RestPat(_) => false, | 296 | ast::Pat::RestPat(_) => false, |
diff --git a/xtask/src/codegen/rust.ungram b/xtask/src/codegen/rust.ungram index b14801f24..aca23890c 100644 --- a/xtask/src/codegen/rust.ungram +++ b/xtask/src/codegen/rust.ungram | |||
@@ -359,7 +359,7 @@ IndexExpr = | |||
359 | Attr* base:Expr '[' index:Expr ']' | 359 | Attr* base:Expr '[' index:Expr ']' |
360 | 360 | ||
361 | TupleExpr = | 361 | TupleExpr = |
362 | Attr* '(' Attr* (Expr (',' Expr)* ','?)? ')' | 362 | Attr* '(' Attr* fields:(Expr (',' Expr)* ','?)? ')' |
363 | 363 | ||
364 | RecordExpr = | 364 | RecordExpr = |
365 | Path RecordExprFieldList | 365 | Path RecordExprFieldList |
@@ -560,16 +560,16 @@ RecordPatField = | |||
560 | Attr* (NameRef ':')? Pat | 560 | Attr* (NameRef ':')? Pat |
561 | 561 | ||
562 | TupleStructPat = | 562 | TupleStructPat = |
563 | Path '(' args:(Pat (',' Pat)* ','?)? ')' | 563 | Path '(' fields:(Pat (',' Pat)* ','?)? ')' |
564 | 564 | ||
565 | TuplePat = | 565 | TuplePat = |
566 | '(' args:(Pat (',' Pat)* ','?)? ')' | 566 | '(' fields:(Pat (',' Pat)* ','?)? ')' |
567 | 567 | ||
568 | ParenPat = | 568 | ParenPat = |
569 | '(' Pat ')' | 569 | '(' Pat ')' |
570 | 570 | ||
571 | SlicePat = | 571 | SlicePat = |
572 | '[' args:(Pat (',' Pat)* ','?)? ']' | 572 | '[' (Pat (',' Pat)* ','?)? ']' |
573 | 573 | ||
574 | PathPat = | 574 | PathPat = |
575 | Path | 575 | Path |