RFR 4358774: Add null InputStream and OutputStream

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