On Thu, 2 Mar 2023 10:24:03 GMT, Raffaello Giulietti <rgiulie...@openjdk.org> 
wrote:

>> Add an `indexOf()` variant allowing to specify both a lower and an upper 
>> bound on the search.
>
> Raffaello Giulietti has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   8302590: Add String.indexOf(int ch, int fromIndex, int toIndex)

The latest version looks good. Thanks for going through the process of getting 
this to a good place. One minor comment is that the test should probably in the 
java/lang/String rather than the CompactString sub-directory.

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

Marked as reviewed by alanb (Reviewer).

PR: https://git.openjdk.org/jdk/pull/12600

Reply via email to