RFR: 8269261: The PlaceHolder code uses Thread everywhere but is always dealing with JavaThreads [v2]

David Holmes dholmes at openjdk.java.net
Sun Jun 27 22:28:29 UTC 2021


On Fri, 25 Jun 2021 20:33:19 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixed alignment
>
> src/hotspot/share/classfile/placeholders.cpp line 56:
> 
>> 54:    }
>> 55:    JavaThread* thread()                const { return _thread;}
>> 56:    void set_thread(JavaThread* thread) { _thread = thread; }
> 
> Can you line these { up also?

Sure. The above matched the style on that little block of code but I can change both bits.

Thanks for the review.

-------------

PR: https://git.openjdk.java.net/jdk/pull/4592


More information about the hotspot-runtime-dev mailing list