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