RFR: 8302590: Add String.indexOf(int ch, int fromIndex, int toIndex) [v7]

Alan Bateman alanb at openjdk.org
Fri Mar 3 18:07:25 UTC 2023


On Thu, 2 Mar 2023 10:24:03 GMT, Raffaello Giulietti <rgiulietti at 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


More information about the core-libs-dev mailing list