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