RFR: 8279283 - BufferedInputStream should override transferTo [v7]
Alan Bateman
alanb at openjdk.org
Mon Sep 5 18:35:52 UTC 2022
On Mon, 5 Sep 2022 18:05:04 GMT, Markus KARG <duke at openjdk.org> wrote:
>> Implementation of JDK-8279283
>
> Markus KARG has updated the pull request incrementally with one additional commit since the last revision:
>
> HexPrinter::transferTo
It looks like this patch is against a repository that hasn't been sync'ed up since late 2021. BIS has changed, esp. the locking, so this is why you get the merge-conflict label. Look at the implXXX methods to see how the existing methods do the locking.
I think I pointed out in one of the early rounds that you'll have to take the mark (if set) into account. It may be that you just call super.transferTo when markpos == -1.
The other issue that needs consideration is that the draining of the buffered bytes will leak the underlying input stream to the target output stream. It may be safer to also call super.transferTo when (count - pos) > 0.
-------------
PR: https://git.openjdk.org/jdk/pull/6935
More information about the core-libs-dev
mailing list