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

Reply via email to