On Tue, 8 Mar 2022 12:33:10 GMT, David Holmes <dhol...@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
