You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by eb...@apache.org on 2017/04/07 13:24:53 UTC
svn commit: r1790548 -
/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/Interpreter.java
Author: ebourg
Date: Fri Apr 7 13:24:52 2017
New Revision: 1790548
URL: http://svn.apache.org/viewvc?rev=1790548&view=rev
Log:
Removed unnecessary null checks (reported by Findbugs)
Modified:
commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/Interpreter.java
Modified: commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/Interpreter.java
URL: http://svn.apache.org/viewvc/commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/Interpreter.java?rev=1790548&r1=1790547&r2=1790548&view=diff
==============================================================================
--- commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/Interpreter.java (original)
+++ commons/proper/jexl/trunk/src/main/java/org/apache/commons/jexl3/internal/Interpreter.java Fri Apr 7 13:24:52 2017
@@ -775,24 +775,20 @@ public class Interpreter extends Interpr
protected Object visit(ASTArrayLiteral node, Object data) {
int childCount = node.jjtGetNumChildren();
JexlArithmetic.ArrayBuilder ab = arithmetic.arrayBuilder(childCount);
- if (ab != null) {
- boolean extended = false;
- for (int i = 0; i < childCount; i++) {
- if (isCancelled()) {
- throw new JexlException.Cancel(node);
- }
- JexlNode child = node.jjtGetChild(i);
- if (child instanceof ASTExtendedLiteral) {
- extended = true;
- } else {
- Object entry = node.jjtGetChild(i).jjtAccept(this, data);
- ab.add(entry);
- }
+ boolean extended = false;
+ for (int i = 0; i < childCount; i++) {
+ if (isCancelled()) {
+ throw new JexlException.Cancel(node);
+ }
+ JexlNode child = node.jjtGetChild(i);
+ if (child instanceof ASTExtendedLiteral) {
+ extended = true;
+ } else {
+ Object entry = node.jjtGetChild(i).jjtAccept(this, data);
+ ab.add(entry);
}
- return ab.create(extended);
- } else {
- return null;
}
+ return ab.create(extended);
}
@Override
@@ -804,36 +800,28 @@ public class Interpreter extends Interpr
protected Object visit(ASTSetLiteral node, Object data) {
int childCount = node.jjtGetNumChildren();
JexlArithmetic.SetBuilder mb = arithmetic.setBuilder(childCount);
- if (mb != null) {
- for (int i = 0; i < childCount; i++) {
- if (isCancelled()) {
- throw new JexlException.Cancel(node);
- }
- Object entry = node.jjtGetChild(i).jjtAccept(this, data);
- mb.add(entry);
+ for (int i = 0; i < childCount; i++) {
+ if (isCancelled()) {
+ throw new JexlException.Cancel(node);
}
- return mb.create();
- } else {
- return null;
+ Object entry = node.jjtGetChild(i).jjtAccept(this, data);
+ mb.add(entry);
}
+ return mb.create();
}
@Override
protected Object visit(ASTMapLiteral node, Object data) {
int childCount = node.jjtGetNumChildren();
JexlArithmetic.MapBuilder mb = arithmetic.mapBuilder(childCount);
- if (mb != null) {
- for (int i = 0; i < childCount; i++) {
- if (isCancelled()) {
- throw new JexlException.Cancel(node);
- }
- Object[] entry = (Object[]) (node.jjtGetChild(i)).jjtAccept(this, data);
- mb.put(entry[0], entry[1]);
+ for (int i = 0; i < childCount; i++) {
+ if (isCancelled()) {
+ throw new JexlException.Cancel(node);
}
- return mb.create();
- } else {
- return null;
+ Object[] entry = (Object[]) (node.jjtGetChild(i)).jjtAccept(this, data);
+ mb.put(entry[0], entry[1]);
}
+ return mb.create();
}
@Override