Hi Brent, On Dec 8, 2017, at 4:50 PM, Brent Christian <brent.christian@oracle.com> wrote:

I just noticed a couple small things in the test:

test/jdk/java/io/InputStream/NullInputStream.java

[…]

Line 113 should read "... != 0", yes?

[…]

On 132, it's "skip() != 0".

You are correct on both accounts. I shuffled that test completely around today converting it to TestNG and obviously made some typos. Webrev updated in place http://cr.openjdk.java.net/~bpb/4358774/webrev.03/. Thanks, Brian