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

Use immutable data structures from im #41

Closed
jparoz opened this issue Mar 2, 2023 · 1 comment
Closed

Use immutable data structures from im #41

jparoz opened this issue Mar 2, 2023 · 1 comment
Labels
improvement Improves the way that an existing feature works, in a backwards-compatible way.
Milestone
0.3.0

Comments

@jparoz
Copy link
Owner

jparoz commented Mar 2, 2023

im

Particular opportunities:

  • name::resolve instead of manually tracking what's in scope and using bind and unbind, use a new method subscope (or something)
  • typecheck::Typechecker's m_stack field
@jparoz jparoz added the improvement Improves the way that an existing feature works, in a backwards-compatible way. label Mar 2, 2023
@jparoz jparoz added this to the 0.3.0 milestone Mar 2, 2023
@jparoz
Copy link
Owner Author

jparoz commented Mar 2, 2023

Probably won't bother with this. Started working on it, but the code was only maybe marginally more readable, and probably less performant. Maybe down the track, or if correctness is a stronger emphasis.

@jparoz jparoz closed this as not planned Won't fix, can't repro, duplicate, stale Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
improvement Improves the way that an existing feature works, in a backwards-compatible way.
Projects
Huck roadmap
Status: Cancelled
Milestone
0.3.0
Development

No branches or pull requests
1 participant
@jparoz