On Tue, 10 Jan 2023 00:42:22 GMT, Justin Lu <d...@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 _@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
