RFR: 8356165: System.in in jshell replace supplementary characters with ??

[Tatsunori Uchino]

On Wed, 7 May 2025 06:44:54 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

> When reading from `System.in` in a JShell snippet, JShell first reads the whole line (getting a `String`), and then converts this characters from this `String` to bytes on demand. But, it does not convert multi-surrogate code points correctly, it tries to convert each surrogate separately, which cannot work.
> 
> The proposal herein is to, when the current character is a high surrogate, peek at the next character, and if it is a low surrogate, convert both the high and low surrogates to bytes together.

src/jdk.jshell/share/classes/jdk/internal/jshell/tool/ConsoleIOContext.java line 986:

> 984:                     Character.isLowSurrogate(pendingLine.charAt(pendingLinePointer))) {
> 985:                     dataToConvert.append(readUserInputChar());
> 986:                 }

How about combining `readUserInputChar()` and `pendingLinePointer--`?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/25079#discussion_r2079889528