On Tue, 13 May 2025 03:15:25 GMT, Naoto Sato <na...@openjdk.org> wrote:
>> With the introduction of `stdin.encoding` >> ([JDK-8350703](https://bugs.openjdk.org/browse/JDK-8350703)), some guidance >> for users to decode `System.in` would be desirable. Adding examples in the >> field description would help. > > Naoto Sato has updated the pull request incrementally with one additional > commit since the last revision: > > Removed an erroneous curly brace src/java.base/share/classes/java/io/InputStreamReader.java line 59: > 57: * new InputStreamReader(System.in, > System.getProperty("stdin.encoding")); > 58: * } > 59: * There are several places in the sources that still contain `new InputStreamReader(System.in)`. Shouldn't they be replaced with `new InputStreamReader(System.in, System.getProperty("stdin.encoding"))` too?) ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/25155#discussion_r2086612688