RFR: 8264208: Console charset API

Joe Wang joehw at openjdk.java.net
Fri Apr 9 19:34:21 UTC 2021


On Fri, 9 Apr 2021 16:47:55 GMT, Naoto Sato <naoto at openjdk.org> wrote:

> Please review the changes for the subject issue.  This has been suggested in a recent discussion thread for the JEP 400 [[1](https://mail.openjdk.java.net/pipermail/core-libs-dev/2021-March/075214.html)]. A CSR has also been drafted, and comments are welcome [[2](https://bugs.openjdk.java.net/browse/JDK-8264209)].

Marked as reviewed by joehw (Reviewer).

src/java.base/share/classes/java/io/Console.java line 404:

> 402:      *
> 403:      * @return A {@code Charset} object used in this {@code Console}.
> 404:      * @since 17

A couple of minor comments: 
May replace {@code Charset} with @link;
Add "the" to the sentence: The returned charset corresponds to "the" input...
@return: javadoc guide suggests "do not capitalize and do not end with a period" when writing a phrase. But I guess for consistency in this class, it's okay to capitalize.

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

PR: https://git.openjdk.java.net/jdk/pull/3419



More information about the security-dev mailing list