RFR: 8320788: The system properties page is missing some properties [v2]

Naoto Sato naoto at openjdk.org
Tue Jan 9 19:28:25 UTC 2024


On Tue, 9 Jan 2024 19:02:39 GMT, Iris Clark <iris 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 818:
> 
>> 816:      * {@link #getProperty(String)} operation.
>> 817:      * <p>
>> 818:      * In addition to the above set of system properties, locale related
> 
> I think I'd consider a slight reduction in the first/last phrases leaving this equivalent statement:
> 
> Additional locale-related system properties defined by the {@link Locale##default_locale Default Locale} section in the {@code Locale} class description may also be obtained with this method.
> 
> (Note that I couldn't find the referenced Locale section, but perhaps I was looking at an older version of the spec.)

That's concise and clearer. Replaced the wording. As to the referenced Locale section, it is in the PR (#17065) which this PR is dependent on and is not yet integrated. Sorry for the confusion.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17317#discussion_r1446509254


More information about the core-libs-dev mailing list