RFR: JDK-8322141: SequenceInputStream.transferTo should not return as soon as Long.MAX_VALUE bytes have been transferred [v4]

Vladimir Sitnikov vsitnikov at openjdk.org
Sat Dec 16 19:05:40 UTC 2023


On Sat, 16 Dec 2023 14:07:52 GMT, Markus KARG <duke at openjdk.org> wrote:

>> Fixes JDK-8322141
>> 
>> As suggested by @vlsi and documented by @bplb this PR does not return, but only sets the maximum result value.
>
> Markus KARG has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Test was using Integer but must use Long

test/jdk/java/io/SequenceInputStream/TransferTo.java line 154:

> 152:      * but fails with any other operation.
> 153:      */
> 154:     private static InputStream repeat(int b, long count) {

`b` parameter seems to be unused. Wdyt of removing it?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17119#discussion_r1428889592


More information about the core-libs-dev mailing list