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

@@ -19,49 +19,53 @@
  * 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 represents a reference
  * to the source file of this class.  At most one SourceFile attribute
  * should appear per classfile.  The intention of this class is that it is
  * instantiated from the <em>Attribute.readAttribute()</em> method.
  *
- * @author  <A HREF="mailto:markus.dahm@berlin.de">M. Dahm</A>
+ * @version $Id: SourceFile.java 1749603 2016-06-21 20:50:19Z ggregory $
  * @see     Attribute
  */
 public final class SourceFile extends Attribute {
+
   private int sourcefile_index;
 
+
   /**
    * Initialize from another object. Note that both objects use the same
    * references (shallow copy). Use clone() for a physical copy.
    */
-  public SourceFile(SourceFile c) {
-    this(c.getNameIndex(), c.getLength(), c.getSourceFileIndex(),
-         c.getConstantPool());
+    public SourceFile(final SourceFile c) {
+        this(c.getNameIndex(), c.getLength(), c.getSourceFileIndex(), c.getConstantPool());
   }
 
+
   /**
-   * 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
    */
-  SourceFile(int name_index, int length, DataInputStream file,
-             ConstantPool constant_pool) throws IOException
-  {
-    this(name_index, length, file.readUnsignedShort(), constant_pool);
+    SourceFile(final int name_index, final int length, final DataInput input, final ConstantPool constant_pool)
+            throws IOException {
+        this(name_index, length, input.readUnsignedShort(), constant_pool);
   }
 
+
   /**
    * @param name_index Index in constant pool to CONSTANT_Utf8, which
    * should represent the string "SourceFile".
    * @param length Content length in bytes, the value should be 2.
    * @param constant_pool The constant pool that this attribute is

@@ -71,70 +75,80 @@
    * class was compiled.  It will not be interpreted as indicating the name
    * of the directory contqining the file or an absolute path; this
    * information has to be supplied the consumer of this attribute - in
    * many cases, the JVM.
    */
-  public SourceFile(int name_index, int length, int sourcefile_index,
-                    ConstantPool constant_pool)
-  {
-    super(Constants.ATTR_SOURCE_FILE, name_index, length, constant_pool);
+    public SourceFile(final int name_index, final int length, final int sourcefile_index, final ConstantPool constant_pool) {
+        super(Const.ATTR_SOURCE_FILE, name_index, length, constant_pool);
     this.sourcefile_index = sourcefile_index;
   }
 
+
   /**
    * 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.visitSourceFile(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(sourcefile_index);
   }
 
+
   /**
    * @return Index in constant pool of source file name.
    */
-  public final int getSourceFileIndex() { return sourcefile_index; }
+    public final int getSourceFileIndex() {
+        return sourcefile_index;
+    }
+
 
   /**
-   * @param sourcefile_index.
+     * @param sourcefile_index
    */
-  public final void setSourceFileIndex(int sourcefile_index) {
+    public final void setSourceFileIndex( final int sourcefile_index ) {
     this.sourcefile_index = sourcefile_index;
   }
 
+
   /**
    * @return Source file name.
    */
   public final String getSourceFileName() {
-    ConstantUtf8 c = (ConstantUtf8)constant_pool.getConstant(sourcefile_index,
-                                                             Constants.CONSTANT_Utf8);
+        final ConstantUtf8 c = (ConstantUtf8) super.getConstantPool().getConstant(sourcefile_index,
+                Const.CONSTANT_Utf8);
     return c.getBytes();
   }
 
+
   /**
    * @return String representation
    */
+    @Override
   public final String toString() {
-    return "SourceFile(" + getSourceFileName() + ")";
+        return "SourceFile: " + getSourceFileName();
   }
 
+
   /**
    * @return deep copy of this attribute
    */
-  public Attribute copy(ConstantPool constant_pool) {
-    return (SourceFile)clone();
+    @Override
+    public Attribute copy( final ConstantPool _constant_pool ) {
+        return (Attribute) clone();
   }
 }