You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by ry...@apache.org on 2008/07/19 06:21:22 UTC

svn commit: r678096 - /lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java

Author: ryan
Date: Fri Jul 18 21:21:20 2008
New Revision: 678096

URL: http://svn.apache.org/viewvc?rev=678096&view=rev
Log:
SOLR-619 -- switch to System.arraycopy() for array copy

Modified:
    lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java

Modified: lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java?rev=678096&r1=678095&r2=678096&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java Fri Jul 18 21:21:20 2008
@@ -689,14 +689,10 @@
       dynamicCopyFields = new DynamicCopy[] {dcopy};
     }
     else {
-      int i=0;
-      DynamicCopy[] old = dynamicCopyFields;
-      dynamicCopyFields = new DynamicCopy[dynamicCopyFields.length+1];
-      for( DynamicCopy dc : old ) {
-        dynamicCopyFields[i++] = dc;
-      }
-      dynamicCopyFields[i++] = dcopy;
-      old = null;
+      DynamicCopy[] temp = new DynamicCopy[dynamicCopyFields.length+1];
+      System.arraycopy(dynamicCopyFields,0,temp,0,dynamicCopyFields.length);
+      temp[temp.length -1] = dcopy;
+      dynamicCopyFields = temp;
     }
     log.finest("Dynamic Copy Field:" + dcopy );
   }