RFR: 8304745: Lazily initialize byte[] in java.io.BufferedInputStream [v5]

Chen Liang liach at openjdk.org
Fri Mar 24 16:15:42 UTC 2023


On Fri, 24 Mar 2023 15:40:40 GMT, Sergey Tsypanov <stsypanov at openjdk.org> wrote:

>> By default `BufferedInputStream` is constructed with internal buffer with capacity 8192. In some cases this buffer is never used, e.g. when we call `IS.readNBytes()` or `IS.readAllBytes()` (relying on `BIS.read1()`) or when `BufferedInputStream` is cascaded.
>
> Sergey Tsypanov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8304745: Add benchmark

test/micro/org/openjdk/bench/java/io/BufferedInputStreamBenchmark.java line 2:

> 1: /*
> 2:  * Copyright (c) 2020, 2022, Red Hat Inc. All rights reserved.

Copyright year, and is this really from redhat?

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/13150#discussion_r1147794637


More information about the core-libs-dev mailing list