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(); }