Interpreter (aka REPL) history should be by statement, not by textual line.

[scabug]

If you write the following in the interpreter:

scala> trait A {
     |   val x : String
     | }
defined trait A

And use up-arrow (or whatever your particular binding is), you get

scala> }

Which is nearly useless. The history should instead group an entire statement. For example, in zsh, if you write:

% for i in *.xml
for> wc -c $$i
8866 build.examples.xml
46544 build.xml

Then hitting up arrow gives you:

% for i in *.xml
wc -c $$i

rlwrap is not smart enough to either, it would probably require more configurability than is reasonable for it to offer, so we should fix the interaction with jline to get the more useful behavior.

[scabug]

Imported From: https://issues.scala-lang.org/browse/SI-1067?orig=1
Reporter: Geoffrey Alan Washburn (washburn)

[scabug]

@odersky said:
Milestone next_feature deleted

[scabug]

@paulp said:
At some point I implemented the logic for this, but the real obstacle is jline. It has enough trouble figuring out where the cursor is under the simplest conditions. Start throwing big multiline blocks into the history and it breaks down in tears. Would love to see this and #2547 addressed by the community.

[scabug]

@SethTisue said:
close as wontfix?

[scabug]

@paulp said:
I expect to fix this soon too, but it depends on how well the recent jline work goes. I implemented it long ago, and display issues are the impediment.

[scabug]

@SethTisue said:
At the time this ticket was created Paul hadn't yet added the :paste feature to the REPL. Now that it exists this ticket is even more relevant.

Stuff pasted into the REPL via :paste probably shouldn't become part of the history when you up-arrow through it. I'm typically pasting 5 or 10 or 20 or 50 lines of code, and it's annoying to have to up-arrow through all that stuff to get to the last expression I actually typed myself with my fingers.

I can imagine this might not be totally trivial to implement since :paste'd code should certainly be considered part of the history for :replay purposes, just not for up-arrow-ing purposes.

I'm adding this as a comment on this ticket, but it isn't strictly the same issue. If fixing JLine not to choke on multiline stuff is too hard, then it would still be very nice to have the "don't up-arrow into pastes" change just by itself.

[scabug]

Li Haoyi (lihaoyi) said:
This has been fixed in the Ammonite REPL.

@ trait A{
    val x: String
  }
defined trait A
@ <UP>
@ trait A{
  val x: String
}
defined trait A
@ <ENTER>
@

It looks kind of awkward sometimes, but it does work

[scabug]

Hamish Dickson (hamish.dickson-at-gmail.com) said:
Is this something people are still interested in? I've been looking for something to work on and would be happy to investigate

[Jasper-M]

Dotty has this. Can't we steal it?