RFR: 8298416: Console should be declared `sealed` [v3]

Naoto Sato naoto at openjdk.org
Thu Dec 15 17:12:34 UTC 2022


On Thu, 15 Dec 2022 12:24:34 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Moved echo() to ConsoleImpl, clean-ups
>
> src/java.base/share/classes/java/io/Console.java line 111:
> 
>> 109:     public PrintWriter writer() {
>> 110:         throw new UnsupportedOperationException(
>> 111:                 "Console class itself does not provide implementation");
> 
> Minor comment is the same "throw new UOE(message)" is repeated in every method. You could add a static method to return it, then it would reduce to "throw newUnsupportedOperationException();".
> 
> Also looks like the method descriptions on all these methods starts at position 3. Pre-dates your change but since every method is changed then maybe we should fix that too, up to you.

Thanks, Alan. Addressed both.

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

PR: https://git.openjdk.org/jdk/pull/11615


More information about the core-libs-dev mailing list