Xueming Shen wrote:
Thanks! webrev has been updated as suggested.

http://cr.openjdk.java.net/~sherman/6975829/webrev/ <http://cr.openjdk.java.net/%7Esherman/6975829/webrev/>

-Sherman
Looks fine. A minor comment is that you don't need to update this_off and it might be clear to just set the offset field to this_off + consumed.

-Alan.

Reply via email to