commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From s...@apache.org
Subject svn commit: r777530 - in /commons/proper/math/trunk/src: java/org/apache/commons/math/util/ test/org/apache/commons/math/stat/descriptive/
Date Fri, 22 May 2009 14:04:56 GMT
Author: sebb
Date: Fri May 22 14:04:56 2009
New Revision: 777530

URL: http://svn.apache.org/viewvc?rev=777530&view=rev
Log:
ReMove Serializable from interface
Implementations already implemented it

Modified:
    commons/proper/math/trunk/src/java/org/apache/commons/math/util/DefaultTransformer.java
    commons/proper/math/trunk/src/java/org/apache/commons/math/util/NumberTransformer.java
    commons/proper/math/trunk/src/java/org/apache/commons/math/util/TransformerMap.java
    commons/proper/math/trunk/src/test/org/apache/commons/math/stat/descriptive/MixedListUnivariateImplTest.java

Modified: commons/proper/math/trunk/src/java/org/apache/commons/math/util/DefaultTransformer.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/java/org/apache/commons/math/util/DefaultTransformer.java?rev=777530&r1=777529&r2=777530&view=diff
==============================================================================
--- commons/proper/math/trunk/src/java/org/apache/commons/math/util/DefaultTransformer.java
(original)
+++ commons/proper/math/trunk/src/java/org/apache/commons/math/util/DefaultTransformer.java
Fri May 22 14:04:56 2009
@@ -30,7 +30,9 @@
  * @version $Revision$ $Date$
  */
 public class DefaultTransformer implements NumberTransformer, Serializable {
-    
+    // TODO: Add Serializable documentation
+    // TODO: Check Serializable implementation
+   
     /** Serializable version identifier */
     private static final long serialVersionUID = 4019938025047800455L;
     

Modified: commons/proper/math/trunk/src/java/org/apache/commons/math/util/NumberTransformer.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/java/org/apache/commons/math/util/NumberTransformer.java?rev=777530&r1=777529&r2=777530&view=diff
==============================================================================
--- commons/proper/math/trunk/src/java/org/apache/commons/math/util/NumberTransformer.java
(original)
+++ commons/proper/math/trunk/src/java/org/apache/commons/math/util/NumberTransformer.java
Fri May 22 14:04:56 2009
@@ -16,15 +16,16 @@
  */
 package org.apache.commons.math.util;
 
-import java.io.Serializable;
-
 import org.apache.commons.math.MathException;
 
 /**
  * Subclasses implementing this interface can transform Objects to doubles.
  * @version $Revision$ $Date$
+ * 
+ * No longer extends Serializable since 2.0
+ * 
  */
-public interface NumberTransformer extends Serializable {
+public interface NumberTransformer {
     
     /**
      * Implementing this interface provides a facility to transform

Modified: commons/proper/math/trunk/src/java/org/apache/commons/math/util/TransformerMap.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/java/org/apache/commons/math/util/TransformerMap.java?rev=777530&r1=777529&r2=777530&view=diff
==============================================================================
--- commons/proper/math/trunk/src/java/org/apache/commons/math/util/TransformerMap.java (original)
+++ commons/proper/math/trunk/src/java/org/apache/commons/math/util/TransformerMap.java Fri
May 22 14:04:56 2009
@@ -32,6 +32,8 @@
  * @version $Revision$ $Date$
  */
 public class TransformerMap implements NumberTransformer, Serializable {
+    // TODO: Add Serializable documentation
+    // TODO: Check Serializable implementation
 
     /** Serializable version identifier */
     private static final long serialVersionUID = 4605318041528645258L;

Modified: commons/proper/math/trunk/src/test/org/apache/commons/math/stat/descriptive/MixedListUnivariateImplTest.java
URL: http://svn.apache.org/viewvc/commons/proper/math/trunk/src/test/org/apache/commons/math/stat/descriptive/MixedListUnivariateImplTest.java?rev=777530&r1=777529&r2=777530&view=diff
==============================================================================
--- commons/proper/math/trunk/src/test/org/apache/commons/math/stat/descriptive/MixedListUnivariateImplTest.java
(original)
+++ commons/proper/math/trunk/src/test/org/apache/commons/math/stat/descriptive/MixedListUnivariateImplTest.java
Fri May 22 14:04:56 2009
@@ -16,6 +16,7 @@
  */
 package org.apache.commons.math.stat.descriptive;
 
+import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.List;
 
@@ -184,7 +185,7 @@
         }
     }
 
-    public static final class FooTransformer implements NumberTransformer {
+    public static final class FooTransformer implements NumberTransformer, Serializable {
         private static final long serialVersionUID = -4252248129291326127L;
         public double transform(Object o) {
             return Double.parseDouble(((Foo) o).heresFoo());
@@ -197,7 +198,7 @@
         }
     }
 
-    public static final class BarTransformer implements NumberTransformer {
+    public static final class BarTransformer implements NumberTransformer, Serializable {
         private static final long serialVersionUID = -1768345377764262043L;
         public double transform(Object o) {
             return Double.parseDouble(((Bar) o).heresBar());



Mime
View raw message