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