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 2008/12/06 19:01:44 UTC

svn commit: r724020 - /commons/proper/math/trunk/src/java/org/apache/commons/math/stat/descriptive/DescriptiveStatistics.java

Author: luc
Date: Sat Dec  6 10:01:43 2008
New Revision: 724020

URL: http://svn.apache.org/viewvc?rev=724020&view=rev
Log:
removed unneeded call to super()
removed spurious data copy (a copy was already done in the ResizableDoubleArray)

Modified:
    commons/proper/math/trunk/src/java/org/apache/commons/math/stat/descriptive/DescriptiveStatistics.java

Modified: commons/proper/math/trunk/src/java/org/apache/commons/math/stat/descriptive/DescriptiveStatistics.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/java/org/apache/commons/math/stat/descriptive/DescriptiveStatistics.java?rev=724020&r1=724019&r2=724020&view=diff
==============================================================================
--- commons/proper/math/trunk/src/java/org/apache/commons/math/stat/descriptive/DescriptiveStatistics.java (original)
+++ commons/proper/math/trunk/src/java/org/apache/commons/math/stat/descriptive/DescriptiveStatistics.java Sat Dec  6 10:01:43 2008
@@ -107,7 +107,6 @@
      * @param window the window size.
      */
     public DescriptiveStatistics(int window) {
-        super();
         setWindowSize(window);
     }
     
@@ -326,10 +325,7 @@
      *         were added to this set
      */
     public double[] getValues() {
-        double[] copiedArray = new double[eDA.getNumElements()];
-        System.arraycopy(eDA.getElements(), 0, copiedArray,
-            0, eDA.getNumElements());
-        return copiedArray;
+        return eDA.getElements();
     }
 
     /**