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

David Holmes dholmes at openjdk.java.net
Fri Jun 25 02:40:45 UTC 2021


On Fri, 25 Jun 2021 02:13:40 GMT, Calvin Cheung <ccheung 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.hpp line 120:
> 
>> 118:   Symbol*           _supername;
>> 119:   JavaThread*           _definer;       // owner of define token
>> 120:   InstanceKlass*    _instanceKlass; // InstanceKlass from successful define
> 
> Can you align the underscore (_) and "//" with the line below?

Fixed.

Thanks for the review.

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

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


More information about the hotspot-runtime-dev mailing list