RFR 6856817: Poor performance of Writer#append with CharBuffer

Aleksey Shipilev aleksey.shipilev at oracle.com
Wed Nov 25 15:22:43 UTC 2015


Hi vyom,

On 11/25/2015 05:29 PM, vyom wrote:
> Hi All,
> 
> Please review my changes for below bug.
> 
> Bug:        JDK-6856817 : Poor performance of Writer#append with CharBuffer
> Webrev: http://cr.openjdk.java.net/~vtewari/6856817/webrev0.4/

Interesting change.

But, I think the "null" handling in new OSW.append methods is incorrect.
Javadoc says that those methods should behave exactly as:

  out.write(csq.toString())

or:

 out.write(csq.subSequence(start, end).toString())

If CharSequence is null, we should throw NullPointerException, not doing
any "null" mangling, as your patch does? If so, can you please fix this
and add more tests?

Cheers,
-Aleksey







More information about the core-libs-dev mailing list