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:03:09 UTC
svn commit: r1606472 -
/commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/ControlFlowGraph.java
Author: ggregory
Date: Sun Jun 29 12:03:08 2014
New Revision: 1606472
URL: http://svn.apache.org/r1606472
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=1606472&r1=1606471&r2=1606472&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:03:08 2014
@@ -376,12 +376,10 @@ public class ControlFlowGraph{
System.arraycopy(matchTargets, 0, ret, 1, matchTargets.length);
return ret;
}
- else{
- final InstructionHandle[] pair = new InstructionHandle[2];
- pair[0] = getInstruction().getNext();
- pair[1] = ((BranchInstruction) inst).getTarget();
- return pair;
- }
+ final InstructionHandle[] pair = new InstructionHandle[2];
+ pair[0] = getInstruction().getNext();
+ pair[1] = ((BranchInstruction) inst).getTarget();
+ return pair;
}
// default case: Fall through.