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