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?
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?
-
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.
>
> `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