Xueming Shen wrote:
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.:Webrev has been updated accordingly. I renamed the getBBuffer to getByteBuffer, now it "looks" better:-)
-Alan