Xueming Shen wrote:
 :

Webrev has been updated accordingly.

I renamed the getBBuffer to getByteBuffer, now it "looks" better:-)
The updated webrev looks fine to me. Personally I wouldn't have cached the byte buffer as the wrapping is not expensive (doesn't copy the array) and it's only used for the malformed case, but what you have is fine as we're out of time now.

-Alan

Reply via email to