-
-
Notifications
You must be signed in to change notification settings - Fork 57
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
Parse complex numbers. #164
Conversation
Things like 2i don't seem to usually parse as complex numbers in some Schemes.
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.
I think this is close. Left some comments for areas I don't understand very well.
I should do some more reading into I closed all the conversations with comments that seem fine so program.rs seems to be the sketchiest part. |
Yeah 😅 things get a little gnarlier in the compiler there. I spot checked it and I'm reasonably confident the change is right. That pass is just checking if the constant is either 0, 1, or 2 (integer) - which you kept in place, so that seems fine. |
oh yeah, the usage was pretty simple. I think this is good to go. I can also split this into smaller PRs if you'd like. The token refactor had a lot more changes than I expected. |
No need, I'll do one more pass and otherwise I think its good as well |
No description provided.