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

Mikael Vidstedt mikael.vidstedt at oracle.com
Mon Mar 19 17:06:33 UTC 2018


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
> 



More information about the portola-dev mailing list