< prev index next >

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

Print this page

        

@@ -19,130 +19,143 @@
  * 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 is derived from <em>Attribute</em> and denotes that this class
  * is an Inner class of another.
  * to the source file of this class.
  * It is instantiated from the <em>Attribute.readAttribute()</em> method.
  *
- * @author  <A HREF="mailto:markus.dahm@berlin.de">M. Dahm</A>
+ * @version $Id: InnerClasses.java 1749603 2016-06-21 20:50:19Z ggregory $
  * @see     Attribute
  */
 public final class InnerClasses extends Attribute {
+
   private InnerClass[] inner_classes;
-  private int          number_of_classes;
+
 
   /**
    * Initialize from another object. Note that both objects use the same
    * references (shallow copy). Use clone() for a physical copy.
    */
-  public InnerClasses(InnerClasses c) {
-    this(c.getNameIndex(), c.getLength(), c.getInnerClasses(),
-         c.getConstantPool());
+    public InnerClasses(final InnerClasses c) {
+        this(c.getNameIndex(), c.getLength(), c.getInnerClasses(), c.getConstantPool());
   }
 
+
   /**
    * @param name_index Index in constant pool to CONSTANT_Utf8
    * @param length Content length in bytes
    * @param inner_classes array of inner classes attributes
    * @param constant_pool Array of constants
-   * @param sourcefile_index Index in constant pool to CONSTANT_Utf8
    */
-  public InnerClasses(int name_index, int length,
-                      InnerClass[] inner_classes,
-                      ConstantPool constant_pool)
-  {
-    super(Constants.ATTR_INNER_CLASSES, name_index, length, constant_pool);
-    setInnerClasses(inner_classes);
+    public InnerClasses(final int name_index, final int length, final InnerClass[] inner_classes,
+            final ConstantPool constant_pool) {
+        super(Const.ATTR_INNER_CLASSES, name_index, length, constant_pool);
+        this.inner_classes = inner_classes != null ? inner_classes : new InnerClass[0];
   }
 
+
   /**
-   * Construct object from file stream.
+     * Construct object from input stream.
    *
    * @param name_index Index in constant pool to CONSTANT_Utf8
    * @param length Content length in bytes
-   * @param file Input stream
+     * @param input Input stream
    * @param constant_pool Array of constants
    * @throws IOException
    */
-  InnerClasses(int name_index, int length, DataInputStream file,
-               ConstantPool constant_pool) throws IOException
-  {
-    this(name_index, length, (InnerClass[])null, constant_pool);
-
-    number_of_classes = file.readUnsignedShort();
+    InnerClasses(final int name_index, final int length, final DataInput input, final ConstantPool constant_pool)
+            throws IOException {
+        this(name_index, length, (InnerClass[]) null, constant_pool);
+        final int number_of_classes = input.readUnsignedShort();
     inner_classes = new InnerClass[number_of_classes];
-
-    for(int i=0; i < number_of_classes; i++)
-      inner_classes[i] = new InnerClass(file);
+        for (int i = 0; i < number_of_classes; i++) {
+            inner_classes[i] = new InnerClass(input);
   }
+    }
+
+
   /**
    * 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.visitInnerClasses(this);
   }
+
+
   /**
    * Dump source file 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(number_of_classes);
-
-    for(int i=0; i < number_of_classes; i++)
-      inner_classes[i].dump(file);
+        file.writeShort(inner_classes.length);
+        for (final InnerClass inner_class : inner_classes) {
+            inner_class.dump(file);
   }
+    }
+
 
   /**
    * @return array of inner class "records"
    */
-  public final InnerClass[] getInnerClasses() { return inner_classes; }
+    public final InnerClass[] getInnerClasses() {
+        return inner_classes;
+    }
+
 
   /**
-   * @param inner_classes.
+     * @param inner_classes the array of inner classes
    */
-  public final void setInnerClasses(InnerClass[] inner_classes) {
-    this.inner_classes = inner_classes;
-    number_of_classes = (inner_classes == null)? 0 : inner_classes.length;
+    public final void setInnerClasses( final InnerClass[] inner_classes ) {
+        this.inner_classes = inner_classes != null ? inner_classes : new InnerClass[0];
   }
 
+
   /**
    * @return String representation.
    */
+    @Override
   public final String toString() {
-    StringBuffer buf = new StringBuffer();
-
-    for(int i=0; i < number_of_classes; i++)
-      buf.append(inner_classes[i].toString(constant_pool) + "\n");
-
+        final StringBuilder buf = new StringBuilder();
+        buf.append("InnerClasses(");
+        buf.append(inner_classes.length);
+        buf.append("):\n");
+        for (final InnerClass inner_class : inner_classes) {
+            buf.append(inner_class.toString(super.getConstantPool())).append("\n");
+        }
     return buf.toString();
   }
 
+
   /**
    * @return deep copy of this attribute
    */
-  public Attribute copy(ConstantPool constant_pool) {
-    InnerClasses c = (InnerClasses)clone();
-
-    c.inner_classes = new InnerClass[number_of_classes];
-    for(int i=0; i < number_of_classes; i++)
+    @Override
+    public Attribute copy( final ConstantPool _constant_pool ) {
+        // TODO this could be recoded to use a lower level constructor after creating a copy of the inner classes
+        final InnerClasses c = (InnerClasses) clone();
+        c.inner_classes = new InnerClass[inner_classes.length];
+        for (int i = 0; i < inner_classes.length; i++) {
       c.inner_classes[i] = inner_classes[i].copy();
-
-    c.constant_pool = constant_pool;
+        }
+        c.setConstantPool(_constant_pool);
     return c;
   }
 }
< prev index next >