RFR: 8282475: SafeFetch should not rely on existence of Thread::current [v4]
Johannes Bechberger
duke at openjdk.java.net
Tue Mar 8 13:11:06 UTC 2022
On Tue, 8 Mar 2022 12:33:10 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Johannes Bechberger has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Move code to os::current_thread_wx
>> - Small fixes
>
> src/hotspot/os_cpu/bsd_aarch64/os_bsd_aarch64.hpp line 45:
>
>> 43: #ifdef __APPLE__
>> 44:
>> 45: class current_thread_wx {
>
> This violates the style guide for class names. It would be CurrentThreadWX - but ThreadWX seems sufficient to me.
But os is okay? I just use this name for grouping.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7727
More information about the shenandoah-dev
mailing list