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