Yet more javadoc fixes

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@551914 13f79535-47bb-0310-9956-ffa450edef68

src/java/org/apache/poi/hssf/record/CustomField.java

@@ -28,10 +28,7 @@ public interface CustomField
 
     /**
      * Populates this fields data from the byte array passed in.
-     * @param   data raw data
+     * @param in the RecordInputstream to read the record from
-     * @param   size size of data
-     * @param   offset of the record's data (provided a big array of the file)
-     * @return  the number of bytes read.
      */
     int fillField(RecordInputStream in);
 

src/java/org/apache/poi/hssf/record/ExternSheetRecord.java

@@ -65,9 +65,7 @@ public class ExternSheetRecord extends Record {
      * called by the constructor, should set class level fields.  Should throw
      * runtime exception for bad/icomplete data.
      *
-     * @param data raw data
+     * @param in the RecordInputstream to read the record from
-     * @param size size of data
-     * @param offset of the record's data (provided a big array of the file)
      */
     protected void fillFields(RecordInputStream in) {
         field_2_REF_structures           = new ArrayList();

src/java/org/apache/poi/hssf/record/MulBlankRecord.java

@@ -118,9 +118,7 @@ public class MulBlankRecord
 
     /**
      * @param in the RecordInputstream to read the record from
-     * @param size size of data
      */
-
     protected void fillFields(RecordInputStream in)
     {
         //field_1_row       = LittleEndian.getShort(data, 0 + offset);

src/java/org/apache/poi/hssf/record/MulRKRecord.java

@@ -121,9 +121,7 @@ public class MulRKRecord
 
     /**
      * @param in the RecordInputstream to read the record from
-     * @param size size of data
      */
-
     protected void fillFields(RecordInputStream in)
     {
         //field_1_row       = LittleEndian.getShort(data, 0 + offset);

src/java/org/apache/poi/hssf/record/NameRecord.java

@@ -491,7 +491,8 @@ public class NameRecord extends Record {
     /**
      * called by the class that is responsible for writing this sucker.
      * Subclasses should implement this so that their data is passed back in a
-     * @param in the RecordInputstream to read the record from
+	 * @param offset to begin writing at
+	 * @param data byte array containing instance data
      * @return number of bytes written
      */
     public int serialize( int offset, byte[] data )

src/java/org/apache/poi/hssf/record/NoteRecord.java

@@ -194,8 +194,8 @@ public class NoteRecord extends Record {
      * Options flags.
      *
      * @return the options flag
-     * @see NoteRecord.NOTE_VISIBLE
+     * @see #NOTE_VISIBLE
-     * @see NoteRecord.NOTE_HIDDEN
+     * @see #NOTE_HIDDEN
      */
     public short getFlags(){
         return field_3_flags;

src/java/org/apache/poi/hssf/record/PageBreakRecord.java

@@ -35,14 +35,7 @@ import org.apache.poi.util.LittleEndian;
  * <p>REFERENCE:  Microsoft Excel SDK page 322 and 420</p>
  * 
  * @see HorizontalPageBreakRecord
-<<<<<<< PageBreakRecord.java
- * @see VerticalPageBreakREcord
- *
- * REFERENCE:  Microsoft Excel SDK page 322 and 420
-=======
  * @see VerticalPageBreakRecord
- * 
->>>>>>> 1.5
  * @author Danny Mui (dmui at apache dot org)
  */
 public class PageBreakRecord extends Record {

src/java/org/apache/poi/hssf/record/Record.java

@@ -66,9 +66,7 @@ public abstract class Record
      * called by the constructor, should set class level fields.  Should throw
      * runtime exception for bad/icomplete data.
      *
-     * @param data raw data
+     * @param in the RecordInputstream to read the record from
-     * @param size size of data
-     * @param offset of the record's data (provided a big array of the file)
      */
 
     protected abstract void fillFields(RecordInputStream in);

src/java/org/apache/poi/hssf/record/RecordInputStream.java

@@ -220,7 +220,7 @@ public class RecordInputStream extends InputStream
    *     
    * { 0x16, 0x00 } -0x16     
    *      
-   * @param len the length of the final string
+   * @param length the length of the final string
    * @return                                     the converted string
    * @exception  IllegalArgumentException        if len is too large (i.e.,
    *      there is not enough data in string to create a String of that     

src/java/org/apache/poi/hssf/record/SSTRecord.java

@@ -93,10 +93,7 @@ public class SSTRecord
     /**
      * Constructs an SST record and sets its fields appropriately.
      *
-     * @param id must be 0xfc or an exception will be throw upon
+     * @param in the RecordInputstream to read the record from
-     *           validation
-     * @param size the size of the data area of the record
-     * @param data of the record (should not contain sid/len)
      */
 
     public SSTRecord( RecordInputStream in )
@@ -352,8 +349,7 @@ public class SSTRecord
      * associated data. The UnicodeString class can handle the byte[]
      * vs short[] nature of the actual string data
      *
-     * @param data raw data
+     * @param in the RecordInputstream to read the record from
-     * @param size size of the raw data
      */
 
     protected void fillFields( RecordInputStream in )

src/java/org/apache/poi/hssf/record/StringRecord.java

@@ -67,8 +67,6 @@ public class StringRecord
 
     /**
      * @param in the RecordInputstream to read the record from
-     * @param size size of data
-     * @param offset of the record's data (provided a big array of the file)
      */
     protected void fillFields( RecordInputStream in)
     {

src/java/org/apache/poi/hssf/record/SupBookRecord.java

@@ -62,8 +62,6 @@ public class SupBookRecord extends Record
 
     /**
      * @param in the RecordInputstream to read the record from
-     * @param size size of data
-     * @param offset of the record's data (provided a big array of the file)
      */
     protected void fillFields(RecordInputStream in)
     {

src/java/org/apache/poi/hssf/record/TopMarginRecord.java

@@ -35,10 +35,7 @@ public class TopMarginRecord extends Record implements Margin
     /**
      * Constructs a TopMargin record and sets its fields appropriately.
      *
-     * @param id    id must be 0x28 or an exception
+     * @param in the RecordInputstream to read the record from
-     *               will be throw upon validation
-     * @param size  size the size of the data area of the record
-     * @param data  data of the record (should not contain sid/len)
      */
     public TopMarginRecord( RecordInputStream in )
     {        super( in );    }
@@ -56,6 +53,9 @@ public class TopMarginRecord extends Record implements Margin
         }
     }
 
+    /**
+     * @param in the RecordInputstream to read the record from
+     */
     protected void fillFields( RecordInputStream in )
     {
         field_1_margin = in.readDouble();
@@ -102,4 +102,4 @@ public class TopMarginRecord extends Record implements Margin
         rec.field_1_margin = this.field_1_margin;
         return rec;
     }
-}  // END OF CLASS
+}  // END OF CLASS

src/java/org/apache/poi/hssf/record/UnknownRecord.java

@@ -136,9 +136,7 @@ public class UnknownRecord
      * called by the constructor, should set class level fields.  Should throw
      * runtime exception for bad/icomplete data.
      *
-     * @param data raw data
+     * @param in the RecordInputstream to read the record from
-     * @param size size of data
-     * @param offset of the records data (provided a big array of the file)
      */
 
     protected void fillFields(RecordInputStream in)

src/java/org/apache/poi/hssf/record/aggregates/RowRecordsAggregate.java

@@ -230,14 +230,8 @@ public class RowRecordsAggregate
     }
 
     /**
-     * called by the constructor, should set class level fields.  Should throw
+     * You never fill an aggregate
-     * runtime exception for bad/incomplete data.
-     *
-     * @param data raw data
-     * @param size size of data
-     * @param offset of the record's data (provided a big array of the file)
      */
-
     protected void fillFields(RecordInputStream in)
     {
     }

src/java/org/apache/poi/hssf/record/aggregates/ValueRecordsAggregate.java

@@ -242,14 +242,8 @@ public class ValueRecordsAggregate
 
     
     /**
-     * called by the constructor, should set class level fields.  Should throw
+     * You never fill an aggregate
-     * runtime exception for bad/icomplete data.
-     *
-     * @param data raw data
-     * @param size size of data
-     * @param offset of the record's data (provided a big array of the file)
      */
-
     protected void fillFields(RecordInputStream in)
     {
     }

src/java/org/apache/poi/hssf/record/formula/GreaterThanPtg.java

@@ -49,8 +49,7 @@ public class GreaterThanPtg
 
     /**
      * Constructor. Create a new GreaterThanPtg.
-     * @param data the byte array to have the PTG added to
+     * @param in the RecordInputstream to read the record from
-     * @param offset the offset to the PTG to.
      */
     public GreaterThanPtg(RecordInputStream in)
     {

src/java/org/apache/poi/hssf/record/formula/LessThanPtg.java

@@ -58,8 +58,7 @@ public class LessThanPtg
 
     /**
      * Constructor. Create a new LessThanPtg.
-     * @param data the byte array to have the PTG added to
+     * @param in the RecordInputstream to read the record from
-     * @param offset the offset to the PTG to.
      */
     public LessThanPtg(RecordInputStream in)
     {

src/java/org/apache/poi/hssf/usermodel/HSSFDateUtil.java

@@ -160,6 +160,7 @@ public class HSSFDateUtil
      *  
      * @param formatIndex The index of the format, eg from ExtendedFormatRecord.getFormatIndex
      * @param formatString The format string
+     * @see #isInternalDateFormat(int,String) 
      */
     public static boolean isADateFormat(int formatIndex, String formatString) {
     	// First up, is this an internal date format?
@@ -187,7 +188,7 @@ public class HSSFDateUtil
     /**
      * Given a format ID this will check whether the format represents
      *  an internal excel date format or not.
-     * @see isDateFormat(int,String) 
+     * @see #isDateFormat(int,String) 
      */
     public static boolean isInternalDateFormat(int format) {
       boolean retval =false;

src/java/org/apache/poi/hssf/usermodel/HSSFFont.java

@@ -383,7 +383,6 @@ public class HSSFFont
 
     /**
      * set character-set to use.
-     * @return character-set
      * @see #ANSI_CHARSET
      * @see #DEFAULT_CHARSET
      * @see #SYMBOL_CHARSET