Quantcast

RFR(S) 8178731: compiler/ciReplay/SABase.java does not compile

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR(S) 8178731: compiler/ciReplay/SABase.java does not compile

Ekaterina Pavlova
Hi everyone,

Please review this small change that fixes hotspot/test/compiler/ciReplay/SABase.java
This file used ProcessHandle.getPid() which was recently renamed in ProcessHandle.pid()
as part of JDK-8178347.
Also fixed test/serviceability/sa/sadebugd/SADebugDTest.java which had the same issue.

   bug: https://bugs.openjdk.java.net/browse/JDK-8178731
webrev: http://cr.openjdk.java.net/~iignatyev/epavlova/8178731/webrev.00/

Tested by running jprt.

thanks,
-katya

p.s.
 Igor Ignatyev volunteered to sponsor this change.
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR(S) 8178731: compiler/ciReplay/SABase.java does not compile

Igor Veresov
Looks good.

igor

> On Apr 14, 2017, at 3:48 PM, Ekaterina Pavlova <[hidden email]> wrote:
>
> Hi everyone,
>
> Please review this small change that fixes hotspot/test/compiler/ciReplay/SABase.java
> This file used ProcessHandle.getPid() which was recently renamed in ProcessHandle.pid()
> as part of JDK-8178347.
> Also fixed test/serviceability/sa/sadebugd/SADebugDTest.java which had the same issue.
>
>   bug: https://bugs.openjdk.java.net/browse/JDK-8178731
> webrev: http://cr.openjdk.java.net/~iignatyev/epavlova/8178731/webrev.00/
>
> Tested by running jprt.
>
> thanks,
> -katya
>
> p.s.
> Igor Ignatyev volunteered to sponsor this change.

Loading...