G: value_parameters, patterns & let statement

grammar.ron

@@ -27,6 +27,8 @@ Grammar(
         "mut",
         "unsafe",
         "type",
+        "ref",
+        "let",
     ],
     contextual_keywords: [
         "auto",
@@ -113,6 +115,12 @@ Grammar(
         "FN_POINTER_TYPE",
         "FOR_TYPE",
 
+        "REF_PAT",
+        "BIND_PAT",
+        "PLACEHOLDER_PAT",
+
+        "TUPLE_EXPR",
+
         "EXTERN_BLOCK",
         "ENUM_VARIANT",
         "NAMED_FIELD",
@@ -132,5 +140,8 @@ Grammar(
         "ABI",
         "NAME",
         "NAME_REF",
+        "VALUE_PARAMETER",
+        "BLOCK",
+        "LET_STMT",
     ]
 )

src/parser/grammar/expressions.rs

@@ -14,7 +14,20 @@ pub(super) fn literal(p: &mut Parser) -> bool {
 }
 
 pub(super) fn expr(p: &mut Parser) {
-    if !literal(p) {
-        p.error("expected expression");
+    if literal(p) {
+        return;
     }
+
+    match p.current() {
+        L_PAREN => tuple_expr(p),
+        _ => p.error("expected expression"),
+    }
+}
+
+fn tuple_expr(p: &mut Parser) {
+    assert!(p.at(L_PAREN));
+    let m = p.start();
+    p.expect(L_PAREN);
+    p.expect(R_PAREN);
+    m.complete(p, TUPLE_EXPR);
 }

src/parser/grammar/items/mod.rs

@@ -218,9 +218,33 @@ fn fn_item(p: &mut Parser) {
         p.error("expected function arguments");
     }
 
-    if p.at(L_CURLY) {
-        p.expect(L_CURLY);
+    block(p);
+
+    fn block(p: &mut Parser) {
+        if !p.at(L_CURLY) {
+            p.error("expected block");
+        }
+        let m = p.start();
+        p.bump();
+        while !p.at(EOF) && !p.at(R_CURLY) {
+            match p.current() {
+                LET_KW => let_stmt(p),
+                _ => p.err_and_bump("expected statement"),
+            }
+        }
         p.expect(R_CURLY);
+        m.complete(p, BLOCK);
+    }
+
+    fn let_stmt(p: &mut Parser) {
+        assert!(p.at(LET_KW));
+        let m = p.start();
+        p.bump();
+        patterns::pattern(p);
+        p.expect(EQ);
+        expressions::expr(p);
+        p.expect(SEMI);
+        m.complete(p, LET_STMT);
     }
 }
 

src/parser/grammar/mod.rs

@@ -30,6 +30,7 @@ mod items;
 mod attributes;
 mod expressions;
 mod types;
+mod patterns;
 mod paths;
 mod type_params;
 
@@ -85,10 +86,29 @@ fn abi(p: &mut Parser) {
     abi.complete(p, ABI);
 }
 
+// test fn_value_parameters
+// fn a() {}
+// fn b(x: i32) {}
+// fn c(x: i32, ) {}
+// fn d(x: i32, y: ()) {}
 fn fn_value_parameters(p: &mut Parser) {
     assert!(p.at(L_PAREN));
     p.bump();
+    while !p.at(EOF) && !p.at(R_PAREN) {
+        value_parameter(p);
+        if !p.at(R_PAREN) {
+            p.expect(COMMA);
+        }
+    }
     p.expect(R_PAREN);
+
+    fn value_parameter(p: &mut Parser) {
+        let m = p.start();
+        patterns::pattern(p);
+        p.expect(COLON);
+        types::type_(p);
+        m.complete(p, VALUE_PARAMETER);
+    }
 }
 
 fn fn_ret_type(p: &mut Parser) {

src/parser/grammar/patterns.rs

@@ -0,0 +1,52 @@
+use super::*;
+
+pub(super) fn pattern(p: &mut Parser) {
+    match p.current() {
+        UNDERSCORE => placeholder_pat(p),
+        AMPERSAND => ref_pat(p),
+        IDENT | REF_KW => bind_pat(p),
+        _ => p.err_and_bump("expected pattern"),
+    }
+}
+
+// test placeholder_pat
+// fn main() { let _ = (); }
+fn placeholder_pat(p: &mut Parser) {
+    assert!(p.at(UNDERSCORE));
+    let m = p.start();
+    p.bump();
+    m.complete(p, PLACEHOLDER_PAT);
+}
+
+// test ref_pat
+// fn main() {
+//     let &a = ();
+//     let &mut b = ();
+// }
+fn ref_pat(p: &mut Parser) {
+    assert!(p.at(AMPERSAND));
+    let m = p.start();
+    p.bump();
+    p.eat(MUT_KW);
+    pattern(p);
+    m.complete(p, REF_PAT);
+}
+
+// test bind_pat
+// fn main() {
+//     let a = ();
+//     let ref b = ();
+//     let ref mut c = ();
+//     let d @ _ = ();
+// }
+fn bind_pat(p: &mut Parser) {
+    let m = p.start();
+    if p.eat(REF_KW) {
+        p.eat(MUT_KW);
+    }
+    name(p);
+    if p.eat(AT) {
+        pattern(p);
+    }
+    m.complete(p, BIND_PAT);
+}

src/syntax_kinds.rs

@@ -84,6 +84,8 @@ pub enum SyntaxKind {
     MUT_KW,
     UNSAFE_KW,
     TYPE_KW,
+    REF_KW,
+    LET_KW,
     AUTO_KW,
     DEFAULT_KW,
     UNION_KW,
@@ -110,6 +112,10 @@ pub enum SyntaxKind {
     PLACEHOLDER_TYPE,
     FN_POINTER_TYPE,
     FOR_TYPE,
+    REF_PAT,
+    BIND_PAT,
+    PLACEHOLDER_PAT,
+    TUPLE_EXPR,
     EXTERN_BLOCK,
     ENUM_VARIANT,
     NAMED_FIELD,
@@ -129,6 +135,9 @@ pub enum SyntaxKind {
     ABI,
     NAME,
     NAME_REF,
+    VALUE_PARAMETER,
+    BLOCK,
+    LET_STMT,
 
     // Technical SyntaxKinds: they appear temporally during parsing,
     // but never end up in the final tree
@@ -220,6 +229,8 @@ impl SyntaxKind {
             MUT_KW => &SyntaxInfo { name: "MUT_KW" },
             UNSAFE_KW => &SyntaxInfo { name: "UNSAFE_KW" },
             TYPE_KW => &SyntaxInfo { name: "TYPE_KW" },
+            REF_KW => &SyntaxInfo { name: "REF_KW" },
+            LET_KW => &SyntaxInfo { name: "LET_KW" },
             AUTO_KW => &SyntaxInfo { name: "AUTO_KW" },
             DEFAULT_KW => &SyntaxInfo { name: "DEFAULT_KW" },
             UNION_KW => &SyntaxInfo { name: "UNION_KW" },
@@ -246,6 +257,10 @@ impl SyntaxKind {
             PLACEHOLDER_TYPE => &SyntaxInfo { name: "PLACEHOLDER_TYPE" },
             FN_POINTER_TYPE => &SyntaxInfo { name: "FN_POINTER_TYPE" },
             FOR_TYPE => &SyntaxInfo { name: "FOR_TYPE" },
+            REF_PAT => &SyntaxInfo { name: "REF_PAT" },
+            BIND_PAT => &SyntaxInfo { name: "BIND_PAT" },
+            PLACEHOLDER_PAT => &SyntaxInfo { name: "PLACEHOLDER_PAT" },
+            TUPLE_EXPR => &SyntaxInfo { name: "TUPLE_EXPR" },
             EXTERN_BLOCK => &SyntaxInfo { name: "EXTERN_BLOCK" },
             ENUM_VARIANT => &SyntaxInfo { name: "ENUM_VARIANT" },
             NAMED_FIELD => &SyntaxInfo { name: "NAMED_FIELD" },
@@ -265,6 +280,9 @@ impl SyntaxKind {
             ABI => &SyntaxInfo { name: "ABI" },
             NAME => &SyntaxInfo { name: "NAME" },
             NAME_REF => &SyntaxInfo { name: "NAME_REF" },
+            VALUE_PARAMETER => &SyntaxInfo { name: "VALUE_PARAMETER" },
+            BLOCK => &SyntaxInfo { name: "BLOCK" },
+            LET_STMT => &SyntaxInfo { name: "LET_STMT" },
 
             TOMBSTONE => &SyntaxInfo { name: "TOMBSTONE" },
             EOF => &SyntaxInfo { name: "EOF" },
@@ -301,6 +319,8 @@ pub(crate) fn ident_to_keyword(ident: &str) -> Option<SyntaxKind> {
         "mut" => Some(MUT_KW),
         "unsafe" => Some(UNSAFE_KW),
         "type" => Some(TYPE_KW),
+        "ref" => Some(REF_KW),
+        "let" => Some(LET_KW),
         _ => None,
     }
 }

tests/data/lexer/0011_keywords.rs

@@ -1,3 +1,3 @@
 fn use struct trait enum impl true false as extern crate
 mod pub self super in where for loop while if match const
-static mut type
+static mut type ref let

tests/data/lexer/0011_keywords.txt

@@ -49,4 +49,8 @@ WHITESPACE 1 " "
 MUT_KW 3 "mut"
 WHITESPACE 1 " "
 TYPE_KW 4 "type"
+WHITESPACE 1 " "
+REF_KW 3 "ref"
+WHITESPACE 1 " "
+LET_KW 3 "let"
 WHITESPACE 1 "\n"

tests/data/parser/err/0005_attribute_recover.txt

@@ -28,11 +28,12 @@ FILE@[0; 54)
       IDENT@[22; 25) "foo"
     L_PAREN@[25; 26)
     R_PAREN@[26; 27)
-    WHITESPACE@[27; 28)
-    L_CURLY@[28; 29)
-    WHITESPACE@[29; 30)
-    R_CURLY@[30; 31)
-    WHITESPACE@[31; 34)
+    BLOCK@[27; 34)
+      WHITESPACE@[27; 28)
+      L_CURLY@[28; 29)
+      WHITESPACE@[29; 30)
+      R_CURLY@[30; 31)
+      WHITESPACE@[31; 34)
   FN_ITEM@[34; 54)
     ATTR@[34; 41)
       POUND@[34; 35)
@@ -49,8 +50,9 @@ FILE@[0; 54)
       IDENT@[44; 47) "foo"
     L_PAREN@[47; 48)
     R_PAREN@[48; 49)
-    WHITESPACE@[49; 50)
-    L_CURLY@[50; 51)
-    WHITESPACE@[51; 52)
-    R_CURLY@[52; 53)
-    WHITESPACE@[53; 54)
+    BLOCK@[49; 54)
+      WHITESPACE@[49; 50)
+      L_CURLY@[50; 51)
+      WHITESPACE@[51; 52)
+      R_CURLY@[52; 53)
+      WHITESPACE@[53; 54)

tests/data/parser/err/0007_stray_curly_in_file.txt

@@ -21,9 +21,10 @@ FILE@[0; 31)
       IDENT@[20; 23) "foo"
     L_PAREN@[23; 24)
     R_PAREN@[24; 25)
-    L_CURLY@[25; 26)
-    R_CURLY@[26; 27)
-    WHITESPACE@[27; 29)
+    BLOCK@[25; 29)
+      L_CURLY@[25; 26)
+      R_CURLY@[26; 27)
+      WHITESPACE@[27; 29)
   ERROR@[29; 31)
   err: `expected item`
     R_CURLY@[29; 30)

tests/data/parser/err/0008_item_block_recovery.txt

@@ -6,11 +6,12 @@ FILE@[0; 95)
       IDENT@[3; 6) "foo"
     L_PAREN@[6; 7)
     R_PAREN@[7; 8)
-    WHITESPACE@[8; 9)
-    L_CURLY@[9; 10)
-    WHITESPACE@[10; 11)
-    R_CURLY@[11; 12)
-    WHITESPACE@[12; 14)
+    BLOCK@[8; 14)
+      WHITESPACE@[8; 9)
+      L_CURLY@[9; 10)
+      WHITESPACE@[10; 11)
+      R_CURLY@[11; 12)
+      WHITESPACE@[12; 14)
   ERROR@[14; 17)
   err: `expected item`
     IDENT@[14; 17) "bar"
@@ -56,8 +57,9 @@ FILE@[0; 95)
       IDENT@[85; 88) "baz"
     L_PAREN@[88; 89)
     R_PAREN@[89; 90)
-    WHITESPACE@[90; 91)
-    L_CURLY@[91; 92)
-    WHITESPACE@[92; 93)
-    R_CURLY@[93; 94)
-    WHITESPACE@[94; 95)
+    BLOCK@[90; 95)
+      WHITESPACE@[90; 91)
+      L_CURLY@[91; 92)
+      WHITESPACE@[92; 93)
+      R_CURLY@[93; 94)
+      WHITESPACE@[94; 95)

tests/data/parser/inline/0001_const_unsafe_fn.txt

@@ -10,7 +10,8 @@ FILE@[0; 25)
       IDENT@[16; 19) "foo"
     L_PAREN@[19; 20)
     R_PAREN@[20; 21)
-    WHITESPACE@[21; 22)
-    L_CURLY@[22; 23)
-    R_CURLY@[23; 24)
-    WHITESPACE@[24; 25)
+    BLOCK@[21; 25)
+      WHITESPACE@[21; 22)
+      L_CURLY@[22; 23)
+      R_CURLY@[23; 24)
+      WHITESPACE@[24; 25)

tests/data/parser/inline/0002_const_fn.txt

@@ -8,7 +8,8 @@ FILE@[0; 18)
       IDENT@[9; 12) "foo"
     L_PAREN@[12; 13)
     R_PAREN@[13; 14)
-    WHITESPACE@[14; 15)
-    L_CURLY@[15; 16)
-    R_CURLY@[16; 17)
-    WHITESPACE@[17; 18)
+    BLOCK@[14; 18)
+      WHITESPACE@[14; 15)
+      L_CURLY@[15; 16)
+      R_CURLY@[16; 17)
+      WHITESPACE@[17; 18)

tests/data/parser/inline/0004_extern_fn.txt

@@ -9,7 +9,8 @@ FILE@[0; 19)
       IDENT@[10; 13) "foo"
     L_PAREN@[13; 14)
     R_PAREN@[14; 15)
-    WHITESPACE@[15; 16)
-    L_CURLY@[16; 17)
-    R_CURLY@[17; 18)
-    WHITESPACE@[18; 19)
+    BLOCK@[15; 19)
+      WHITESPACE@[15; 16)
+      L_CURLY@[16; 17)
+      R_CURLY@[17; 18)
+      WHITESPACE@[18; 19)

tests/data/parser/inline/0011_unsafe_fn.txt

@@ -8,7 +8,8 @@ FILE@[0; 19)
       IDENT@[10; 13) "foo"
     L_PAREN@[13; 14)
     R_PAREN@[14; 15)
-    WHITESPACE@[15; 16)
-    L_CURLY@[16; 17)
-    R_CURLY@[17; 18)
-    WHITESPACE@[18; 19)
+    BLOCK@[15; 19)
+      WHITESPACE@[15; 16)
+      L_CURLY@[16; 17)
+      R_CURLY@[17; 18)
+      WHITESPACE@[18; 19)