RFR: 8343110: Add getChars(int, int, char[], int) to CharSequence and CharBuffer [v5]

Markus KARG duke at openjdk.org
Wed Apr 16 16:12:58 UTC 2025


On Wed, 16 Apr 2025 16:08:21 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

>> Markus KARG has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Applied changes requested by Chen: 'We might need to specify the IOOBE behavior - when an IOOBE is thrown, some characters may be already transferred (this is important for concurrent char sequences)'
>
> src/java.base/share/classes/java/lang/CharSequence.java line 342:
> 
>> 340:      * @since 25
>> 341:      */
>> 342:     public default void getChars(int srcBegin, int srcEnd, char[] dst, int dstBegin) {
> 
> Hello Markus, given these new methods on `CharSequence` and the `CharBuffer`, we should add a jtreg test to verify that the default implementation of these methods do work as specified. 
> 
> Edit: I now see Chen too had a similar comment about adding a regression test.

This is already discussed in a separate thread.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21730#discussion_r2047265794


More information about the core-libs-dev mailing list