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/21 03:23:34 UTC

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

Author: sebb
Date: Fri Jan 21 02:23:34 2011
New Revision: 1061598

URL: http://svn.apache.org/viewvc?rev=1061598&view=rev
Log:
Only check 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=1061598&r1=1061597&r2=1061598&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 Fri Jan 21 02:23:34 2011
@@ -17,6 +17,7 @@
 package org.apache.commons.math.util;
 
 import java.lang.reflect.Method;
+import java.lang.reflect.Modifier;
 import java.lang.reflect.Type;
 
 import org.apache.commons.math.dfp.Dfp;
@@ -1027,7 +1028,7 @@ public class FastMathTest {
         Method math[] = StrictMath.class.getDeclaredMethods();
         for(Method mathMethod : math) {
             Type ret = mathMethod.getGenericReturnType();
-            if (ret.equals(type)){
+            if (ret.equals(type) && Modifier.isPublic(mathMethod.getModifiers())){
                 Type []params = mathMethod.getGenericParameterTypes();
                 if (params.length ==1 && params[0].equals(type)) {
                     Method fastMethod = null;