You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by se...@apache.org on 2011/01/23 01:53:08 UTC

svn commit: r1062302 - /commons/proper/math/branches/MATH_2_X/src/test/java/org/apache/commons/math/util/FastMathTest.java

Author: sebb
Date: Sun Jan 23 00:53:08 2011
New Revision: 1062302

URL: http://svn.apache.org/viewvc?rev=1062302&view=rev
Log:
Should only test against public methods.

Modified:
    commons/proper/math/branches/MATH_2_X/src/test/java/org/apache/commons/math/util/FastMathTest.java

Modified: commons/proper/math/branches/MATH_2_X/src/test/java/org/apache/commons/math/util/FastMathTest.java
URL: http://svn.apache.org/viewvc/commons/proper/math/branches/MATH_2_X/src/test/java/org/apache/commons/math/util/FastMathTest.java?rev=1062302&r1=1062301&r2=1062302&view=diff
==============================================================================
--- commons/proper/math/branches/MATH_2_X/src/test/java/org/apache/commons/math/util/FastMathTest.java (original)
+++ commons/proper/math/branches/MATH_2_X/src/test/java/org/apache/commons/math/util/FastMathTest.java Sun Jan 23 00:53:08 2011
@@ -1071,7 +1071,11 @@ public class FastMathTest {
                     String name = mathMethod.getName();
                     try {
                         fastMethod = FastMath.class.getDeclaredMethod(name, (Class[]) params);
-                        comparer.compareSpecials(mathMethod, fastMethod, params);
+                        if (Modifier.isPublic(fastMethod.getModifiers())) {
+                            comparer.compareSpecials(mathMethod, fastMethod, params);
+                        } else {
+                            System.out.println("Cannot find public FastMath method corresponding to: "+mathMethod);                            
+                        }
                     } catch (NoSuchMethodException e) {
                         System.out.println("Cannot find FastMath method corresponding to: "+mathMethod);
                     }