RFR: 8282475: SafeFetch should not rely on existence of Thread::current [v4]
Thomas Stuefe
stuefe at openjdk.java.net
Wed Mar 9 06:52:58 UTC 2022
On Tue, 8 Mar 2022 13:08:13 GMT, Johannes Bechberger <duke at openjdk.java.net> wrote:
>> 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.
"os" is okay. Its historical. Its also more of a namespace really.
-------------
PR: https://git.openjdk.java.net/jdk/pull/7727
More information about the shenandoah-dev
mailing list