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

ast: remove necessity for 'Ω's #30

Open
DeedleFake opened this issue Oct 18, 2017 · 2 comments
Open

ast: remove necessity for 'Ω's #30

DeedleFake opened this issue Oct 18, 2017 · 2 comments

Comments

@DeedleFake
Copy link
Owner

It should not be necessary to specify where the EOF is expected in the grammar. This was originally a workaround, but it's now causing problems with expression evaluation. In particular, the expression parser is incapable of parsing compounds at the moment.

I've kind of started working on fixing this in the dev/omega branch, but I'm not sure what exactly I've done wrong. This might require a fairly heavy refactoring of the parser code, although I hope it doesn't.

@DeedleFake
Copy link
Owner Author

Depending on how this works out, this might remove the need for a separate grammar for parsing standalone expressions, which could, in turn, remove the Go 1.9 requirement.

@DeedleFake
Copy link
Owner Author

This will likely become basically unecessary once #53 is done.

@DeedleFake DeedleFake moved this from Doing to Percolating in Initial Implementation Mar 12, 2018
@DeedleFake DeedleFake changed the title Remove Necessity for 'Ω's in the Grammars res/grammar: remove necessity for 'Ω's Nov 28, 2018
@DeedleFake DeedleFake changed the title res/grammar: remove necessity for 'Ω's ast: remove necessity for 'Ω's Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Initial Implementation
  
Percolating
Development

No branches or pull requests

1 participant