On Tue, 21 Mar 2023 15:41:13 GMT, Per Minborg <pminb...@openjdk.org> wrote:
> This PR proposed to lazily initialize the scratch arrays only if/when needed.
Looks good. Many DataInputStreams probably never read UTF. (Not a reviewer)
src/java.base/share/classes/java/io/DataInputStream.java line 576:
> 574: char[] chararr = null;
> 575: if (in instanceof DataInputStream dis) {
> 576: if (dis.bytearr == null || dis.bytearr.length < utflen){
Add a space between ')' and '{'?
Suggestion:
if (dis.bytearr == null || dis.bytearr.length < utflen) {
-------------
Marked as reviewed by eir...@github.com (no known OpenJDK username).
PR Review: https://git.openjdk.org/jdk/pull/13121#pullrequestreview-1350859010
PR Review Comment: https://git.openjdk.org/jdk/pull/13121#discussion_r1143638420
