RFR 4358774: Add null InputStream and OutputStream (original) (raw)

Brian Burkhalter brian.burkhalter at oracle.com
Thu Jan 4 23:42:08 UTC 2018

On Dec 11, 2017, at 12:52 PM, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:

On Dec 8, 2017, at 3:12 PM, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:

All previous suggested changes have been made in

http://cr.openjdk.java.net/~bpb/4358774/webrev.03/ except for the possible change of name for the IS and OS nullStream() methods which awaits consensus. As long as we’re still at it, given java.util.stream.Stream then InputStream.nullInput() OutputStream.nullOutput() Reader.nullReader() Writer.nullWriter() might not be a bad alternative.

Re-reading the discussion [1] of last year regarding [2], it seems that there was convergence on all points raised except the names of the new methods. The alternatives are:

A) InputStream.nullStream() and OutputStream.nullStream() as in the most recent version, webrev.03, B) InputStream.nullInput() and OutputStream.nullOutput() as shown above, and C) InputStream.nullInputStream() and OutputStream.nullOutputStream().

Does anyone have any more comments on this point? I would be inclined to choose option C.

Thanks,

Brian

[1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2017-December/050367.html [2] https://bugs.openjdk.java.net/browse/JDK-4358774

More information about the core-libs-dev mailing list