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 f742967176a8c8313645d2091d53071ece42c104
Author: Pascal Schumacher <[email protected]>
AuthorDate: Mon Apr 6 20:30:15 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/maven/packaging/XRefCheckMojo.java | 4 ++--
31 files changed, 49 insertions(+), 49 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/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 fad050b..38e508e 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
@@ -119,8 +119,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());