Hi Katya,

looks good to me. since it also changes svc tests, I cc'ed svc-dev alias.

Thanks,
-- Igor

> On Apr 14, 2017, at 3:52 PM, Ekaterina Pavlova <[email protected]> 
> 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.

Reply via email to