RFR: 8263560: Remove needless wrapping with BufferedInputStream [v6]
Daniel Fuchs
dfuchs at openjdk.java.net
Mon Mar 15 18:04:27 UTC 2021
On Mon, 15 Mar 2021 18:01:20 GMT, Сергей Цыпанов <github.com+10835776+stsypanov at openjdk.org> wrote:
>> In some cases wrapping of `InputStream` with `BufferedInputStream` is redundant, e.g. in case the wrapped one is `ByteArrayOutputStream` which does not require any buffer having one within.
>>
>> Other cases are related to reading either a byte or short `byte[]`: in both cases `BufferedInputStream.fill()` will be called resulting in load of much bigger amount of data (8192 by default) than required.
>
> Сергей Цыпанов has updated the pull request incrementally with one additional commit since the last revision:
>
> Revert HttpClient
LGTM. Thanks!
-------------
Marked as reviewed by dfuchs (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/2992
More information about the net-dev
mailing list