RFR: 8299836: Make `user.timezone` system property searchable [v2]
Jaikiran Pai
jpai at openjdk.org
Tue Jan 10 01:37:54 UTC 2023
On Tue, 10 Jan 2023 00:42:22 GMT, Justin Lu <duke at openjdk.org> wrote:
>> src/java.base/share/classes/java/util/TimeZone.java line 634:
>>
>>> 632: *
>>> 633: * <ul>
>>> 634: * <li>Use the {@systemProperty user.timezone} property value as the default
>>
>> I'd only modify this location and keep others intact. Otherwise, the search would show all locations which may not best serve the user's needs.
>
> Good point, I reverted the rest, so that only this first instance is tagged as _ at systemProperty_
Like Naoto notes, my understanding of it is that the `@systemProperty` should appear only once at the place where the semantics of that system property is being defined. This mail, which was sent when this tag was introduced, has more details https://mail.openjdk.org/pipermail/core-libs-dev/2018-November/056653.html.
I think using it here is fine, but you may want to check how it "reads" on the generated javadoc index page for system properties.
-------------
PR: https://git.openjdk.org/jdk/pull/11915
More information about the core-libs-dev
mailing list