hmm are you concerned about the extra Math.min that happens in the copyOf method? I don't how that relates to "intrinsic" and java 1.7
I don't have strong feelings here just checking if you mix something up in the comment you put there... I am happy to keep the old and now current code simon On Thu, Jun 30, 2011 at 2:42 PM, <rm...@apache.org> wrote: > Author: rmuir > Date: Thu Jun 30 12:42:17 2011 > New Revision: 1141510 > > URL: http://svn.apache.org/viewvc?rev=1141510&view=rev > Log: > LUCENE-3239: remove use of slow Arrays.copyOf > > Modified: > > lucene/dev/trunk/modules/facet/src/java/org/apache/lucene/util/UnsafeByteArrayOutputStream.java > > Modified: > lucene/dev/trunk/modules/facet/src/java/org/apache/lucene/util/UnsafeByteArrayOutputStream.java > URL: > http://svn.apache.org/viewvc/lucene/dev/trunk/modules/facet/src/java/org/apache/lucene/util/UnsafeByteArrayOutputStream.java?rev=1141510&r1=1141509&r2=1141510&view=diff > ============================================================================== > --- > lucene/dev/trunk/modules/facet/src/java/org/apache/lucene/util/UnsafeByteArrayOutputStream.java > (original) > +++ > lucene/dev/trunk/modules/facet/src/java/org/apache/lucene/util/UnsafeByteArrayOutputStream.java > Thu Jun 30 12:42:17 2011 > @@ -2,7 +2,6 @@ package org.apache.lucene.util; > > import java.io.IOException; > import java.io.OutputStream; > -import java.util.Arrays; > > /** > * Licensed to the Apache Software Foundation (ASF) under one or more > @@ -72,7 +71,11 @@ public class UnsafeByteArrayOutputStream > } > > private void grow(int newLength) { > - buffer = Arrays.copyOf(buffer, newLength); > + // It actually should be: (Java 1.7, when its intrinsic on all machines) > + // buffer = Arrays.copyOf(buffer, newLength); > + byte[] newBuffer = new byte[newLength]; > + System.arraycopy(buffer, 0, newBuffer, 0, buffer.length); > + buffer = newBuffer; > } > > /** > > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org
