Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test lexer / remove unused functionality #6

Open
Ploppz opened this issue Mar 26, 2017 · 0 comments
Open

Test lexer / remove unused functionality #6

Ploppz opened this issue Mar 26, 2017 · 0 comments
Labels

Comments

@Ploppz
Copy link
Member

Ploppz commented Mar 26, 2017

Generally need tests for the lexer module.

next_word in parser/lexer/mod.rs:

/// Used by next, peek and back - returns substring and new position
/// If forward, places pointer at the next non-whitespace character.
/// If backward, places pointer at the start of the current word.

Needs a test to validate this.

And I don't think that the "previous lexeme"/back functionality is used anyway. Anyway - where back() is used, we can probably use peek() instead.

@Ploppz Ploppz added the todo label Mar 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant