commons-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ggreg...@apache.org
Subject svn commit: r1606484 - /commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass1Verifier.java
Date Sun, 29 Jun 2014 12:10:25 GMT
Author: ggregory
Date: Sun Jun 29 12:10:24 2014
New Revision: 1606484

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

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

Modified: commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass1Verifier.java
URL: http://svn.apache.org/viewvc/commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass1Verifier.java?rev=1606484&r1=1606483&r2=1606484&view=diff
==============================================================================
--- commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass1Verifier.java
(original)
+++ commons/proper/bcel/trunk/src/main/java/org/apache/bcel/verifier/statics/Pass1Verifier.java
Sun Jun 29 12:10:24 2014
@@ -162,11 +162,9 @@ public final class Pass1Verifier extends
         if (jc != null){
             return VerificationResult.VR_OK;
         }
-        else{
-            //TODO: Maybe change Repository's behaviour to throw a LoadingException instead
of just returning "null"
-            //      if a class file cannot be found or in another way be looked up.
-            return new VerificationResult(VerificationResult.VERIFIED_REJECTED, "Repository.lookup()
failed. FILE NOT FOUND?");
-        }
+        //TODO: Maybe change Repository's behaviour to throw a LoadingException instead of
just returning "null"
+        //      if a class file cannot be found or in another way be looked up.
+        return new VerificationResult(VerificationResult.VERIFIED_REJECTED, "Repository.lookup()
failed. FILE NOT FOUND?");
     }
 
     /**



Mime
View raw message