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

[New module system] editing interface files causes a panic #1456

Closed
yav opened this issue Oct 13, 2022 · 0 comments
Closed

[New module system] editing interface files causes a panic #1456

yav opened this issue Oct 13, 2022 · 0 comments
Assignees
Labels
bug Something not working correctly

Comments

@yav
Copy link
Member

yav commented Oct 13, 2022

If we try to load an interface file, Cryptol correctly says that it can't load the file. However, we do set the current module as the interface file, so if we do :edit we can edit the file. Exiting the editor tries to reload the file, and again fails (correctly), but then the REPL is left in an inconsistent state where trying to evaluate anything panic because the focused module is not loaded.

@yav yav added the bug Something not working correctly label Oct 13, 2022
@yav yav self-assigned this Oct 14, 2022
yav added a commit that referenced this issue Oct 14, 2022
When loaded an interface is treated like an empty functor.
Adding this capability would also be useful if we generalize
interfaces to allow for definitions, as discussed in #1457

This also fixes #1456
@yav yav closed this as completed Oct 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something not working correctly
Projects
None yet
Development

No branches or pull requests

1 participant