On Wed, 6 Dec 2023 17:42:47 GMT, Brett Okken wrote:
>> The static AtomicInteger used for the nextHashCode should be final.
>
> Brett Okken has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Update full name
@bokken, you are good to `/integrat
On Wed, 6 Dec 2023 17:42:47 GMT, Brett Okken wrote:
>> The static AtomicInteger used for the nextHashCode should be final.
>
> Brett Okken has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Update full name
The change looks fine to me.
-
> The static AtomicInteger used for the nextHashCode should be final.
Brett Okken has updated the pull request incrementally with one additional
commit since the last revision:
Update full name
-
Changes:
- all: https://git.openjdk.org/jdk/pull/16987/files
- new: https://git.
On Wed, 6 Dec 2023 00:52:48 GMT, Brett Okken wrote:
> The static AtomicInteger used for the nextHashCode should be final.
Looks okay to me!
Since this is new contribution, I would like someone else to take a look.
-
Marked as reviewed by shade (Reviewer).
PR Review: https://git.o
On Wed, 6 Dec 2023 00:52:48 GMT, Brett Okken wrote:
> The static AtomicInteger used for the nextHashCode should be final.
Submitted: https://bugs.openjdk.org/browse/JDK-8321470
Please change this PR title to "8321470: ThreadLocal.nextHashCode can be static
final", and bots would do the rest.
