commit 3cb117f6a7a270873d3c4d1a26dff5bc47c038ce
author Saša Živkov <sasa.zivkov@sap.com> Tue Apr 14 15:31:16 2015 +0200
committer Saša Živkov <sasa.zivkov@sap.com> Tue Apr 14 15:31:16 2015 +0200
tree e7ca14a959f718b4e8e6358dc736179decb01a34
parent baa2f2e4b24a972e2a0e6d416111347f7c637812

Avoid toLowerCase on ListImportedProjects.match multiple times

Change-Id: I506275f3720eedf93d12016dc0ec88e48efb1f27

src/main/java/com/googlesource/gerrit/plugins/importer/ListImportedProjects.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/importer/ListImportedProjects.java b/src/main/java/com/googlesource/gerrit/plugins/importer/ListImportedProjects.java
index d37bf9f..1eb265d 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/importer/ListImportedProjects.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/importer/ListImportedProjects.java
@@ -41,7 +41,7 @@
   @Option(name = "--match", metaVar = "MATCH",
       usage = "List only projects containing this substring, case insensitive")
   public void setMatch(String match) {
-    this.match = match.toLowerCase();
+    this.match = match.toLowerCase(Locale.ENGLISH);
   }
 
   private String match;
@@ -62,12 +62,11 @@
   }
 
   private Collection<File> listImportFiles() {
-    match = Strings.nullToEmpty(match).toLowerCase(Locale.ENGLISH);
     Collection<File> importFiles = new HashSet<>();
     for (File f : Files.fileTreeTraverser().preOrderTraversal(projects.FS_LAYOUT.getLockRoot())) {
       if (f.isFile()
           && !f.getName().endsWith(".lock")
-          && f.getName().toLowerCase(Locale.ENGLISH).contains(match)) {
+          && (match == null || f.getName().toLowerCase(Locale.ENGLISH).contains(match))) {
         importFiles.add(f);
       }
     }