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 2009/08/09 13:09:16 UTC
svn commit: r802516 -
/commons/sandbox/nabla/trunk/src/test/java/org/apache/commons/nabla/AbstractStaticFunctionsTest.java
Author: luc
Date: Sun Aug 9 11:09:16 2009
New Revision: 802516
URL: http://svn.apache.org/viewvc?rev=802516&view=rev
Log:
fixed findbugs warnings
Modified:
commons/sandbox/nabla/trunk/src/test/java/org/apache/commons/nabla/AbstractStaticFunctionsTest.java
Modified: commons/sandbox/nabla/trunk/src/test/java/org/apache/commons/nabla/AbstractStaticFunctionsTest.java
URL: http://svn.apache.org/viewvc/commons/sandbox/nabla/trunk/src/test/java/org/apache/commons/nabla/AbstractStaticFunctionsTest.java?rev=802516&r1=802515&r2=802516&view=diff
==============================================================================
--- commons/sandbox/nabla/trunk/src/test/java/org/apache/commons/nabla/AbstractStaticFunctionsTest.java (original)
+++ commons/sandbox/nabla/trunk/src/test/java/org/apache/commons/nabla/AbstractStaticFunctionsTest.java Sun Aug 9 11:09:16 2009
@@ -211,7 +211,11 @@
}
}
return ((Double) javaMethod.invoke(null, changed)).doubleValue();
- } catch (Exception e) {
+ } catch (IllegalArgumentException e) {
+ return Double.NaN;
+ } catch (IllegalAccessException e) {
+ return Double.NaN;
+ } catch (InvocationTargetException e) {
return Double.NaN;
}
}
@@ -241,9 +245,9 @@
} else if (types[i].equals(Double.TYPE)) {
arguments[i] = new Double(values[i]);
} else if (types[i].equals(Integer.TYPE)) {
- arguments[i] = new Integer((int) values[i]);
+ arguments[i] = Integer.valueOf((int) values[i]);
} else if (types[i].equals(Long.TYPE)) {
- arguments[i] = new Long((long) values[i]);
+ arguments[i] = Long.valueOf((long) values[i]);
}
}
return arguments;