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

Chen Liang liach at openjdk.org
Fri Mar 24 18:02:35 UTC 2023


On Fri, 24 Mar 2023 17:02:30 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: Amend copyright

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

> 1: /*

Need to update the line ending of this file, like through intellij idea bottom right or sublime text view->line endings

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

> 1: /*
> 2:  * Copyright (c) 2020, 2023, Oracle and/or its affiliates. All rights reserved.

Suggestion:

 * Copyright (c) 2023, Oracle and/or its affiliates. All rights reserved.

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

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


More information about the core-libs-dev mailing list