You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@commons.apache.org by se...@apache.org on 2009/06/18 00:41:15 UTC

svn commit: r785848 - /commons/proper/collections/trunk/src/test/org/apache/commons/collections/comparators/TestFixedOrderComparator.java

Author: sebb
Date: Wed Jun 17 22:41:14 2009
New Revision: 785848

URL: http://svn.apache.org/viewvc?rev=785848&view=rev
Log:
No reason for the array to be public (and therefore externally mutable)

Modified:
    commons/proper/collections/trunk/src/test/org/apache/commons/collections/comparators/TestFixedOrderComparator.java

Modified: commons/proper/collections/trunk/src/test/org/apache/commons/collections/comparators/TestFixedOrderComparator.java
URL: http://svn.apache.org/viewvc/commons/proper/collections/trunk/src/test/org/apache/commons/collections/comparators/TestFixedOrderComparator.java?rev=785848&r1=785847&r2=785848&view=diff
==============================================================================
--- commons/proper/collections/trunk/src/test/org/apache/commons/collections/comparators/TestFixedOrderComparator.java (original)
+++ commons/proper/collections/trunk/src/test/org/apache/commons/collections/comparators/TestFixedOrderComparator.java Wed Jun 17 22:41:14 2009
@@ -40,7 +40,7 @@
     /**
      * Top cities of the world, by population including metro areas.
      */
-    public static final String topCities[] = new String[] {
+    private static final String topCities[] = new String[] {
         "Tokyo",
         "Mexico City",
         "Mumbai",