RFR: 8308591: JLine as the default Console provider [v5]

Naoto Sato naoto at openjdk.org
Tue Jul 18 17:09:05 UTC 2023


On Tue, 18 Jul 2023 15:21:50 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   `true`
>
> src/java.base/share/classes/java/io/Console.java line 354:
> 
>> 352:      * character devices are available to the {@code Console}. Otherwise it could
>> 353:      * mean that the implementation of the {@code Console} may simulate those
>> 354:      * devices within.
> 
> The implNote helps as this is a difficult method to specify in a platform independent way.
> 
> The class description uses "interactive command line" and  "will typically be connected to the keyboard and display" and maybe we could re-use that here. Here's a suggestion for the second paragraph:
> 
> "This method returns true if the console device, associated with the current Java virtual machine, is a terminal, typically an interactive command line connected to a keyboard and display."
> 
> I don't have any other comments on the implementation changes, they look fine.

Thanks. Replaced the second paragraph with your suggested sentence.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14271#discussion_r1267070257


More information about the core-libs-dev mailing list