On Tue, 8 Mar 2022 13:08:13 GMT, Johannes Bechberger <d...@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

Reply via email to