RFR 8231451: ZipFileInputStream::skip handling of negative values with STORED entries
Lance Andersen
lance.andersen at oracle.com
Fri Oct 25 18:42:06 UTC 2019
> On Oct 23, 2019, at 4:58 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>
> On 23/10/2019 19:06, Lance Andersen wrote:
>>
>>> On Oct 22, 2019, at 2:37 PM, Alan Bateman <Alan.Bateman at oracle.com <mailto:Alan.Bateman at oracle.com>> wrote:
>>>
>>> I assume skip(Long.MAX_VALUE) will cause n+pos to overflow and it will skip backwards rather than to the end.
>>
>> That is correct. If you prefer it to skip to the end, I will adjust accordingly.
> It would be surprising to specify a positive value and have it skip backwards so I think this should be fixed.
I have updated the patch per your suggestion.
The updated webrev can be found at: http://cr.openjdk.java.net/~lancea/8231451/webrev.02/index.html <http://cr.openjdk.java.net/~lancea/8231451/webrev.02/index.html>
mach5 jdk-tier1 through jdk-tier3 are clean.
Best
Lance
>
> -Alan
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
<http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
More information about the core-libs-dev
mailing list