src/java.xml/share/classes/com/sun/org/apache/bcel/internal/classfile/StackMap.java

@@ -19,134 +19,147 @@
  * limitations under the License.
  */
 
 package com.sun.org.apache.bcel.internal.classfile;
 
+import java.io.DataInput;
+import java.io.DataOutputStream;
+import java.io.IOException;
 
-import  com.sun.org.apache.bcel.internal.Constants;
-import  java.io.*;
+import com.sun.org.apache.bcel.internal.Const;
 
 /**
  * This class represents a stack map attribute used for
  * preverification of Java classes for the <a
  * href="http://java.sun.com/j2me/"> Java 2 Micro Edition</a>
  * (J2ME). This attribute is used by the <a
  * href="http://java.sun.com/products/cldc/">KVM</a> and contained
  * within the Code attribute of a method. See CLDC specification
  * 5.3.1.2
  *
- * @author  <A HREF="mailto:markus.dahm@berlin.de">M. Dahm</A>
+ * @version $Id: StackMap.java 1749603 2016-06-21 20:50:19Z ggregory $
  * @see     Code
  * @see     StackMapEntry
  * @see     StackMapType
  */
-public final class StackMap extends Attribute implements Node {
-  private int             map_length;
+public final class StackMap extends Attribute {
+
   private StackMapEntry[] map; // Table of stack map entries
 
+
   /*
    * @param name_index Index of name
    * @param length Content length in bytes
    * @param map Table of stack map entries
    * @param constant_pool Array of constants
    */
-  public StackMap(int name_index, int length,  StackMapEntry[] map,
-                  ConstantPool constant_pool)
-  {
-    super(Constants.ATTR_STACK_MAP, name_index, length, constant_pool);
-
-    setStackMap(map);
+    public StackMap(final int name_index, final int length, final StackMapEntry[] map, final ConstantPool constant_pool) {
+        super(Const.ATTR_STACK_MAP, name_index, length, constant_pool);
+        this.map = map;
   }
 
+
   /**
-   * Construct object from file stream.
+     * Construct object from input stream.
+     *
    * @param name_index Index of name
    * @param length Content length in bytes
-   * @param file Input stream
-   * @throws IOException
+     * @param input Input stream
    * @param constant_pool Array of constants
+     * @throws IOException
    */
-  StackMap(int name_index, int length, DataInputStream file,
-           ConstantPool constant_pool) throws IOException
-  {
-    this(name_index, length, (StackMapEntry[])null, constant_pool);
-
-    map_length = file.readUnsignedShort();
+    StackMap(final int name_index, final int length, final DataInput input, final ConstantPool constant_pool) throws IOException {
+        this(name_index, length, (StackMapEntry[]) null, constant_pool);
+        final int map_length = input.readUnsignedShort();
     map = new StackMapEntry[map_length];
-
-    for(int i=0; i < map_length; i++)
-      map[i] = new StackMapEntry(file, constant_pool);
+        for (int i = 0; i < map_length; i++) {
+            map[i] = new StackMapEntry(input, constant_pool);
   }
+    }
+
 
   /**
    * Dump line number table attribute to file stream in binary format.
    *
    * @param file Output file stream
    * @throws IOException
    */
-  public final void dump(DataOutputStream file) throws IOException
-  {
+    @Override
+    public final void dump( final DataOutputStream file ) throws IOException {
     super.dump(file);
-    file.writeShort(map_length);
-    for(int i=0; i < map_length; i++)
-      map[i].dump(file);
+        file.writeShort(map.length);
+        for (final StackMapEntry entry : map) {
+            entry.dump(file);
   }
+    }
+
 
   /**
    * @return Array of stack map entries
    */
-  public final StackMapEntry[] getStackMap() { return map; }
+    public final StackMapEntry[] getStackMap() {
+        return map;
+    }
+
 
   /**
    * @param map Array of stack map entries
    */
-  public final void setStackMap(StackMapEntry[] map) {
+    public final void setStackMap( final StackMapEntry[] map ) {
     this.map = map;
-
-    map_length = (map == null)? 0 : map.length;
+        int len = 2; // Length of 'number_of_entries' field prior to the array of stack maps
+        for (final StackMapEntry element : map) {
+            len += element.getMapEntrySize();
+        }
+        setLength(len);
   }
 
+
   /**
    * @return String representation.
    */
+    @Override
   public final String toString() {
-    StringBuffer buf = new StringBuffer("StackMap(");
-
-    for(int i=0; i < map_length; i++) {
-      buf.append(map[i].toString());
-
-      if(i < map_length - 1)
+        final StringBuilder buf = new StringBuilder("StackMap(");
+        for (int i = 0; i < map.length; i++) {
+            buf.append(map[i]);
+            if (i < map.length - 1) {
         buf.append(", ");
     }
-
+        }
     buf.append(')');
-
     return buf.toString();
   }
 
+
   /**
    * @return deep copy of this attribute
    */
-  public Attribute copy(ConstantPool constant_pool) {
-    StackMap c = (StackMap)clone();
-
-    c.map = new StackMapEntry[map_length];
-    for(int i=0; i < map_length; i++)
+    @Override
+    public Attribute copy( final ConstantPool _constant_pool ) {
+        final StackMap c = (StackMap) clone();
+        c.map = new StackMapEntry[map.length];
+        for (int i = 0; i < map.length; i++) {
       c.map[i] = map[i].copy();
-
-    c.constant_pool = constant_pool;
+        }
+        c.setConstantPool(_constant_pool);
     return c;
   }
 
+
   /**
    * Called by objects that are traversing the nodes of the tree implicitely
    * defined by the contents of a Java class. I.e., the hierarchy of methods,
    * fields, attributes, etc. spawns a tree of objects.
    *
    * @param v Visitor object
    */
-   public void accept(Visitor v) {
+    @Override
+    public void accept( final Visitor v ) {
      v.visitStackMap(this);
    }
 
-  public final int getMapLength() { return map_length; }
+
+    public final int getMapLength() {
+        return map == null ? 0 : map.length;
+    }
 }