Re: Code review request "6631046: BufferedInputStream.available() reports negative int on very large inputstream"

2010-11-19 Thread Alan Bateman
Mandy Chung wrote: 6631046: BufferedInputStream.available() reports negative int on very large inputstream Webrev at: http://cr.openjdk.java.net/~mchung/6631046/webrev.00/ InputStream.available() returns an int. For streams larger than 2GB, the FileInputStream.available() implementation retu

Re: Code review request "6631046: BufferedInputStream.available() reports negative int on very large inputstream"

2010-11-18 Thread Mike Duigou
Looks good to me. Mike On Nov 18 2010, at 14:34 , Mandy Chung wrote: > 6631046: BufferedInputStream.available() reports negative int on very large > inputstream > > Webrev at: > http://cr.openjdk.java.net/~mchung/6631046/webrev.00/ > > InputStream.available() returns an int. For streams large

Re: Code review request "6631046: BufferedInputStream.available() reports negative int on very large inputstream"

2010-11-18 Thread David Holmes
Hi Mandy, Looks good to me. David Mandy Chung said the following on 11/19/10 08:34: 6631046: BufferedInputStream.available() reports negative int on very large inputstream Webrev at: http://cr.openjdk.java.net/~mchung/6631046/webrev.00/ InputStream.available() returns an int. For streams l

Code review request "6631046: BufferedInputStream.available() reports negative int on very large inputstream"

2010-11-18 Thread Mandy Chung
6631046: BufferedInputStream.available() reports negative int on very large inputstream Webrev at: http://cr.openjdk.java.net/~mchung/6631046/webrev.00/ InputStream.available() returns an int. For streams larger than 2GB, the FileInputStream.available() implementation returns Integer.MAX_VALU