]> git.lizzy.rs Git - rust.git/commit
Rollup merge of #25537 - carols10cents:grammar-oops, r=alexcrichton
authorManish Goregaokar <manishsmail@gmail.com>
Mon, 18 May 2015 03:25:38 +0000 (08:55 +0530)
committerManish Goregaokar <manishsmail@gmail.com>
Mon, 18 May 2015 03:25:38 +0000 (08:55 +0530)
commit5ed02b5c0eced7c9e4aa72b2096e537bede31df8
treec8c050b2e23bd046792caaf22df68b981fe7526c
parent252027768eac85cfd98427ba2cbf1d51bfc77d53
parente70c8584d562c86fdca9c98895fb981683d18859
Rollup merge of #25537 - carols10cents:grammar-oops, r=alexcrichton

Hiiii soooo I'm trying to get the reference grammar and associated tests running again, and I swear I tested before but I must have had multiple things going on when I did, because the change I made in #25137 to verify.rs is totally wrong. The RustLexer.tokens file that antlr generates has two sections:

```
EQ=1
LT=2
LE=3
EQEQ=4
NE=5
...
COMMENT=56
SHEBANG=57
UTF8_BOM=58
'='=1
'<'=2
'<='=3
'=='=4
...
```

and verify.rs is only interested in the first half-- the `continue` is to ignore the second half. In 9c7d5ae, I made it panic instead. I was trying to make sure verify.rs handled everything that might happen in the first half and complain if it didn't. That would mean the reference grammar was out of sync with at least verify.rs, if not the real grammar. But it's totally ok for verify.rs to not handle the entire second half of the file.

I'm sorry for breaking this :( Good thing these tests aren't being run regularly yet...? :flushed: