On 06/06/2018 02:16 PM, Valerie Peng wrote:
Hi Tony,

Can you please help reviewing this fix? The original impl uses buffering for both input and output and is broken in several places. I ended up re-writing most of them. The buffering is done only on input. The new regression test covers all 4 combination of input/output ByteBuffers as well as various sizes.

Webrev: http://cr.openjdk.java.net/~valeriep/8178374/webrev.00/

Bug: https://bugs.openjdk.java.net/browse/JDK-8178374

Thanks,
Valerie


The change looks fine to me.

Thanks

Tony

Reply via email to