You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by he...@apache.org on 2016/02/23 21:02:47 UTC
svn commit: r1731936 -
/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/NumberParser.java
Author: henrib
Date: Tue Feb 23 20:02:47 2016
New Revision: 1731936
URL: http://svn.apache.org/viewvc?rev=1731936&view=rev
Log:
JEXL:
FindBugs warning
Modified:
commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/NumberParser.java
Modified: commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/NumberParser.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/NumberParser.java?rev=1731936&r1=1731935&r2=1731936&view=diff
==============================================================================
--- commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/NumberParser.java (original)
+++ commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/parser/NumberParser.java Tue Feb 23 20:02:47 2016
@@ -36,21 +36,19 @@ public final class NumberParser {
return "NaN";
}
if (BigDecimal.class.equals(clazz)) {
- return BIGDF.format(literal);
+ return BIGDF.format(literal);
}
StringBuilder strb = new StringBuilder(literal.toString());
- if (clazz != null) {
- if (Float.class.equals(clazz)) {
- strb.append('f');
- } else if (Double.class.equals(clazz)) {
- strb.append('d');
- } else if (BigDecimal.class.equals(clazz)) {
- strb.append('b');
- } else if (BigInteger.class.equals(clazz)) {
- strb.append('h');
- } else if (Long.class.equals(clazz)) {
- strb.append('l');
- }
+ if (Float.class.equals(clazz)) {
+ strb.append('f');
+ } else if (Double.class.equals(clazz)) {
+ strb.append('d');
+ } else if (BigDecimal.class.equals(clazz)) {
+ strb.append('b');
+ } else if (BigInteger.class.equals(clazz)) {
+ strb.append('h');
+ } else if (Long.class.equals(clazz)) {
+ strb.append('l');
}
return strb.toString();
}