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

wdte: use *Frame #101

Open
DeedleFake opened this issue Mar 28, 2018 · 0 comments
Open

wdte: use *Frame #101

DeedleFake opened this issue Mar 28, 2018 · 0 comments

Comments

@DeedleFake
Copy link
Owner

DeedleFake commented Mar 28, 2018

The current implementation uses Frame instead of *Frame, despite it being a linked list. I think the implementation of Scope is good evidence that this isn't necessary. An upside of using *Frame would be that a user could pass nil as the top-level frame.

It might make sense as well to allow the default frame to be overridden. This would allow the user to just use nil but still get access to, for example, std. Maybe something like a DefaultFrame global variable that gets used in the case of a nil frame.

@DeedleFake DeedleFake self-assigned this Mar 28, 2018
@DeedleFake DeedleFake changed the title Use *Frame wdte: use *Frame Nov 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant