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

[wish] Put the "L" in "REPL" by making it loop #145

Open
shonfeder opened this issue Apr 30, 2022 · 1 comment
Open

[wish] Put the "L" in "REPL" by making it loop #145

shonfeder opened this issue Apr 30, 2022 · 1 comment

Comments

@shonfeder
Copy link
Contributor

shonfeder commented Apr 30, 2022

The current behavior of the repl is to read an expression, query the goal, and quit. It would be much nicer if it looped, and allowed posing repeated queries :)

Additionally, I usually use the repl wrapped in rlwrap to make for a slightly nicer experience, but it'd be nice to include some readline functionality with the program. IIUC, this might be accomplished via https://github.com/ocaml-community/ocaml-linenoise?

I'd be happy to help with this kind of effort, if you thought it worth pursuing.

@gares
Copy link
Contributor

gares commented Apr 30, 2022

Right the repl can take some love.

Im ok merging code, only two points:

  • I don't want external dependencies which are both non optional and not rock solid
  • I want to keep windows support, even if it can have less features

Example: it was a boodbath to replace campl5 by menhir, but I'm not looking back. External non standard perl dependencies? Strong coupling with the ocaml compiler? I just needed a parser.

You surely know better the records of lineoise library. Looks nice and self contained, but I never used it.

About the L, sure, no problem. Help is always welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants