> 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>



Reply via email to