let-chains parsing is not resilient to stray `;` between `let`s

[estebank]

Code

#![feature(let_chains)]
fn main() {
    if let () = ()
        && let () = ();
        && let () = ()
    {
    }
}

Current output

error: expected `{`, found `;`
 --> src/main.rs:4:23
  |
4 |         && let () = ();
  |                       ^ expected `{`
  |
note: the `if` expression is missing a block after this condition
 --> src/main.rs:3:8
  |
3 |       if let () = ()
  |  ________^
4 | |         && let () = ();
  | |______________________^

Desired output

error: expected `{`, found `;`
 --> src/main.rs:4:23
  |
4 |         && let () = ();
  |                       ^
  |                       |
  |                       expected `{`
  |                       help: remove it to parse the following `let` as part of the same chain
5 |         && let () = ()
  |         -------------- you likely meant to continue parsing the let-chain here

Rationale and extra context

This is a common typo I make when hoisting an inconditional let binding into the let chain to merge two let-chains that could be one. The parser only needs to do look-ahead 1 and 2 to detect the case and recover.

Other cases

No response

Anything else?

No response

[sjwang05]

@rustbot claim