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

Cannot assign to constants #183

Closed
wants to merge 1 commit into from
Closed

Conversation

dvdsgl
Copy link

@dvdsgl dvdsgl commented Oct 9, 2021

num and denom are potentially assigned, so cannot be const.

@jcubic jcubic changed the base branch from master to devel October 10, 2021 08:42
@jcubic
Copy link
Collaborator

jcubic commented Oct 10, 2021

Please use devel branch (I've change it but there are conflicts).

@jcubic
Copy link
Collaborator

jcubic commented Oct 10, 2021

Sorry, I didn't notice that this is no longer valid. See devel branch code:
https://github.com/jcubic/lips/blob/devel/src/lips.js#L5792

@dvdsgl
Copy link
Author

dvdsgl commented Oct 10, 2021

Cool!

@dvdsgl dvdsgl closed this Oct 10, 2021
@dvdsgl dvdsgl deleted the patch-1 branch October 10, 2021 11:13
@jcubic
Copy link
Collaborator

jcubic commented Oct 10, 2021

I will publish the next beta version with the fix but I first need to fix #176 there is a better way to bootstrap the system where there is a binary file that was parsed at the built time, reading that file is much faster than parse whole standard library each time, but first I need to include the library. I need to find time to do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants