This is an automated email from the ASF dual-hosted git repository.

pascalschumacher pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 93e11b338ca61415a84c500622ac8f7a2024ce99
Author: Pascal Schumacher <pascalschumac...@gmx.net>
AuthorDate: Sun Apr 5 23:22:55 2020 +0200

    Switch to faster String#indexOf method taking a char parameter.
---
 .../src/main/java/org/apache/camel/parser/RouteBuilderParser.java | 2 +-
 .../src/main/java/org/apache/camel/parser/XmlRouteParser.java     | 2 +-
 .../component/aws/xray/decorators/AbstractSegmentDecorator.java   | 6 +++---
 .../camel/component/aws/xray/component/TestXRayProducer.java      | 2 +-
 .../main/java/org/apache/camel/language/bean/BeanLanguage.java    | 2 +-
 .../java/org/apache/camel/dataformat/bindy/UnicodeHelperTest.java | 8 ++++----
 .../java/org/apache/camel/component/cxf/jaxrs/CxfRsProducer.java  | 4 ++--
 .../java/org/apache/camel/component/dozer/ExpressionMapper.java   | 4 ++--
 .../java/org/apache/camel/component/file/remote/FtpComponent.java | 4 ++--
 .../org/apache/camel/component/file/remote/SftpComponent.java     | 2 +-
 .../google/bigquery/sql/GoogleBigQuerySQLConfiguration.java       | 2 +-
 .../src/main/java/org/apache/camel/http/base/HttpHelper.java      | 2 +-
 .../src/main/java/org/apache/camel/http/common/HttpHelper.java    | 2 +-
 .../main/java/org/apache/camel/component/http/HttpProducer.java   | 2 +-
 .../java/org/apache/camel/component/irc/IrcConfiguration.java     | 2 +-
 .../main/java/org/apache/camel/component/scp/ScpComponent.java    | 2 +-
 .../java/org/apache/camel/component/mail/MailConfiguration.java   | 2 +-
 .../org/apache/camel/component/milo/AbstractMiloServerTest.java   | 2 +-
 .../org/apache/camel/component/netty/http/NettyHttpHelper.java    | 2 +-
 .../main/java/org/apache/camel/component/rest/RestProducer.java   | 4 ++--
 .../salesforce/internal/streaming/SubscriptionHelper.java         | 2 +-
 .../org/apache/camel/component/websocket/WebsocketComponent.java  | 8 ++++----
 .../camel/component/zookeeper/operations/ZooKeeperHelper.java     | 2 +-
 .../java/org/apache/camel/catalog/impl/AbstractCamelCatalog.java  | 6 +++---
 .../src/main/java/org/apache/camel/impl/RouteIdFactory.java       | 2 +-
 .../java/org/apache/camel/builder/xml/XPathTransformTest.java     | 2 +-
 .../src/main/java/org/apache/camel/main/BaseMainSupport.java      | 4 ++--
 .../src/main/java/org/apache/camel/support/ResourceHelper.java    | 2 +-
 .../src/main/java/org/apache/camel/xml/io/XmlStreamReader.java    | 6 +++---
 .../src/test/java/org/apache/camel/xml/in/ModelParserTest.java    | 2 +-
 .../main/java/org/apache/camel/tooling/util/PackageHelper.java    | 6 +++---
 .../org/apache/camel/maven/bom/generator/BomGeneratorMojo.java    | 2 +-
 .../main/java/org/apache/camel/maven/packaging/XRefCheckMojo.java | 4 ++--
 33 files changed, 53 insertions(+), 53 deletions(-)

diff --git 
a/catalog/camel-route-parser/src/main/java/org/apache/camel/parser/RouteBuilderParser.java
 
