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 serviceability-dev mailing list