RFR: 8343110: Add getChars(int, int, char[], int) to CharSequence and CharBuffer [v3]
Markus KARG
duke at openjdk.org
Wed Mar 26 11:37:22 UTC 2025
On Tue, 25 Mar 2025 13:33:15 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Markus KARG has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Removed apiNote and implNote from CharBuffer, as suggested by Chen
>
> src/java.base/share/classes/java/lang/CharSequence.java line 307:
>
>> 305: /**
>> 306: * Characters are copied from this sequence into the
>> 307: * destination character array {@code dst}. The first character to
>
> I think the "Characters are copied ..." sentence goes back to StringBuffer in JDK 1.0. CS doesn't need to copy this exactly and I think better to say that it copies chars from this sequence into the given destination array.
Do you like this more: `Copies chars from this sequence into the given destination array.` ?
> src/java.base/share/classes/java/lang/CharSequence.java line 335:
>
>> 333: *
>> 334: * @implSpec
>> 335: * The default implementation iterates over {@link #charAt(int)}.
>
> This sentence doesn't make sense, did something get deleted?
Do you like this more: `The default implementation invokes {@link #charAt(int)} in a loop.`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/21730#discussion_r2013947462
PR Review Comment: https://git.openjdk.org/jdk/pull/21730#discussion_r2013944760
More information about the nio-dev
mailing list