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

Rework local functions #91

Open
FilipToth opened this issue Jan 16, 2022 · 1 comment
Open

Rework local functions #91

FilipToth opened this issue Jan 16, 2022 · 1 comment
Labels
Alto 1 enhancement New feature or request
Milestone
Alto 1.0

Comments

@FilipToth
Copy link
Member

Background and Motivation

The local functions API is quite messy:

private Dictionary<BoundScope, List<Tuple<FunctionSymbol, BoundBlockStatement>>> _localFunctions = new Dictionary<BoundScope, List<Tuple<FunctionSymbol, BoundBlockStatement>>>();

It would be a good idea to rework this for clarity.
@FilipToth
Copy link
Member Author

Probably just add a local functions field to the BoundScope API, this feels so dumb in retrospect 😂.

@FilipToth FilipToth added enhancement New feature or request Alto 1 labels Mar 14, 2022
@FilipToth FilipToth added this to the Alto 1.0 milestone Mar 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Alto 1 enhancement New feature or request
Projects
None yet
Milestone
Alto 1.0
Development

No branches or pull requests
1 participant
@FilipToth