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

Rewrite lexer and parser and overhaul general syntax #51

Merged
merged 42 commits into from
Apr 10, 2024
Merged

Conversation

kevlarr
Copy link
Collaborator

@kevlarr kevlarr commented Jul 19, 2022

This turned out to be a much larger overhaul than I initially set out to do. Originally it was about cleaning the existing lexer up so that it only concerns itself with valid tokens and not enforcing which tokens are expected next, but after coming back from a very long hiatus it rapidly turned into a "change all the syntax things" situation and, well, it's pretty comprehensive...

Notable changes include, in no particular order:

Incidentally it also addresses the following bugs:

And last, and probably least, it resolves #50

TODO

  • Rewrite lexer
  • Rewrite parser
  • Rewrite analyzer
  • Rewrite loader
  • Enrich lexer & parser error messages with positions
  • Fix tests
  • Review TODO and FIXME comments

@kevlarr kevlarr marked this pull request as draft March 19, 2024 13:51
@kevlarr kevlarr changed the title WIP: Clean up lexer and parser Rewrite lexer and parser Mar 19, 2024
@kevlarr kevlarr changed the title Rewrite lexer and parser Rewrite lexer and parser and overhaul general syntax Mar 28, 2024
@kevlarr kevlarr marked this pull request as ready for review April 10, 2024 23:26
@kevlarr kevlarr merged commit 06b4bb4 into master Apr 10, 2024
@kevlarr kevlarr deleted the clean-up-lexer branch April 10, 2024 23:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant