> On Oct 23, 2019, at 4:58 PM, Alan Bateman <alan.bate...@oracle.com> wrote: > > On 23/10/2019 19:06, Lance Andersen wrote: >> >>> On Oct 22, 2019, at 2:37 PM, Alan Bateman <alan.bate...@oracle.com >>> <mailto:alan.bate...@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.ander...@oracle.com <mailto:lance.ander...@oracle.com>