You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by lu...@apache.org on 2011/05/10 20:19:36 UTC
svn commit: r1101579 - in /commons/proper/math/trunk/src: main/java/org/apache/commons/math/linear/ArrayFieldVector.java site/xdoc/changes.xml

Author: luc
Date: Tue May 10 18:19:36 2011
New Revision: 1101579

URL: http://svn.apache.org/viewvc?rev=1101579&view=rev
Log:
Avoid some array copying in add and subtract ArrayFieldVector.
JIRA: MATH-573

Modified:
    commons/proper/math/trunk/src/main/java/org/apache/commons/math/linear/ArrayFieldVector.java
    commons/proper/math/trunk/src/site/xdoc/changes.xml

Modified: commons/proper/math/trunk/src/main/java/org/apache/commons/math/linear/ArrayFieldVector.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/linear/ArrayFieldVector.java?rev=1101579&r1=1101578&r2=1101579&view=diff
==============================================================================
--- commons/proper/math/trunk/src/main/java/org/apache/commons/math/linear/ArrayFieldVector.java (original)
+++ commons/proper/math/trunk/src/main/java/org/apache/commons/math/linear/ArrayFieldVector.java Tue May 10 18:19:36 2011
@@ -383,7 +383,7 @@ public class ArrayFieldVector<T extends 
             for (int i = 0; i < data.length; i++) {
                 out[i] = data[i].add(v.getEntry(i));
             }
-            return new ArrayFieldVector<T>(out);
+            return new ArrayFieldVector<T>(field, out, false);
         }
     }
 
@@ -394,7 +394,7 @@ public class ArrayFieldVector<T extends 
         for (int i = 0; i < data.length; i++) {
             out[i] = data[i].add(v[i]);
         }
-        return new ArrayFieldVector<T>(out);
+        return new ArrayFieldVector<T>(field, out, false);
     }
 
     /**
@@ -417,7 +417,7 @@ public class ArrayFieldVector<T extends 
             for (int i = 0; i < data.length; i++) {
                 out[i] = data[i].subtract(v.getEntry(i));
             }
-            return new ArrayFieldVector<T>(out);
+            return new ArrayFieldVector<T>(field, out, false);
         }
     }
 
@@ -428,7 +428,7 @@ public class ArrayFieldVector<T extends 
         for (int i = 0; i < data.length; i++) {
             out[i] = data[i].subtract(v[i]);
         }
-        return new ArrayFieldVector<T>(out);
+        return new ArrayFieldVector<T>(field, out, false);
     }
 
     /**

Modified: commons/proper/math/trunk/src/site/xdoc/changes.xml
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/site/xdoc/changes.xml?rev=1101579&r1=1101578&r2=1101579&view=diff
==============================================================================
--- commons/proper/math/trunk/src/site/xdoc/changes.xml (original)
+++ commons/proper/math/trunk/src/site/xdoc/changes.xml Tue May 10 18:19:36 2011
@@ -52,6 +52,9 @@ The <action> type attribute can be add,u
     If the output is not quite correct, check for invisible trailing spaces!
      -->
     <release version="3.0" date="TBD" description="TBD">
+      <action dev="luc" type="fix" issue="MATH-573">
+        Avoid some array copying in add and subtract ArrayFieldVector.
+      </action>
       <action dev="luc" type="fix" issue="MATH-572">
         Fixed an unused constructor parameter in ArrayFieldVector.
       </action>