The webrev looks fine to me, too.

Unrelated to this change, but something I noticed:

test/jdk/java/lang/ProcessBuilder/PipelineTest.java

 173         setFileContents(infile, expected);

Shouldn't the contents of 'infile' be set to 'input', rather than 'expected'?

Thanks,
-Brent

On 1/3/19 2:37 PM, Brent Christian wrote:
Speaking of, you might want to change the copyright years to 2019. :)

Reply via email to