RFR: 8273185: Rename the term "atomic" in ReferenceProcessor [v2]

Thomas Schatzl tschatzl at openjdk.java.net
Wed Sep 8 19:15:56 UTC 2021


On Wed, 8 Sep 2021 08:38:38 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Hi all,
>> 
>>   please review this suggestion to rename the term "atomic" in the `ReferenceProcessor` class to "concurrent"/"stw" to avoid the confusion with Atomic operations.
>> 
>> It's a smaller change than expected, which is good, but also might question the need for this change. I grepped for "atomic" in the reference processor sources and hopefully got everything.
>> 
>> Testing: tier1-5
>> 
>> Thanks,
>>   Thomas
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
> 
>   shade review

I looked through the GHA failures, none of that related to this change.

Thanks @shipilev @albertnetymk for your reviews.

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

PR: https://git.openjdk.java.net/jdk/pull/5397



More information about the hotspot-gc-dev mailing list