RFR: 8305457: Implement java.io.IO [v3]
Pavel Rappo
prappo at openjdk.org
Wed May 8 09:03:54 UTC 2024
On Wed, 8 May 2024 05:41:52 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:
>> Pavel Rappo has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Strengthen tests after 8330998
>>
>> https://github.com/openjdk/jdk/pull/18996 now allows us to test
>> Console IO better.
>
> src/java.base/share/classes/java/io/Console.java line 188:
>
>> 186:
>> 187: /**
>> 188: * Writes a prompt as if by calling {@code print}, then reads a single line
>
> Should `{@code print}` instead be `{@link Console#print() print()}`?
It could be done like that, but given that it's the same class (and the same HTML page), I'd skip it.
> src/java.base/share/classes/java/io/Console.java line 192:
>
>> 190: *
>> 191: * @param prompt
>> 192: * A prompt string.
>
> Hello Pavel, should this specify whether `prompt` can be null?
If we specify that, it would be very much unlike all other `Console` methods that are covered by this:
* Unless otherwise specified, passing a {@code null} argument to any method
* in this class will cause a {@link NullPointerException} to be thrown.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19112#discussion_r1593687355
PR Review Comment: https://git.openjdk.org/jdk/pull/19112#discussion_r1593684673
More information about the core-libs-dev
mailing list