Re: RFR: 8297632: InputStream.transferTo() method should specify what the return value should be when the number of bytes transfered is larger than Long.MAX_VALUE [v3]

2022-11-30 Thread Lance Andersen
On Wed, 30 Nov 2022 17:33:17 GMT, Brian Burkhalter wrote: > > Can you check ZipInputStream.transferTo? > > That implementation just calls the overridden method, but the single sentence > added to the `InputStream::transferTo` javadoc should probably be inserted > here as well. Do you concur?

Re: RFR: 8297632: InputStream.transferTo() method should specify what the return value should be when the number of bytes transfered is larger than Long.MAX_VALUE [v3]

2022-11-30 Thread Brian Burkhalter
On Wed, 30 Nov 2022 16:26:59 GMT, Alan Bateman wrote: > Can you check ZipInputStream.transferTo? That implementation just calls the overridden method, but the single sentence added to the `InputStream::transferTo` javadoc should probably be inserted here as well. Do you concur? -

Re: RFR: 8297632: InputStream.transferTo() method should specify what the return value should be when the number of bytes transfered is larger than Long.MAX_VALUE [v3]

2022-11-30 Thread Alan Bateman
On Tue, 29 Nov 2022 19:36:40 GMT, Brian Burkhalter wrote: >> `java.io.InputStream::transferTo` could conceivably return a negative value >> if the count of bytes transferred overflows a `long`. Modify the method to >> limit the number of bytes transferred to `Long.MAX_VALUE` per invocation. >

Re: RFR: 8297632: InputStream.transferTo() method should specify what the return value should be when the number of bytes transfered is larger than Long.MAX_VALUE [v3]

2022-11-29 Thread Brian Burkhalter
> `java.io.InputStream::transferTo` could conceivably return a negative value > if the count of bytes transferred overflows a `long`. Modify the method to > limit the number of bytes transferred to `Long.MAX_VALUE` per invocation. Brian Burkhalter has updated the pull request incrementally with