On Sat, 16 Dec 2023 19:03:26 GMT, Vladimir Sitnikov <vsitni...@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
