[groovy] branch master updated: Trivial refactoring for `JavaStubCompilationUnit.hasAcceptedFileExtension`

[su...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 66596d2  Trivial refactoring for `JavaStubCompilationUnit.hasAcceptedFileExtension`
66596d2 is described below

commit 66596d28f7874a3c9e4fc2726d72f82aed83a7c9
Author: Daniel Sun <su...@apache.org>
AuthorDate: Sun Aug 29 23:12:49 2021 +0800

    Trivial refactoring for `JavaStubCompilationUnit.hasAcceptedFileExtension`
---
 .../org/codehaus/groovy/tools/javac/JavaStubCompilationUnit.java   | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/codehaus/groovy/tools/javac/JavaStubCompilationUnit.java b/src/main/java/org/codehaus/groovy/tools/javac/JavaStubCompilationUnit.java
index 46e1d42..89af49f 100644
--- a/src/main/java/org/codehaus/groovy/tools/javac/JavaStubCompilationUnit.java
+++ b/src/main/java/org/codehaus/groovy/tools/javac/JavaStubCompilationUnit.java
@@ -111,10 +111,7 @@ public class JavaStubCompilationUnit extends CompilationUnit {
 
     private boolean hasAcceptedFileExtension(String name) {
         String lowerCasedName = name.toLowerCase();
-        for (String extension : configuration.getScriptExtensions()) {
-            if (lowerCasedName.endsWith(extension))
-                return true;
-        }
-        return false;
+        String extension = lowerCasedName.substring(lowerCasedName.lastIndexOf('.') + 1);
+        return configuration.getScriptExtensions().contains(extension);
     }
 }