While there appears now to be agreement on using naming option C below, formal 
Reviewer approval appears still to be lacking. It would be good if there were 
at least two approvals forthcoming, if appropriate.

http://cr.openjdk.java.net/~bpb/4358774/webrev.03/

Thanks,

Brian

On Jan 4, 2018, at 3:42 PM, Brian Burkhalter <brian.burkhal...@oracle.com> 
wrote:

> 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

Reply via email to