andygrove / sqlparser-rs / 233

90%

master: 92%

Pull #86

travis-ci

Allow calling `prev_token()` after EOF

Before this `next_token()` would only increment the index when returning
`Some(token)`. This means that the caller wishing to rewind must be
careful not to call `prev_token()` on EOF (`None`), while not forgetting
to call it for `Some`. Not doing this resulted in bugs in the
undertested code that does error handling.

After making this mistake several times, I'm changing `next_token()` /
`prev_token()` so that calling `next_token(); prev_token(); next_token()`
returns the same token in the first and the last invocation.

31 of 31 new or added lines in 1 file covered. (100.0%)

3347 of 3715 relevant lines covered (90.09%)

0.9 hits per line