RFR: 8288387: GetLocalXXX/SetLocalXXX spec should require suspending target thread [v2]

Serguei Spitsyn sspitsyn at openjdk.org
Mon Oct 10 23:10:05 UTC 2022


On Mon, 10 Oct 2022 10:08:59 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   addressed review comments about is_JavaThread_current and @enablePreview tag
>
> test/hotspot/jtreg/serviceability/jvmti/GetLocalVariable/GetSetLocalUnsuspended.java line 41:
> 
>> 39:     static private volatile boolean doStop;
>> 40: 
>> 41:     static private void sleep(long millis) {
> 
> Let's use default modifiers order
> Suggestion:
> 
>     private static void sleep(long millis) {

Okay, fixed.
Thank you for the comment!

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

PR: https://git.openjdk.org/jdk/pull/10586


More information about the hotspot-dev mailing list