RFR: 8325340: Add ASCII fast-path to Data-/ObjectInputStream.readUTF [v7]
Claes Redestad
redestad at openjdk.org
Mon Feb 26 16:09:03 UTC 2024
On Mon, 26 Feb 2024 15:50:55 GMT, Roger Riggs <rriggs at openjdk.org> wrote:
>> Claes Redestad has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Update test/micro/org/openjdk/bench/java/io/DataInputStreamTest.java
>>
>> Co-authored-by: Raffaello Giulietti <raffaello.giulietti at oracle.com>
>> - Update test/micro/org/openjdk/bench/java/io/ObjectInputStreamTest.java
>>
>> Co-authored-by: Raffaello Giulietti <raffaello.giulietti at oracle.com>
>
> src/java.base/share/classes/java/io/DataInputStream.java line 622:
>
>> 620: "malformed input around byte " + count);
>> 621: chararr[chararr_count++]=(char)(((c & 0x1F) << 6) |
>> 622: (char2 & 0x3F));
>
> Seems like the extra space isn't necessary for alignment.
Fixed, thanks for reviewing!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17734#discussion_r1502894731
More information about the nio-dev
mailing list