svn commit: r808311 - in /commons/proper/math/trunk/src: main/java/org/apache/commons/math/optimization/linear/SimplexTableau.java site/xdoc/changes.xml

[luc]

Author: luc
Date: Thu Aug 27 07:57:51 2009
New Revision: 808311

URL: http://svn.apache.org/viewvc?rev=808311&view=rev
Log:
removed an unused argument in a private method
JIRA: MATH-289

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

Modified: 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/optimization/linear/SimplexTableau.java
URL: 
http://svn.apache.org/viewvc/commons/proper/math/trunk/src/main/java/org/apache/commons/math/optimization/linear/SimplexTableau.java?rev=808311&r1=808310&r2=808311&view=diff
==============================================================================
--- 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/optimization/linear/SimplexTableau.java
 (original)
+++ 
commons/proper/math/trunk/src/main/java/org/apache/commons/math/optimization/linear/SimplexTableau.java
 Thu Aug 27 07:57:51 2009
@@ -136,7 +136,7 @@
         matrix[zIndex][zIndex] = maximize ? 1 : -1;
         RealVector objectiveCoefficients =
             maximize ? f.getCoefficients().mapMultiply(-1) : 
f.getCoefficients();
-        copyArray(objectiveCoefficients.getData(), matrix[zIndex], 
getNumObjectiveFunctions());
+        copyArray(objectiveCoefficients.getData(), matrix[zIndex]);
         matrix[zIndex][width - 1] =
             maximize ? f.getConstantTerm() : -1 * f.getConstantTerm();
 
@@ -153,7 +153,7 @@
             int row = getNumObjectiveFunctions() + i;
 
             // decision variable coefficients
-            copyArray(constraint.getCoefficients().getData(), matrix[row], 1);
+            copyArray(constraint.getCoefficients().getData(), matrix[row]);
 
             // x-
             if (!restrictToNonNegative) {
@@ -325,10 +325,8 @@
     /**
      * @param src the source array
      * @param dest the destination array
-     * @param destPos the destination position
      */
-    private void copyArray(final double[] src, final double[] dest,
-                           final int destPos) {
+    private void copyArray(final double[] src, final double[] dest) {
         System.arraycopy(src, 0, dest, getNumObjectiveFunctions(), 
src.length); 
     }
 

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=808311&r1=808310&r2=808311&view=diff
==============================================================================
--- commons/proper/math/trunk/src/site/xdoc/changes.xml (original)
+++ commons/proper/math/trunk/src/site/xdoc/changes.xml Thu Aug 27 07:57:51 2009
@@ -43,6 +43,9 @@
         Fixed a NullPointerException when no solution is possible and some 
constraints
         are negative
       </action>
+      <action dev="luc" type="fix" issue="MATH-289" >
+        Removed an unused argument in a private method
+      </action>
       <action dev="luc" type="fix" issue="MATH-288" due-to="Benjamin McCann">
         Fixed an error induced by entries set to 0
       </action>