Hi Katya,

It looks good.

Thanks,
Serguei


On 4/14/17 15:54, Igor Ignatyev wrote:
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