RFR: Fix jtreg test java/lang/ProcessHandle/InfoTest.java

[Mikael Vidstedt]

Looks good, please send me the patch and I’ll push!

Cheers,
Mikael

> On Mar 20, 2018, at 7:00 AM, Siebenborn, Axel <axel.siebenborn at sap.com> wrote:
> 
> Hi Mikael,
> 
> thanks for your review.
> 
> I uploaded a  new webrev:
> http://cr.openjdk.java.net/~asiebenborn/jtreg_ProcessHandler/webrev.01/
> 
> Regards,
> Axel
> 
>> -----Original Message-----
>> From: Mikael Vidstedt [mailto:mikael.vidstedt at oracle.com]
>> Sent: Montag, 19. März 2018 18:07
>> To: Siebenborn, Axel <axel.siebenborn at sap.com>
>> Cc: portola-dev at openjdk.java.net
>> Subject: Re: RFR: Fix jtreg test java/lang/ProcessHandle/InfoTest.java
>> 
>> 
>> Looks like the are superfluous parenthesis around the new File() call on line
>> 304, apart from that looks good. Send me a patch and I’ll help push it.
>> 
>> 
>> (A reflection mostly unrelated to your change: the various platform checks
>> (Windows, Unix, BusyBox, …) really should be using the common
>> functionality from test/lib/jdk/test/lib/Platform.java.)
>> 
>> Cheers,
>> Mikael
>> 
>>> On Mar 19, 2018, at 4:45 AM, Siebenborn, Axel
>> <axel.siebenborn at sap.com> wrote:
>>> 
>>> Hi,
>>> 
>>> could I have a review for the following webrev:
>>> 
>>> http://cr.openjdk.java.net/~asiebenborn/jtreg_ProcessHandler/webrev/
>>> 
>>> /bin/sleep is a link to busybox and thats what ProcessHandle.Info
>> command() returns.
>>> 
>>> Regards,
>>> Axel
>>> 
>