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:12:03 UTC

svn commit: r1606486 - /commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java

Author: ggregory
Date: Sun Jun 29 12:12:03 2014
New Revision: 1606486

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

Modified:
    commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java?rev=1606486&r1=1606485&r2=1606486&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java (original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass3aVerifier.java Sun Jun 29 12:12:03 2014
@@ -207,9 +207,8 @@ public final class Pass3aVerifier extend
             }
             return vr;
         }
-        else{ //did not pass Pass 2.
-            return VerificationResult.VR_NOTYET;
-        }
+        //did not pass Pass 2.
+        return VerificationResult.VR_NOTYET;
         } catch (ClassNotFoundException e) {
         // FIXME: maybe not the best way to handle this
         throw new AssertionViolatedException("Missing class: " + e, e);