RFR: 8341413: Stop including osThread_os.hpp in the middle of the OSThread class [v2]
    Stefan Karlsson 
    stefank at openjdk.org
       
    Thu Oct  3 11:34:35 UTC 2024
    
    
  
On Thu, 3 Oct 2024 01:30:50 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Stefan Karlsson has updated the pull request incrementally with three additional commits since the last revision:
>> 
>>  - Move NONCOPYABLE
>>  - Move VMStructs fields out of the CPU files
>>  - Add comment to the include of the platform specific class
>
> src/hotspot/share/runtime/osThread.hpp line 29:
> 
>> 27: 
>> 28: #include "utilities/macros.hpp"
>> 29: #include OS_HEADER(osThread)
> 
> Suggestion:
> 
> 
> // The actual class declaration is platform specific.
> #include OS_HEADER(osThread)
Fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21306#discussion_r1786066476
    
    
More information about the graal-dev
mailing list