You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2014/06/29 14:15:09 UTC
svn commit: r1606488 -
/commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/ControlFlowGraph.java
Author: ggregory
Date: Sun Jun 29 12:15:09 2014
New Revision: 1606488
URL: http://svn.apache.org/r1606488
Log:
Statement unnecessarily nested within else clause.
Modified:
commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/ControlFlowGraph.java
Modified: commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/ControlFlowGraph.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/ControlFlowGraph.java?rev=1606488&r1=1606487&r2=1606488&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/ControlFlowGraph.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/ControlFlowGraph.java Sun Jun 29 12:15:09 2014
@@ -242,11 +242,7 @@ public class ControlFlowGraph{
extendMessageWithFlow(sce);
throw sce;
}
- if (oldstack.equals(inF.getStack()) && oldlocals.equals(inF.getLocals())) {
- return false;
- } else {
- return true;
- }
+ return !(oldstack.equals(inF.getStack()) && oldlocals.equals(inF.getLocals()));
}
/**