Re: RFR: 8389573: InputStreamReader.readAllAsString() should override the generic Reader default implementation to avoid unnecessary buffer copies [v6]

[Markus KARG]

On Sat, 29 Aug 2026 14:49:30 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

>> Markus KARG has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Removed readCalled=true from read() and implRead()
>
> test/jdk/java/io/Reader/ReadAll.java line 199:
> 
>> 197:         assertEquals(stringExpected.substring(n), string);
>> 198: 
>> 199:         // InputStreamReader implementation: Called directly after 
>> construction (Fast Path)
> 
> I haven't focused on the test yet, but just a drive by comment for now - I 
> think we should remove the mentions of fast/slow path from this test update.

Correct. Removed it in latest commit.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/32264#discussion_r3924310613