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());