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
add more consistency in ast #523
Conversation
Pull Request Test Coverage Report for Build 2491105118
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @frolovdev -- looks like a nice cleanup.
cc @andygrove
src/parser.rs
Outdated
match self.peek_token() { | ||
Token::Word(w) => match w.keyword { | ||
Keyword::EXISTS => { | ||
let negated = self.parse_keyword(Keyword::EXISTS); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
don't we know here that negated
is true? I think the code might be clearer if this were something more like
let negated = true;
self.parse_keyword(Keyword::Exists);
self.parse_exists_expr(negated)
What do you think?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I will make an update in a minute
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made an update but use placeholder (let _
) because of clippy checks
Looks good -- thanks @frolovdev |
* add more consistency in ast * refactor styling
Closes #472