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:08:43 UTC

svn commit: r1606479 - /commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/Subroutines.java

Author: ggregory
Date: Sun Jun 29 12:08:43 2014
New Revision: 1606479

URL: http://svn.apache.org/r1606479
Log:
Statement unnecessarily nested within else clause.

Modified:
    commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/Subroutines.java

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/Subroutines.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/Subroutines.java?rev=1606479&r1=1606478&r2=1606479&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/Subroutines.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/structurals/Subroutines.java Sun Jun 29 12:08:43 2014
@@ -328,9 +328,7 @@ public class Subroutines{
             if (localVariable != UNSET){
                 throw new AssertionViolatedException("localVariable set twice.");
             }
-            else{
-                localVariable = i;
-            }
+            localVariable = i;
         }
 
         /**