On 28/08/2013 00:09, Xueming Shen wrote:
On 08/27/2013 03:07 PM, Martin Buchholz wrote:
It does seem vaguely reasonable to support any extra data.
Don't you want to also handle arbitrary byte arrays, if e.g. one the
16-bit size fields overflows the extra data?
It looks to me like getExtraLen could return a negative number.
probably I should. The webrev has been updated to simply
copy the rest of the extra data, if the "sz" is either < 0 or
out of the range.
http://cr.openjdk.java.net/~sherman/8023713/webrev/
The updated webrev looks okay, just odd to see an entry named "
TestExtreTime" (which may have been a continuation of a typo in the
original test).
-Alan.