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

astPP needs to parenthesise nested case expressions #16

Closed
xrchz opened this issue Mar 5, 2015 · 2 comments
Closed

astPP needs to parenthesise nested case expressions #16

xrchz opened this issue Mar 5, 2015 · 2 comments
Assignees
@tanyongkiam

Comments

@xrchz
Copy link
Member

xrchz commented Mar 5, 2015

nested case expressions are often ambiguous to parse; the pretty-printer should include enough parentheses for disambiguous parsing.
@tanyongkiam tanyongkiam self-assigned this Mar 5, 2015
@tanyongkiam
Copy link
Contributor

This is partially fixed on the reg_alloc branch. However, I think there might be the same problem with letrecs (not completely sure).

@xrchz
Copy link
Member Author

xrchz commented May 25, 2015

Although I'm not completely certain, I think the case expression problem is now fixed (now that reg_alloc has been merged into master). (No idea about letrec.)

@xrchz xrchz closed this as completed May 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tanyongkiam tanyongkiam

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xrchz @tanyongkiam