RFR 8196298 Add null Reader and Writer with latest changes

Brian Burkhalter brian.burkhalter at oracle.com
Tue Mar 6 22:12:22 UTC 2018


Yes, I think so and and also the parameter ‘csq' is allowed to be null so Objects.requireNonNull(csq) should be removed at lines 100 and 107 as no NPE is specified for these methods [1, 2].

Thanks,

Brian

[1] https://download.java.net/java/jdk10/docs/api/java/io/Writer.html#append(java.lang.CharSequence)
[2] https://download.java.net/java/jdk10/docs/api/java/io/Writer.html#append(java.lang.CharSequence,int,int)

On Mar 6, 2018, at 11:49 AM, Bernd Eckenfels <ecki at zusammenkunft.net> wrote:

> Just a nit, Should append(CharSequence,int,int) also use checkFromIndexSize?



More information about the core-libs-dev mailing list