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.

Sign up for GitHub

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

Jump to bottom

Parse attributes rather than just bool #14

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Parse attributes rather than just bool #14

wants to merge 3 commits into from

Conversation

Quincunx271
Copy link
Owner

For tracking the work in parsing attributes.

Closes #11

@Quincunx271 Quincunx271 added the enhancement New feature or request label Feb 3, 2022
@Quincunx271
Add the basic idea of the AST types
2479059
@Quincunx271
Add leaf nodes to the runtime AST
c6c07da
@Quincunx271
LL(1) parse an AST
120b419 
This "AST" is more accurately described as a parse tree. The basic idea
was to accumulate the attributes on a separate stack, but make note of
when we needed to merge the tree node into the parent node by adding a
sentinel token to the LL(1) parsing stack.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

The CFG really needs to be an attribute grammar
1 participant
@Quincunx271