On Wed, 27 Apr 2022 17:53:01 GMT, Mandy Chung <[email protected]> wrote:
>> Alan Bateman has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Refresh 7965cc6b168e567ac2596f2fbc3b00a7d99b7e1e
>
> src/java.base/share/classes/jdk/internal/misc/UnsafeConstants.java line 122:
>
>> 120: */
>> 121:
>> 122: public static final int SCOPED_CACHE_SHIFT;
>
> I can't find this constant being used. If added for future, maybe keep
> `UnsafeConstants` class and this static field package-private for now.
It originally configured the number of lines in extent local cache but the
implementation has changed. @theRealAph would be best to comment on this, it
may be possible to delete it.
> src/java.management/share/classes/sun/management/ThreadImpl.java line 447:
>
>> 445: if (threads != null) {
>> 446: long[] tids = Stream.of(threads)
>> 447: .filter(t -> !(t instanceof
>> jdk.internal.misc.CarrierThread))
>
> Returning an array of thread IDs of just the platform threads is good. The
> javadoc needs to be updated to say "Returns an array of thread identifiers
> for the platform threads...."
I think you mean the comment on the private method. Yes, that could be clearer
that it just returns platform threads.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8166
