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

Markus KARG duke at openjdk.org
Sun Dec 17 13:59:42 UTC 2023


On Sat, 16 Dec 2023 19:03:26 GMT, Vladimir Sitnikov <vsitnikov at openjdk.org> wrote:

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

Kept it for future use: If in future we like to add `write(b)` in `transferTo` then we do not have to rework the call sites.

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

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


More information about the core-libs-dev mailing list