On Sun, 14 Mar 2021 19:35:25 GMT, Сергей Цыпанов
<[email protected]> 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:
>
> Use InputStream.readNBytes() and fix JLinkNegativeTest
src/java.desktop/share/classes/sun/awt/image/ByteArrayImageSource.java line 54:
> 52:
> 53: protected ImageDecoder getDecoder() {
> 54: InputStream is = new ByteArrayInputStream(imagedata, imageoffset,
> imagelength);
This file use 80 chars per line code style.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2992