RFR: 8341566: Add Reader.of(CharSequence) [v7]
Roger Riggs
rriggs at openjdk.org
Thu Oct 10 14:35:16 UTC 2024
On Thu, 10 Oct 2024 04:53:05 GMT, Chen Liang <liach at openjdk.org> wrote:
>> src/java.base/share/classes/java/io/StringReader.java line 38:
>>
>>> 36: * {@code Reader.of(String)} should generally be used in preference to this one,
>>> 37: * as it supports all of the same operations but it is faster, as it performs no
>>> 38: * synchronization.
>>
>> We shouldn't using "release 24" or "supplemented" here. The API note just needs to `Reader.of(CharSequence)` provides a method to read from any CharSequence that may be more efficient than `StringReader`.
>
> I think "release 24" is fine for now as similar text exists in `java.util.Vector`. We can normalize such texts when we implement https://bugs.openjdk.org/browse/JDK-8341736.
Please remove the reference to JDK 24; it will age rapidly and would require extra maintenance later.
The javadoc should focus on the functionality provided.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21371#discussion_r1795581242
More information about the core-libs-dev
mailing list