RFR: 8287497: Use String.contains() instead of String.indexOf() in java.naming

Aleksei Efimov aefimov at openjdk.java.net
Mon May 30 12:58:34 UTC 2022


On Sun, 29 May 2022 14:00:20 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

> `String.contains` was introduced in Java 5.
> Some code in java.naming still uses old approach with `String.indexOf` to check if String contains specified substring.
> I propose to migrate such usages. Makes code shorter and easier to read.

The change looks good to me.
I've checked the fix with our CI system and no `java.naming` failures have been discovered.

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

Marked as reviewed by aefimov (Committer).

PR: https://git.openjdk.java.net/jdk/pull/8938


More information about the core-libs-dev mailing list