You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@groovy.apache.org by su...@apache.org on 2020/04/11 11:48:53 UTC

[groovy] 02/03: Switch to faster String#indexOf method taking a char parameter.

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

sunlan pushed a commit to branch GROOVY_3_0_X
in repository https://gitbox.apache.org/repos/asf/groovy.git

commit 9840a11d7f31832d7eacf7b6551e418ecc7f893a
Author: Pascal Schumacher <pa...@gmx.net>
AuthorDate: Sat Apr 11 10:55:23 2020 +0200

    Switch to faster String#indexOf method taking a char parameter.
    
    (cherry picked from commit 6d341b14342cd442141a8422dbad567351c303f5)
---
 src/main/java/groovy/lang/MetaClassImpl.java                     | 2 +-
 src/main/java/org/codehaus/groovy/tools/LoaderConfiguration.java | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/main/java/groovy/lang/MetaClassImpl.java b/src/main/java/groovy/lang/MetaClassImpl.java
index 4ade193..27b19bf 100644
--- a/src/main/java/groovy/lang/MetaClassImpl.java
+++ b/src/main/java/groovy/lang/MetaClassImpl.java
@@ -565,7 +565,7 @@ public class MetaClassImpl implements MetaClass, MutableMetaClass {
             private String[] decomposeMopName(final String mopName) {
                 int idx = mopName.indexOf('$');
                 if (idx > 0) {
-                    int eidx = mopName.indexOf("$", idx + 1);
+                    int eidx = mopName.indexOf('$', idx + 1);
                     if (eidx > 0) {
                         return new String[]{
                                 mopName.substring(0, idx + 1),
diff --git a/src/main/java/org/codehaus/groovy/tools/LoaderConfiguration.java b/src/main/java/org/codehaus/groovy/tools/LoaderConfiguration.java
index 9840317..eb2e261 100644
--- a/src/main/java/org/codehaus/groovy/tools/LoaderConfiguration.java
+++ b/src/main/java/org/codehaus/groovy/tools/LoaderConfiguration.java
@@ -170,7 +170,7 @@ public class LoaderConfiguration {
             if (propertyIndexStart == -1) break;
             result.append(str, propertyIndexEnd, propertyIndexStart);
 
-            propertyIndexEnd = str.indexOf("}", propertyIndexStart);
+            propertyIndexEnd = str.indexOf('}', propertyIndexStart);
             if (propertyIndexEnd == -1) break;
 
             String propertyKey = str.substring(propertyIndexStart + 2, propertyIndexEnd);