You are viewing a plain text version of this content. The canonical link for it is here.
Posted to bcel-dev@jakarta.apache.org by db...@apache.org on 2006/08/30 23:57:25 UTC

svn commit: r438661 - in /jakarta/bcel/trunk/src/main/java/org/apache/bcel: classfile/Utility.java util/BCELFactory.java

Author: dbrosius
Date: Wed Aug 30 14:57:25 2006
New Revision: 438661

URL: http://svn.apache.org/viewvc?rev=438661&view=rev
Log:
make it clear that switch case fall thru was intended with added comments.

Modified:
    jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java
    jakarta/bcel/trunk/src/main/java/org/apache/bcel/util/BCELFactory.java

Modified: jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java
URL: http://svn.apache.org/viewvc/jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java?rev=438661&r1=438660&r2=438661&view=diff
==============================================================================
--- jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java (original)
+++ jakarta/bcel/trunk/src/main/java/org/apache/bcel/classfile/Utility.java Wed Aug 30 14:57:25 2006
@@ -328,6 +328,7 @@
             case Constants.NEW:
             case Constants.CHECKCAST:
                 buf.append("\t");
+                /* FALL THRU */
             case Constants.INSTANCEOF:
                 index = bytes.readUnsignedShort();
                 buf.append("\t<").append(

Modified: jakarta/bcel/trunk/src/main/java/org/apache/bcel/util/BCELFactory.java
URL: http://svn.apache.org/viewvc/jakarta/bcel/trunk/src/main/java/org/apache/bcel/util/BCELFactory.java?rev=438661&r1=438660&r2=438661&view=diff
==============================================================================
--- jakarta/bcel/trunk/src/main/java/org/apache/bcel/util/BCELFactory.java (original)
+++ jakarta/bcel/trunk/src/main/java/org/apache/bcel/util/BCELFactory.java Wed Aug 30 14:57:25 2006
@@ -181,6 +181,7 @@
                 break;
             case Constants.MULTIANEWARRAY:
                 dim = ((MULTIANEWARRAY) i).getDimensions();
+                /* FALL THRU */
             case Constants.ANEWARRAY:
             case Constants.NEWARRAY:
                 if (type instanceof ArrayType) {



---------------------------------------------------------------------
To unsubscribe, e-mail: bcel-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: bcel-dev-help@jakarta.apache.org