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

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-io.git


The following commit(s) were added to refs/heads/master by this push:
     new 7045200  No need to allocate empty Comparator arrays over and over.
7045200 is described below

commit 7045200f2a607371649990c7290a939627b88372
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Wed Dec 25 11:43:34 2019 -0500

    No need to allocate empty Comparator arrays over and over.
---
 .../java/org/apache/commons/io/comparator/CompositeFileComparator.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git 
a/src/main/java/org/apache/commons/io/comparator/CompositeFileComparator.java 
b/src/main/java/org/apache/commons/io/comparator/CompositeFileComparator.java
index 18d3850..fc48db0 100644
--- 
a/src/main/java/org/apache/commons/io/comparator/CompositeFileComparator.java
+++ 
b/src/main/java/org/apache/commons/io/comparator/CompositeFileComparator.java
@@ -43,6 +43,7 @@ import java.util.List;
  */
 public class CompositeFileComparator extends AbstractFileComparator implements 
Serializable {
 
+    private static final Comparator<?>[] EMPTY_COMPARATOR_ARRAY = new 
Comparator<?>[0];
     private static final long serialVersionUID = -2224170307287243428L;
     private static final Comparator<?>[] NO_COMPARATORS = {};
     private final Comparator<File>[] delegates;
@@ -76,7 +77,7 @@ public class CompositeFileComparator extends 
AbstractFileComparator implements S
             for (final Comparator<File> comparator : delegates) {
                 list.add(comparator);
             }
-            this.delegates = (Comparator<File>[]) list.toArray(new 
Comparator<?>[0]); //2
+            this.delegates = (Comparator<File>[]) 
list.toArray(EMPTY_COMPARATOR_ARRAY); //2
         }
     }
 

Reply via email to