-
Notifications
You must be signed in to change notification settings - Fork 1.1k
REPL doesn't work on Windows: Consider directly using JLine 3 for the REPL #4309
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
Comments
Here is the Ammonite PR for reference: com-lihaoyi/Ammonite#775 |
I'm looking forward to using jline3 to implement dotty repl, because jline3's complement feature is very cool |
+1 to use jline3 to support colourful output and mutiply lines coding in windows. |
Renamed to help GitHub search find this: https://github.com/lampepfl/dotty/search?utf8=%E2%9C%93&q=repl+windows&type=Issues |
Now, there is still a problem with our Dotty REPL support for Chinese characters: Jline 3 can help us solve this problem: |
@Glavo Ah, so those are two characters but our REPL shows the cursor one character before it should be, as if they were one? |
@Blaisorblade Yes, and Ammonite also has this problem when not using the jline 3 front end. |
As for the issue of Chinese characters, note that JLine's output transfer “你好” to its codepoint |
@W4anD0eR96 Yes but I think that's a separate difference, independent from JLine and due to Ammonite's and Dotty's approaches at printing values — Dotty prints strings with toString (so |
Fixed in #4680 |
The REPL UI code in Dotty comes from ammonite-terminal and lives in https://github.com/lampepfl/dotty/tree/master/compiler/src/dotty/tools/repl/terminal, it works well enough but does not support Windows. This has been recently fixed in Ammonite by using JLine 3 under Windows. We could merge these changes in our version of ammonite-terminal, but if we're going to depend on JLine anyway we could also drop our version of ammonite-terminal and just use JLine directly (unless there's some features of ammonite-terminal we really need).
The text was updated successfully, but these errors were encountered: