RFR: 8341975: Unable to set encoding for IO.println, IO.print and IO.readln [v4]

[Jaikiran Pai]

On Tue, 22 Oct 2024 15:44:41 GMT, Chen Liang <liach at openjdk.org> wrote:

>> Naoto Sato has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Reflects review comments
>
> src/java.base/share/classes/java/lang/System.java line 150:
> 
>> 148:      * specified by the host environment or user. The encoding used
>> 149:      * in the conversion from characters to bytes is equivalent to
>> 150:      * <a href="#stdout.encoding">stdout.encoding</a>.
> 
> Suggestion:
> 
>      * {@systemProperty stdout.encoding}.
> 
> 
> This defines the anchor with the same id, and also makes this property available for javadoc index search. Example includes the system property at https://download.java.net/java/early_access/jdk24/docs/api/java.base/java/lang/ClassLoader.html#java.system.class.loader

Hello Chen, the `{@systemProperty}` tag is expected to be used where the system property is being defined (the "Where should the tag be used?" section in https://mail.openjdk.org/pipermail/core-libs-dev/2018-November/056653.html). 
On this line, it's merely being referenced and the actual definition of this property is in the documentation of System.getProperties() method and there it correctly already uses a `{@systemProperty}` tag for this property https://github.com/openjdk/jdk/blob/master/src/java.base/share/classes/java/lang/System.java#L790

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21569#discussion_r1811041477