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

Cleanup main.go #71

Closed
skx opened this issue Oct 28, 2022 · 0 comments · Fixed by #72
Closed

Cleanup main.go #71

skx opened this issue Oct 28, 2022 · 0 comments · Fixed by #72

Comments

@skx
Copy link
Owner

skx commented Oct 28, 2022

It is hard to modify, due to the argument stuff being a little dense.

(Specifically the *hlp flag).

Once cleaned up we can add a repl, as noted in #70.
skx added a commit that referenced this issue Oct 29, 2022
@skx
Updated the main driver to simplify the code.
136806c 
This closes #71, by simplifying the main.go driver.  We also added
a means to execute code within the existing/persistent interpreter.

This had to be done to ensures we didn't reset our aliases, etc,
between executions - partly because this is obvious, and partly
anticipating #72.
@skx skx mentioned this issue Oct 29, 2022
Merged
@skx skx closed this as completed in #72 Oct 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updated the main driver to simplify the code. skx/yal
1 participant
@skx