b/catalog/camel-route-parser/src/main/java/org/apache/camel/parser/RouteBuilderParser.java
index 76d51e1..af5bb24 100644
--- 
a/catalog/camel-route-parser/src/main/java/org/apache/camel/parser/RouteBuilderParser.java
+++ 
b/catalog/camel-route-parser/src/main/java/org/apache/camel/parser/RouteBuilderParser.java
@@ -432,7 +432,7 @@ public final class RouteBuilderParser {
 
     private static String endpointComponentName(String uri) {
         if (uri != null) {
-            int idx = uri.indexOf(":");
+            int idx = uri.indexOf(':');
             if (idx > 0) {
                 return uri.substring(0, idx);
             }
diff --git 
a/catalog/camel-route-parser/src/main/java/org/apache/camel/parser/XmlRouteParser.java
 
b/catalog/camel-route-parser/src/main/java/org/apache/camel/parser/XmlRouteParser.java
index 1c1f9d3..e1a2cb0 100644
--- 
a/catalog/camel-route-parser/src/main/java/org/apache/camel/parser/XmlRouteParser.java
+++ 
b/catalog/camel-route-parser/src/main/java/org/apache/camel/parser/XmlRouteParser.java
@@ -308,7 +308,7 @@ public final class XmlRouteParser {
 
     private static String endpointComponentName(String uri) {
         if (uri != null) {
-            int idx = uri.indexOf(":");
+            int idx = uri.indexOf(':');
             if (idx > 0) {
                 return uri.substring(0, idx);
             }
diff --git 
a/components/camel-aws-xray/src/main/java/org/apache/camel/component/aws/xray/decorators/AbstractSegmentDecorator.java
 
b/components/camel-aws-xray/src/main/java/org/apache/camel/component/aws/xray/decorators/AbstractSegmentDecorator.java
index aeb457b..c5e0adc 100644
--- 
a/components/camel-aws-xray/src/main/java/org/apache/camel/component/aws/xray/decorators/AbstractSegmentDecorator.java
+++ 
b/components/camel-aws-xray/src/main/java/org/apache/camel/component/aws/xray/decorators/AbstractSegmentDecorator.java
@@ -67,13 +67,13 @@ public abstract class AbstractSegmentDecorator implements 
SegmentDecorator {
      * @return The stripped value from the URI
      */
     public static String stripSchemeAndOptions(Endpoint endpoint) {
-        int start = endpoint.getEndpointUri().indexOf(":");
+        int start = endpoint.getEndpointUri().indexOf(':');
         start++;
         // Remove any leading '/'
         while (endpoint.getEndpointUri().charAt(start) == '/') {
             start++;
         }
-        int end = endpoint.getEndpointUri().indexOf("?");
+        int end = endpoint.getEndpointUri().indexOf('?');
         return end == -1 ? endpoint.getEndpointUri().substring(start) : 
endpoint.getEndpointUri().substring(start, end);
     }
 
@@ -84,7 +84,7 @@ public abstract class AbstractSegmentDecorator implements 
SegmentDecorator {
      * @return A {@link Map} representation of the contained parameters of the 
provided URI
      */
     public static Map<String,  String> toQueryParameters(String uri) {
-        int index = uri.indexOf("?");
+        int index = uri.indexOf('?');
         if (index != -1) {
             String queryString = uri.substring(index + 1);
             Map<String, String> map = new HashMap<>();
diff --git 
a/components/camel-aws-xray/src/test/java/org/apache/camel/component/aws/xray/component/TestXRayProducer.java
 
b/components/camel-aws-xray/src/test/java/org/apache/camel/component/aws/xray/component/TestXRayProducer.java
index f77fac2..23e7d17 100644
--- 
a/components/camel-aws-xray/src/test/java/org/apache/camel/component/aws/xray/component/TestXRayProducer.java
+++ 
b/components/camel-aws-xray/src/test/java/org/apache/camel/component/aws/xray/component/TestXRayProducer.java
@@ -47,6 +47,6 @@ public class TestXRayProducer extends DefaultProducer {
     }
 
     private static String trim(String endpoint) {
-        return endpoint.substring(endpoint.indexOf(":") + 1);
+        return endpoint.substring(endpoint.indexOf(':') + 1);
     }
 }
diff --git 
a/components/camel-bean/src/main/java/org/apache/camel/language/bean/BeanLanguage.java
 
b/components/camel-bean/src/main/java/org/apache/camel/language/bean/BeanLanguage.java
index bde8853..2ae6b7b 100644
--- 
a/components/camel-bean/src/main/java/org/apache/camel/language/bean/BeanLanguage.java
+++ 
b/components/camel-bean/src/main/java/org/apache/camel/language/bean/BeanLanguage.java
@@ -129,7 +129,7 @@ public class BeanLanguage extends LanguageSupport 
implements GeneratedPropertyCo
             //first check case :: because of my.own.Bean::method
             int doubleColonIndex = expression.indexOf("::");
             //need to check that not inside params
-            int beginOfParameterDeclaration = expression.indexOf("(");
+            int beginOfParameterDeclaration = expression.indexOf('(');
             if (doubleColonIndex > 0 && (!expression.contains("(") || 
doubleColonIndex < beginOfParameterDeclaration)) {
                 beanName = expression.substring(0, doubleColonIndex);
                 method = expression.substring(doubleColonIndex + 2);
diff --git 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/UnicodeHelperTest.java
 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/UnicodeHelperTest.java
index b2e7fdb..0dc5533 100644
--- 
a/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/UnicodeHelperTest.java
+++ 
b/components/camel-bindy/src/test/java/org/apache/camel/dataformat/bindy/UnicodeHelperTest.java
@@ -166,7 +166,7 @@ public class UnicodeHelperTest {
     @Test
     public void testIndexOf() {
         final UnicodeHelper lh = new UnicodeHelper("a", Method.CODEPOINTS);
-        Assert.assertEquals(-1, lh.indexOf("b"));
+        Assert.assertEquals(-1, lh.indexOf('b'));
 
         final UnicodeHelper lh2 = new UnicodeHelper(
             "a" + new String(Character.toChars(0x1f600)) + "a" + UCSTR + "A" + 
"k\u035fh" + "z"
@@ -186,7 +186,7 @@ public class UnicodeHelperTest {
     @Test
     public void testIndexOf2() {
         final UnicodeHelper lh = new UnicodeHelper("a", Method.GRAPHEME);
-        Assert.assertEquals(-1, lh.indexOf("b"));
+        Assert.assertEquals(-1, lh.indexOf('b'));
 
         final UnicodeHelper lh2 = new UnicodeHelper(
             "a" + new String(Character.toChars(0x1f600)) + "a" + UCSTR + "A" + 
"k\u035fh" + "z"
@@ -201,8 +201,8 @@ public class UnicodeHelperTest {
         Assert.assertEquals(11, lh2.indexOf(UCSTR), 8);
         
         final UnicodeHelper lh3 = new UnicodeHelper("mm̂mm̂m", 
Method.GRAPHEME);
-        Assert.assertEquals(0, lh3.indexOf("m"));
-        Assert.assertEquals(2, lh3.indexOf("m", 1));
+        Assert.assertEquals(0, lh3.indexOf('m'));
+        Assert.assertEquals(2, lh3.indexOf('m', 1));
         Assert.assertEquals(3, lh3.indexOf("m̂", 2));
     }
     
diff --git 
a/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/jaxrs/CxfRsProducer.java
 
b/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/jaxrs/CxfRsProducer.java
index feedf4f..0d57864 100644
--- 
a/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/jaxrs/CxfRsProducer.java
+++ 
b/components/camel-cxf/src/main/java/org/apache/camel/component/cxf/jaxrs/CxfRsProducer.java
@@ -264,8 +264,8 @@ public class CxfRsProducer extends DefaultAsyncProducer {
         if (cxfMessage != null) {
             String requestURL = 
(String)cxfMessage.get("org.apache.cxf.request.uri"); 
             String matrixParam = null;
-            int matrixStart = requestURL.indexOf(";");
-            int matrixEnd = requestURL.indexOf("?") > -1 ? 
requestURL.indexOf("?") : requestURL.length();
+            int matrixStart = requestURL.indexOf(';');
+            int matrixEnd = requestURL.indexOf('?') > -1 ? 
requestURL.indexOf('?') : requestURL.length();
             Map<String, String> maps = null;
             if (requestURL != null && matrixStart > 0) {
                 matrixParam = requestURL.substring(matrixStart + 1, matrixEnd);
diff --git 
a/components/camel-dozer/src/main/java/org/apache/camel/component/dozer/ExpressionMapper.java
 
b/components/camel-dozer/src/main/java/org/apache/camel/component/dozer/ExpressionMapper.java
index f9ba88e..0d7edf8 100644
--- 
a/components/camel-dozer/src/main/java/org/apache/camel/component/dozer/ExpressionMapper.java
+++ 
b/components/camel-dozer/src/main/java/org/apache/camel/component/dozer/ExpressionMapper.java
@@ -112,14 +112,14 @@ public class ExpressionMapper extends BaseConverter {
      * The actual expression, without the language prefix.
      */
     public String getExpressionPart() {
-        return getParameter().substring(getParameter().indexOf(":") + 1);
+        return getParameter().substring(getParameter().indexOf(':') + 1);
     }
     
     /**
      * The expression language used for this mapping.
      */
     public String getLanguagePart() {
-        return getParameter().substring(0, getParameter().indexOf(":"));
+        return getParameter().substring(0, getParameter().indexOf(':'));
     }
     
     /**
diff --git 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/FtpComponent.java
 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/FtpComponent.java
index e78bfd0..c0b4dbb 100644
--- 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/FtpComponent.java
+++ 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/FtpComponent.java
@@ -70,8 +70,8 @@ public class FtpComponent extends 
RemoteFileComponent<FTPFile> {
      */
     protected String getBaseUri(String uri) {
         String baseUri = uri;
-        if (uri.indexOf("?") != -1) {
-            baseUri = uri.substring(0, uri.indexOf("?"));
+        if (uri.indexOf('?') != -1) {
+            baseUri = uri.substring(0, uri.indexOf('?'));
         }
         return baseUri;
     }
diff --git 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/SftpComponent.java
 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/SftpComponent.java
index 697bac9..d3fd36d 100644
--- 
a/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/SftpComponent.java
+++ 
b/components/camel-ftp/src/main/java/org/apache/camel/component/file/remote/SftpComponent.java
@@ -48,7 +48,7 @@ public class SftpComponent extends 
RemoteFileComponent<SftpRemoteFile> {
         // to escape the $ for the expression (file language)
         String baseUri = uri;
         if (uri.contains("?")) {
-            baseUri = uri.substring(0, uri.indexOf("?"));
+            baseUri = uri.substring(0, uri.indexOf('?'));
         }
 
         // lets make sure we create a new configuration as each endpoint can
diff --git 
a/components/camel-google-bigquery/src/main/java/org/apache/camel/component/google/bigquery/sql/GoogleBigQuerySQLConfiguration.java
 
b/components/camel-google-bigquery/src/main/java/org/apache/camel/component/google/bigquery/sql/GoogleBigQuerySQLConfiguration.java
index 0f9143a..1e918fc 100644
--- 
a/components/camel-google-bigquery/src/main/java/org/apache/camel/component/google/bigquery/sql/GoogleBigQuerySQLConfiguration.java
+++ 
b/components/camel-google-bigquery/src/main/java/org/apache/camel/component/google/bigquery/sql/GoogleBigQuerySQLConfiguration.java
@@ -35,7 +35,7 @@ public class GoogleBigQuerySQLConfiguration {
     private String query;
 
     public void parseRemaining(String remaining) {
-        int indexOfColon = remaining.indexOf(":");
+        int indexOfColon = remaining.indexOf(':');
 
         if (indexOfColon < 0) {
             throw new IllegalArgumentException("Google BigQuery Endpoint 
format \"projectId:query\"");
diff --git 
a/components/camel-http-base/src/main/java/org/apache/camel/http/base/HttpHelper.java
 
b/components/camel-http-base/src/main/java/org/apache/camel/http/base/HttpHelper.java
index 9ddd58c..1073618 100644
--- 
a/components/camel-http-base/src/main/java/org/apache/camel/http/base/HttpHelper.java
+++ 
b/components/camel-http-base/src/main/java/org/apache/camel/http/base/HttpHelper.java
@@ -46,7 +46,7 @@ public final class HttpHelper {
             throw new ProtocolException("Invalid HTTP version string: " + s);
         }
         int i1 = "HTTP/".length();
-        int i2 = s.indexOf(".", i1);
+        int i2 = s.indexOf('.', i1);
         if (i2 == -1) {
             throw new ProtocolException("Invalid HTTP version number: " + s);
         }
diff --git 
a/components/camel-http-common/src/main/java/org/apache/camel/http/common/HttpHelper.java
 
b/components/camel-http-common/src/main/java/org/apache/camel/http/common/HttpHelper.java
index 66a3d1c..1dde940 100644
--- 
a/components/camel-http-common/src/main/java/org/apache/camel/http/common/HttpHelper.java
+++ 
b/components/camel-http-common/src/main/java/org/apache/camel/http/common/HttpHelper.java
@@ -223,7 +223,7 @@ public final class HttpHelper {
             }
             if (path.length() > 0) {
                 // inject the dynamic path before the query params, if there 
are any
-                int idx = uri.indexOf("?");
+                int idx = uri.indexOf('?');
 
                 // if there are no query params
                 if (idx == -1) {
diff --git 
a/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java
 
b/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java
index 9d58cbb..b688a5e 100644
--- 
a/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java
+++ 
b/components/camel-http/src/main/java/org/apache/camel/component/http/HttpProducer.java
@@ -509,7 +509,7 @@ public class HttpProducer extends DefaultProducer {
                     //it removes "boundary" from Content-Type; I have to use 
contentType.create method.
                     if (contentTypeString != null) {
                         // using ContentType.parser for charset
-                        if (contentTypeString.indexOf("charset") > 0 || 
contentTypeString.indexOf(";") > 0) {
+                        if (contentTypeString.indexOf("charset") > 0 || 
contentTypeString.indexOf(';') > 0) {
                             contentType = ContentType.parse(contentTypeString);
                         } else {
                             contentType = 
ContentType.create(contentTypeString);
diff --git 
a/components/camel-irc/src/main/java/org/apache/camel/component/irc/IrcConfiguration.java
 
b/components/camel-irc/src/main/java/org/apache/camel/component/irc/IrcConfiguration.java
index 5443e2d..1b3fb65 100644
--- 
a/components/camel-irc/src/main/java/org/apache/camel/component/irc/IrcConfiguration.java
+++ 
b/components/camel-irc/src/main/java/org/apache/camel/component/irc/IrcConfiguration.java
@@ -160,7 +160,7 @@ public class IrcConfiguration implements Cloneable {
         String username = null;
         String password = null;
         if (userInfo != null) {
-            int colonIndex = userInfo.indexOf(":");
+            int colonIndex = userInfo.indexOf(':');
             if (colonIndex != -1) {
                 username = userInfo.substring(0, colonIndex);
                 password = userInfo.substring(colonIndex + 1);
diff --git 
a/components/camel-jsch/src/main/java/org/apache/camel/component/scp/ScpComponent.java
 
b/components/camel-jsch/src/main/java/org/apache/camel/component/scp/ScpComponent.java
index 5ee2311..f1fc631 100644
--- 
a/components/camel-jsch/src/main/java/org/apache/camel/component/scp/ScpComponent.java
+++ 
b/components/camel-jsch/src/main/java/org/apache/camel/component/scp/ScpComponent.java
@@ -44,7 +44,7 @@ public class ScpComponent extends 
RemoteFileComponent<ScpFile> {
 
     @Override
     protected GenericFileEndpoint<ScpFile> buildFileEndpoint(String uri, 
String remaining, Map<String, Object> parameters) throws Exception {
-        int query = uri.indexOf("?");
+        int query = uri.indexOf('?');
         return new ScpEndpoint(uri, this, new ScpConfiguration(new URI(query 
>= 0 ? uri.substring(0, query) : uri)));
     }
 
diff --git 
a/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailConfiguration.java
 
b/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailConfiguration.java
index 2688c19..29d04d1 100644
--- 
a/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailConfiguration.java
+++ 
b/components/camel-mail/src/main/java/org/apache/camel/component/mail/MailConfiguration.java
@@ -455,7 +455,7 @@ public class MailConfiguration implements Cloneable {
             // set default destination to username@host for backwards 
compatibility
             // can be overridden by URI parameters
             String address = username;
-            if (address.indexOf("@") == -1) {
+            if (address.indexOf('@') == -1) {
                 address += "@" + host;
             }
             setTo(address);
diff --git 
a/components/camel-milo/src/test/java/org/apache/camel/component/milo/AbstractMiloServerTest.java
 
b/components/camel-milo/src/test/java/org/apache/camel/component/milo/AbstractMiloServerTest.java
index 085beb7..c2c6989 100644
--- 
a/components/camel-milo/src/test/java/org/apache/camel/component/milo/AbstractMiloServerTest.java
+++ 
b/components/camel-milo/src/test/java/org/apache/camel/component/milo/AbstractMiloServerTest.java
@@ -132,7 +132,7 @@ public abstract class AbstractMiloServerTest extends 
CamelTestSupport {
     boolean isJavaVersionSatisfied(int requiredVersion) {
         String version = System.getProperty("java.version");
         if (!version.startsWith("1.")) {
-            int dot = version.indexOf(".");
+            int dot = version.indexOf('.');
             if (dot != -1) {
                 version = version.substring(0, dot);
             }
diff --git 
a/components/camel-netty-http/src/main/java/org/apache/camel/component/netty/http/NettyHttpHelper.java
 
b/components/camel-netty-http/src/main/java/org/apache/camel/component/netty/http/NettyHttpHelper.java
index c383152..be42101 100644
--- 
a/components/camel-netty-http/src/main/java/org/apache/camel/component/netty/http/NettyHttpHelper.java
+++ 
b/components/camel-netty-http/src/main/java/org/apache/camel/component/netty/http/NettyHttpHelper.java
@@ -207,7 +207,7 @@ public final class NettyHttpHelper {
             }
 
             // inject the dynamic path before the query params, if there are 
any
-            int idx = uri.indexOf("?");
+            int idx = uri.indexOf('?');
 
             // if there are no query params
             if (idx == -1) {
diff --git 
a/components/camel-rest/src/main/java/org/apache/camel/component/rest/RestProducer.java
 
b/components/camel-rest/src/main/java/org/apache/camel/component/rest/RestProducer.java
index 2f8be9f..121d2aa 100644
--- 
a/components/camel-rest/src/main/java/org/apache/camel/component/rest/RestProducer.java
+++ 
b/components/camel-rest/src/main/java/org/apache/camel/component/rest/RestProducer.java
@@ -232,8 +232,8 @@ public class RestProducer extends DefaultAsyncProducer {
     private String resolveHeaderPlaceholders(String str, Message msg) {
         int startIndex = -1;
         String res = str;
-        while ((startIndex = res.indexOf("{", startIndex + 1)) >= 0) {
-            int endIndex = res.indexOf("}", startIndex);
+        while ((startIndex = res.indexOf('{', startIndex + 1)) >= 0) {
+            int endIndex = res.indexOf('}', startIndex);
             if (endIndex == -1) {
                 continue;
             }
diff --git 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/streaming/SubscriptionHelper.java
 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/streaming/SubscriptionHelper.java
index 3960ef1..8bb3c20 100644
--- 
a/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/streaming/SubscriptionHelper.java
+++ 
b/components/camel-salesforce/camel-salesforce-component/src/main/java/org/apache/camel/component/salesforce/internal/streaming/SubscriptionHelper.java
@@ -526,7 +526,7 @@ public class SubscriptionHelper extends ServiceSupport {
             channelName.append(topicName);
         }
 
-        final int typeIdx = channelName.indexOf("/", 1);
+        final int typeIdx = channelName.indexOf('/', 1);
         if ("event".equals(channelName.substring(1, typeIdx)) && 
!topicName.endsWith("__e")) {
             channelName.append("__e");
         }
diff --git 
a/components/camel-websocket/src/main/java/org/apache/camel/component/websocket/WebsocketComponent.java
 
b/components/camel-websocket/src/main/java/org/apache/camel/component/websocket/WebsocketComponent.java
index 3a34e99..9c563a8 100644
--- 
a/components/camel-websocket/src/main/java/org/apache/camel/component/websocket/WebsocketComponent.java
+++ 
b/components/camel-websocket/src/main/java/org/apache/camel/component/websocket/WebsocketComponent.java
@@ -550,7 +550,7 @@ public class WebsocketComponent extends DefaultComponent 
implements SSLContextPa
         // Is not correct as it does not support to add port in the URI
         //return String.format("/%s/*", remaining);
 
-        int index = remaining.indexOf("/");
+        int index = remaining.indexOf('/');
         if (index != -1) {
             return remaining.substring(index, remaining.length());
         } else {
@@ -559,8 +559,8 @@ public class WebsocketComponent extends DefaultComponent 
implements SSLContextPa
     }
 
     private int extractPortNumber(String remaining) {
-        int index1 = remaining.indexOf(":");
-        int index2 = remaining.indexOf("/");
+        int index1 = remaining.indexOf(':');
+        int index2 = remaining.indexOf('/');
 
         if ((index1 != -1) && (index2 != -1)) {
             String result = remaining.substring(index1 + 1, index2);
@@ -571,7 +571,7 @@ public class WebsocketComponent extends DefaultComponent 
implements SSLContextPa
     }
 
     private String extractHostName(String remaining) {
-        int index = remaining.indexOf(":");
+        int index = remaining.indexOf(':');
         if (index != -1) {
             return remaining.substring(0, index);
         } else {
diff --git 
a/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/operations/ZooKeeperHelper.java
 
b/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/operations/ZooKeeperHelper.java
index 89a8813..13777f2 100644
--- 
a/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/operations/ZooKeeperHelper.java
+++ 
b/components/camel-zookeeper/src/main/java/org/apache/camel/component/zookeeper/operations/ZooKeeperHelper.java
@@ -32,7 +32,7 @@ public final class ZooKeeperHelper {
 
         int pos = 1; // skip first slash, root is guaranteed to exist
         do {
-            pos = path.indexOf("/", pos + 1);
+            pos = path.indexOf('/', pos + 1);
 
             if (pos == -1) {
                 if (makeLastNode) {
diff --git 
a/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/AbstractCamelCatalog.java
 
b/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/AbstractCamelCatalog.java
index 3dab3f7..e0d746f 100644
--- 
a/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/AbstractCamelCatalog.java
+++ 
b/core/camel-core-catalog/src/main/java/org/apache/camel/catalog/impl/AbstractCamelCatalog.java
@@ -397,7 +397,7 @@ public abstract class AbstractCamelCatalog {
             // clip the scheme from the syntax
             alternativeSyntax = CatalogHelper.after(alternativeSyntax, ":");
             // trim so only userinfo
-            int idx = alternativeSyntax.indexOf("@");
+            int idx = alternativeSyntax.indexOf('@');
             String fields = alternativeSyntax.substring(0, idx);
             String[] names = fields.split(":");
 
@@ -408,7 +408,7 @@ public abstract class AbstractCamelCatalog {
                 String password = null;
 
                 // grab unserinfo part before @
-                String userInfo = authority.substring(0, 
authority.indexOf("@"));
+                String userInfo = authority.substring(0, 
authority.indexOf('@'));
                 String[] parts = userInfo.split(":");
                 if (parts.length == 2) {
                     username = parts[0];
@@ -661,7 +661,7 @@ public abstract class AbstractCamelCatalog {
 
     public String endpointComponentName(String uri) {
         if (uri != null) {
-            int idx = uri.indexOf(":");
+            int idx = uri.indexOf(':');
             if (idx > 0) {
                 return uri.substring(0, idx);
             }
diff --git 
a/core/camel-core-engine/src/main/java/org/apache/camel/impl/RouteIdFactory.java
 
b/core/camel-core-engine/src/main/java/org/apache/camel/impl/RouteIdFactory.java
index 8a7eddd..bef8fa6 100644
--- 
a/core/camel-core-engine/src/main/java/org/apache/camel/impl/RouteIdFactory.java
+++ 
b/core/camel-core-engine/src/main/java/org/apache/camel/impl/RouteIdFactory.java
@@ -97,7 +97,7 @@ public class RouteIdFactory implements NodeIdFactory {
         if (colon > 0) {
             String name = uri.substring(colon + 1);
 
-            int questionMark = name.indexOf("?");
+            int questionMark = name.indexOf('?');
 
             if (questionMark > 0) {
                 return Optional.of(name.substring(0, questionMark));
diff --git 
a/core/camel-core/src/test/java/org/apache/camel/builder/xml/XPathTransformTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/builder/xml/XPathTransformTest.java
index 559ae1f..046fada 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/builder/xml/XPathTransformTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/builder/xml/XPathTransformTest.java
@@ -114,7 +114,7 @@ public class XPathTransformTest extends ContextTestSupport {
         if (version.startsWith("1.")) {
             version = version.substring(2, 3);
         } else {
-            int dot = version.indexOf(".");
+            int dot = version.indexOf('.');
             if (dot != -1) {
                 version = version.substring(0, dot);
             }
diff --git 
a/core/camel-main/src/main/java/org/apache/camel/main/BaseMainSupport.java 
b/core/camel-main/src/main/java/org/apache/camel/main/BaseMainSupport.java
index 1679fcf..abb1983 100644
--- a/core/camel-main/src/main/java/org/apache/camel/main/BaseMainSupport.java
+++ b/core/camel-main/src/main/java/org/apache/camel/main/BaseMainSupport.java
@@ -874,7 +874,7 @@ public abstract class BaseMainSupport extends BaseService {
 
         for (String key : prop.stringPropertyNames()) {
             if (key.startsWith("camel.component.properties.")) {
-                int dot = key.indexOf(".", 26);
+                int dot = key.indexOf('.', 26);
                 String option = dot == -1 ? "" : key.substring(dot + 1);
                 String value = prop.getProperty(key, "");
                 validateOptionAndValue(key, option, value);
@@ -1208,7 +1208,7 @@ public abstract class BaseMainSupport extends BaseService 
{
                                             Function<String, Iterable<Object>> 
supplier) {
         if (key.startsWith(keyPrefix)) {
             // grab name
-            final int dot = key.indexOf(".", keyPrefix.length());
+            final int dot = key.indexOf('.', keyPrefix.length());
             final String name = dot == -1 ? key.substring(keyPrefix.length()) 
: key.substring(keyPrefix.length(), dot);
 
             // enabled is a virtual property
diff --git 
a/core/camel-support/src/main/java/org/apache/camel/support/ResourceHelper.java 
b/core/camel-support/src/main/java/org/apache/camel/support/ResourceHelper.java
index 2d4df7a..8046832 100644
--- 
a/core/camel-support/src/main/java/org/apache/camel/support/ResourceHelper.java
+++ 
b/core/camel-support/src/main/java/org/apache/camel/support/ResourceHelper.java
@@ -80,7 +80,7 @@ public final class ResourceHelper {
      */
     public static String getScheme(String uri) {
         if (hasScheme(uri)) {
-            return uri.substring(0, uri.indexOf(":") + 1);
+            return uri.substring(0, uri.indexOf(':') + 1);
         } else {
             return null;
         }
diff --git 
a/core/camel-xml-io/src/main/java/org/apache/camel/xml/io/XmlStreamReader.java 
b/core/camel-xml-io/src/main/java/org/apache/camel/xml/io/XmlStreamReader.java
index 2102b20..fea01f3 100644
--- 
a/core/camel-xml-io/src/main/java/org/apache/camel/xml/io/XmlStreamReader.java
+++ 
b/core/camel-xml-io/src/main/java/org/apache/camel/xml/io/XmlStreamReader.java
@@ -640,7 +640,7 @@ public class XmlStreamReader extends Reader {
     private static String getContentTypeMime(final String httpContentType) {
         String mime = null;
         if (httpContentType != null) {
-            final int i = httpContentType.indexOf(";");
+            final int i = httpContentType.indexOf(';');
             if (i == -1) {
                 mime = httpContentType.trim();
             } else {
@@ -655,7 +655,7 @@ public class XmlStreamReader extends Reader {
     private static String getContentTypeEncoding(final String httpContentType) 
{
         String encoding = null;
         if (httpContentType != null) {
-            final int i = httpContentType.indexOf(";");
+            final int i = httpContentType.indexOf(';');
             if (i > -1) {
                 final String postMime = httpContentType.substring(i + 1);
                 final Matcher m = CHARSET_PATTERN.matcher(postMime);
@@ -739,7 +739,7 @@ public class XmlStreamReader extends Reader {
                 offset += c;
                 max -= c;
                 c = is.read(bytes, offset, max);
-                firstGT = new String(bytes, 0, offset, 
guessedEnc).indexOf(">");
+                firstGT = new String(bytes, 0, offset, 
guessedEnc).indexOf('>');
             }
             if (firstGT == -1) {
                 if (c == -1) {
diff --git 
a/core/camel-xml-io/src/test/java/org/apache/camel/xml/in/ModelParserTest.java 
b/core/camel-xml-io/src/test/java/org/apache/camel/xml/in/ModelParserTest.java
index 05322bb..3f1a1ce 100644
--- 
a/core/camel-xml-io/src/test/java/org/apache/camel/xml/in/ModelParserTest.java
+++ 
b/core/camel-xml-io/src/test/java/org/apache/camel/xml/in/ModelParserTest.java
@@ -64,7 +64,7 @@ public class ModelParserTest {
         if (url.startsWith("file:")) {
             url = url.substring("file:".length(), 
url.indexOf("barInterceptorRoute.xml"));
         } else if (url.startsWith("jar:file:")) {
-            url = url.substring("jar:file:".length(), url.indexOf("!"));
+            url = url.substring("jar:file:".length(), url.indexOf('!'));
         }
         return Paths.get(url);
     }
diff --git 
a/tooling/camel-tooling-util/src/main/java/org/apache/camel/tooling/util/PackageHelper.java
 
b/tooling/camel-tooling-util/src/main/java/org/apache/camel/tooling/util/PackageHelper.java
index 49c6751..3c782ec 100644
--- 
a/tooling/camel-tooling-util/src/main/java/org/apache/camel/tooling/util/PackageHelper.java
+++ 
b/tooling/camel-tooling-util/src/main/java/org/apache/camel/tooling/util/PackageHelper.java
@@ -166,11 +166,11 @@ public final class PackageHelper {
      * Extract the model kind from a given json schema
      */
     public static String getSchemaKind(String json) {
-        int i = json.indexOf("\"kind\"");
+        int i = json.indexOf('\'kind\"");
         if (i >= 0) {
-            int s = json.indexOf("\"", i + 6);
+            int s = json.indexOf('\'", i + 6);
             if (s >= 0) {
-                int e = json.indexOf("\"", s + 1);
+                int e = json.indexOf('\'", s + 1);
                 if (e >= 0) {
                     return json.substring(s + 1, e);
                 }
diff --git 
a/tooling/maven/bom-generator-maven-plugin/src/main/java/org/apache/camel/maven/bom/generator/BomGeneratorMojo.java
 
b/tooling/maven/bom-generator-maven-plugin/src/main/java/org/apache/camel/maven/bom/generator/BomGeneratorMojo.java
index bc1d346..0f4e389 100644
--- 
a/tooling/maven/bom-generator-maven-plugin/src/main/java/org/apache/camel/maven/bom/generator/BomGeneratorMojo.java
+++ 
b/tooling/maven/bom-generator-maven-plugin/src/main/java/org/apache/camel/maven/bom/generator/BomGeneratorMojo.java
@@ -384,7 +384,7 @@ public class BomGeneratorMojo extends AbstractMojo {
     private String resolveVersion(MavenProject project, String version) {
         if (version.contains("${")) {
             int start = version.indexOf("${");
-            int end = version.indexOf("}");
+            int end = version.indexOf('}');
             if (end > start) {
                 String prop = version.substring(start + 2, end);
                 String resolved = project.getProperties().getProperty(prop);
diff --git 
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/XRefCheckMojo.java
 
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/XRefCheckMojo.java
index 979d201..c4f49d7 100644
--- 
a/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/XRefCheckMojo.java
+++ 
b/tooling/maven/camel-package-maven-plugin/src/main/java/org/apache/camel/maven/packaging/XRefCheckMojo.java
@@ -110,8 +110,8 @@ public class XRefCheckMojo extends AbstractMojo {
 
         Pattern xref = 
Pattern.compile("\\b(?<all>xref:(?<link>[^\\[]+.adoc)[^\\]]*\\])");
         for (Map.Entry<String, Path> page : pages.entrySet()) {
-            int firstDot = page.getKey().indexOf(":");
-            int secondDot = firstDot >= 0 ? page.getKey().indexOf(":", 
firstDot + 1) : -1;
+            int firstDot = page.getKey().indexOf(':');
+            int secondDot = firstDot >= 0 ? page.getKey().indexOf(':', 
firstDot + 1) : -1;
             String component = secondDot >= 0 ? page.getKey().substring(0, 
firstDot) : "";
             String module = secondDot >= 0 ? page.getKey().substring(firstDot 
+ 1, secondDot) : "";
             String str = PackageHelper.loadText(page.getValue());

Reply via email to