This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-bcel.git


The following commit(s) were added to refs/heads/master by this push:
     new c26d888  Remove useless returns.
c26d888 is described below

commit c26d888c2571fc6202c6a7d22d17ca3ca6ee624c
Author: Gary Gregory <[email protected]>
AuthorDate: Sat Nov 21 12:19:21 2020 -0500

    Remove useless returns.
---
 src/main/java/org/apache/bcel/classfile/Signature.java                 | 1 -
 .../org/apache/bcel/verifier/structurals/InstConstraintVisitor.java    | 3 ---
 2 files changed, 4 deletions(-)

diff --git a/src/main/java/org/apache/bcel/classfile/Signature.java 
b/src/main/java/org/apache/bcel/classfile/Signature.java
index 88cfdc5..ce3a09d 100644
--- a/src/main/java/org/apache/bcel/classfile/Signature.java
+++ b/src/main/java/org/apache/bcel/classfile/Signature.java
@@ -222,7 +222,6 @@ public final class Signature extends Attribute {
             matchGJIdent(in, buf);
         } else if (ch == ')') {
             in.unread();
-            return;
         } else if (ch != ';') {
             throw new IllegalArgumentException("Illegal signature: " + 
in.getData() + " read " + (char) ch);
         }
diff --git 
a/src/main/java/org/apache/bcel/verifier/structurals/InstConstraintVisitor.java 
b/src/main/java/org/apache/bcel/verifier/structurals/InstConstraintVisitor.java
index 35a71c9..ce816bd 100644
--- 
a/src/main/java/org/apache/bcel/verifier/structurals/InstConstraintVisitor.java
+++ 
b/src/main/java/org/apache/bcel/verifier/structurals/InstConstraintVisitor.java
@@ -992,9 +992,6 @@ public class InstConstraintVisitor extends EmptyVisitor{
                 constraintViolated(o, "If stack top's size is 2, then stack 
next-to-top's size must be 1. But it is '"+
                     stack().peek(1)+"' of size 
'"+stack().peek(1).getSize()+"'.");
             }
-            else{
-                return; // Form 2
-            }
         }
         else{ // stack top is of size 1
             if ( stack().peek(1).getSize() != 1 ) {

Reply via email to