]> source.dussan.org Git - poi.git/commitdiff
Merged revisions 700280,700304,700327,700356 via svnmerge from
authorJosh Micich <josh@apache.org>
Tue, 30 Sep 2008 06:42:27 +0000 (06:42 +0000)
committerJosh Micich <josh@apache.org>
Tue, 30 Sep 2008 06:42:27 +0000 (06:42 +0000)
https://svn.apache.org/repos/asf/poi/trunk

........
  r700280 | josh | 2008-09-29 15:04:20 -0700 (Mon, 29 Sep 2008) | 1 line

  Fix for bug 45876 - allowed BoundSheetRecord to take sheet names longer than 31 chars
........
  r700304 | josh | 2008-09-29 16:12:53 -0700 (Mon, 29 Sep 2008) | 1 line

  Updated formula evaluator documentation due to bugzilla 45768
........
  r700327 | josh | 2008-09-29 19:30:53 -0700 (Mon, 29 Sep 2008) | 1 line

  Removed validateSid method from Record classes
........
  r700356 | josh | 2008-09-29 23:18:44 -0700 (Mon, 29 Sep 2008) | 1 line

  Refactored fillFields() method into constructor in Record class hierarchy
........

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

213 files changed:
src/documentation/content/xdocs/changes.xml
src/documentation/content/xdocs/status.xml
src/java/org/apache/poi/hssf/eventusermodel/dummyrecord/LastCellOfRowDummyRecord.java
src/java/org/apache/poi/hssf/eventusermodel/dummyrecord/MissingCellDummyRecord.java
src/java/org/apache/poi/hssf/eventusermodel/dummyrecord/MissingRowDummyRecord.java
src/java/org/apache/poi/hssf/model/Workbook.java
src/java/org/apache/poi/hssf/record/AbstractEscherHolderRecord.java
src/java/org/apache/poi/hssf/record/AreaFormatRecord.java
src/java/org/apache/poi/hssf/record/AreaRecord.java
src/java/org/apache/poi/hssf/record/AxisLineFormatRecord.java
src/java/org/apache/poi/hssf/record/AxisOptionsRecord.java
src/java/org/apache/poi/hssf/record/AxisParentRecord.java
src/java/org/apache/poi/hssf/record/AxisRecord.java
src/java/org/apache/poi/hssf/record/AxisUsedRecord.java
src/java/org/apache/poi/hssf/record/BOFRecord.java
src/java/org/apache/poi/hssf/record/BackupRecord.java
src/java/org/apache/poi/hssf/record/BarRecord.java
src/java/org/apache/poi/hssf/record/BeginRecord.java
src/java/org/apache/poi/hssf/record/BlankRecord.java
src/java/org/apache/poi/hssf/record/BookBoolRecord.java
src/java/org/apache/poi/hssf/record/BoolErrRecord.java
src/java/org/apache/poi/hssf/record/BottomMarginRecord.java
src/java/org/apache/poi/hssf/record/BoundSheetRecord.java
src/java/org/apache/poi/hssf/record/CFHeaderRecord.java
src/java/org/apache/poi/hssf/record/CFRuleRecord.java
src/java/org/apache/poi/hssf/record/CRNCountRecord.java
src/java/org/apache/poi/hssf/record/CRNRecord.java
src/java/org/apache/poi/hssf/record/CalcCountRecord.java
src/java/org/apache/poi/hssf/record/CalcModeRecord.java
src/java/org/apache/poi/hssf/record/CategorySeriesAxisRecord.java
src/java/org/apache/poi/hssf/record/ChartFormatRecord.java
src/java/org/apache/poi/hssf/record/ChartRecord.java
src/java/org/apache/poi/hssf/record/ChartTitleFormatRecord.java
src/java/org/apache/poi/hssf/record/CodepageRecord.java
src/java/org/apache/poi/hssf/record/ColumnInfoRecord.java
src/java/org/apache/poi/hssf/record/CommonObjectDataSubRecord.java
src/java/org/apache/poi/hssf/record/ContinueRecord.java
src/java/org/apache/poi/hssf/record/CountryRecord.java
src/java/org/apache/poi/hssf/record/DBCellRecord.java
src/java/org/apache/poi/hssf/record/DSFRecord.java
src/java/org/apache/poi/hssf/record/DVALRecord.java
src/java/org/apache/poi/hssf/record/DVRecord.java
src/java/org/apache/poi/hssf/record/DatRecord.java
src/java/org/apache/poi/hssf/record/DataFormatRecord.java
src/java/org/apache/poi/hssf/record/DateWindow1904Record.java
src/java/org/apache/poi/hssf/record/DefaultColWidthRecord.java
src/java/org/apache/poi/hssf/record/DefaultDataLabelTextPropertiesRecord.java
src/java/org/apache/poi/hssf/record/DefaultRowHeightRecord.java
src/java/org/apache/poi/hssf/record/DeltaRecord.java
src/java/org/apache/poi/hssf/record/DimensionsRecord.java
src/java/org/apache/poi/hssf/record/DrawingRecord.java
src/java/org/apache/poi/hssf/record/EOFRecord.java
src/java/org/apache/poi/hssf/record/EmbeddedObjectRefSubRecord.java
src/java/org/apache/poi/hssf/record/EndRecord.java
src/java/org/apache/poi/hssf/record/EndSubRecord.java
src/java/org/apache/poi/hssf/record/EscherAggregate.java
src/java/org/apache/poi/hssf/record/ExtSSTInfoSubRecord.java
src/java/org/apache/poi/hssf/record/ExtSSTRecord.java
src/java/org/apache/poi/hssf/record/ExtendedFormatRecord.java
src/java/org/apache/poi/hssf/record/ExternSheetRecord.java
src/java/org/apache/poi/hssf/record/ExternalNameRecord.java
src/java/org/apache/poi/hssf/record/FilePassRecord.java
src/java/org/apache/poi/hssf/record/FileSharingRecord.java
src/java/org/apache/poi/hssf/record/FnGroupCountRecord.java
src/java/org/apache/poi/hssf/record/FontBasisRecord.java
src/java/org/apache/poi/hssf/record/FontIndexRecord.java
src/java/org/apache/poi/hssf/record/FontRecord.java
src/java/org/apache/poi/hssf/record/FooterRecord.java
src/java/org/apache/poi/hssf/record/FormatRecord.java
src/java/org/apache/poi/hssf/record/FormulaRecord.java
src/java/org/apache/poi/hssf/record/FrameRecord.java
src/java/org/apache/poi/hssf/record/GridsetRecord.java
src/java/org/apache/poi/hssf/record/GroupMarkerSubRecord.java
src/java/org/apache/poi/hssf/record/GutsRecord.java
src/java/org/apache/poi/hssf/record/HCenterRecord.java
src/java/org/apache/poi/hssf/record/HeaderRecord.java
src/java/org/apache/poi/hssf/record/HideObjRecord.java
src/java/org/apache/poi/hssf/record/HorizontalPageBreakRecord.java
src/java/org/apache/poi/hssf/record/HyperlinkRecord.java
src/java/org/apache/poi/hssf/record/IndexRecord.java
src/java/org/apache/poi/hssf/record/InterfaceEndRecord.java
src/java/org/apache/poi/hssf/record/InterfaceHdrRecord.java
src/java/org/apache/poi/hssf/record/IterationRecord.java
src/java/org/apache/poi/hssf/record/LabelRecord.java
src/java/org/apache/poi/hssf/record/LabelSSTRecord.java
src/java/org/apache/poi/hssf/record/LeftMarginRecord.java
src/java/org/apache/poi/hssf/record/LegendRecord.java
src/java/org/apache/poi/hssf/record/LineFormatRecord.java
src/java/org/apache/poi/hssf/record/LinkedDataFormulaField.java
src/java/org/apache/poi/hssf/record/LinkedDataRecord.java
src/java/org/apache/poi/hssf/record/MMSRecord.java
src/java/org/apache/poi/hssf/record/MergeCellsRecord.java
src/java/org/apache/poi/hssf/record/MulBlankRecord.java
src/java/org/apache/poi/hssf/record/MulRKRecord.java
src/java/org/apache/poi/hssf/record/NameRecord.java
src/java/org/apache/poi/hssf/record/NoteRecord.java
src/java/org/apache/poi/hssf/record/NoteStructureSubRecord.java
src/java/org/apache/poi/hssf/record/NumberFormatIndexRecord.java
src/java/org/apache/poi/hssf/record/NumberRecord.java
src/java/org/apache/poi/hssf/record/ObjRecord.java
src/java/org/apache/poi/hssf/record/ObjectLinkRecord.java
src/java/org/apache/poi/hssf/record/ObjectProtectRecord.java
src/java/org/apache/poi/hssf/record/PageBreakRecord.java
src/java/org/apache/poi/hssf/record/PaletteRecord.java
src/java/org/apache/poi/hssf/record/PaneRecord.java
src/java/org/apache/poi/hssf/record/PasswordRecord.java
src/java/org/apache/poi/hssf/record/PasswordRev4Record.java
src/java/org/apache/poi/hssf/record/PlotAreaRecord.java
src/java/org/apache/poi/hssf/record/PlotGrowthRecord.java
src/java/org/apache/poi/hssf/record/PrecisionRecord.java
src/java/org/apache/poi/hssf/record/PrintGridlinesRecord.java
src/java/org/apache/poi/hssf/record/PrintHeadersRecord.java
src/java/org/apache/poi/hssf/record/PrintSetupRecord.java
src/java/org/apache/poi/hssf/record/ProtectRecord.java
src/java/org/apache/poi/hssf/record/ProtectionRev4Record.java
src/java/org/apache/poi/hssf/record/RKRecord.java
src/java/org/apache/poi/hssf/record/RecalcIdRecord.java
src/java/org/apache/poi/hssf/record/Record.java
src/java/org/apache/poi/hssf/record/RefModeRecord.java
src/java/org/apache/poi/hssf/record/RefreshAllRecord.java
src/java/org/apache/poi/hssf/record/RightMarginRecord.java
src/java/org/apache/poi/hssf/record/RowRecord.java
src/java/org/apache/poi/hssf/record/SCLRecord.java
src/java/org/apache/poi/hssf/record/SSTRecord.java
src/java/org/apache/poi/hssf/record/SaveRecalcRecord.java
src/java/org/apache/poi/hssf/record/ScenarioProtectRecord.java
src/java/org/apache/poi/hssf/record/SelectionRecord.java
src/java/org/apache/poi/hssf/record/SeriesChartGroupIndexRecord.java
src/java/org/apache/poi/hssf/record/SeriesIndexRecord.java
src/java/org/apache/poi/hssf/record/SeriesLabelsRecord.java
src/java/org/apache/poi/hssf/record/SeriesListRecord.java
src/java/org/apache/poi/hssf/record/SeriesRecord.java
src/java/org/apache/poi/hssf/record/SeriesTextRecord.java
src/java/org/apache/poi/hssf/record/SeriesToChartGroupRecord.java
src/java/org/apache/poi/hssf/record/SharedValueRecordBase.java
src/java/org/apache/poi/hssf/record/SheetPropertiesRecord.java
src/java/org/apache/poi/hssf/record/StringRecord.java
src/java/org/apache/poi/hssf/record/StyleRecord.java
src/java/org/apache/poi/hssf/record/SubRecord.java
src/java/org/apache/poi/hssf/record/SupBookRecord.java
src/java/org/apache/poi/hssf/record/TabIdRecord.java
src/java/org/apache/poi/hssf/record/TextObjectBaseRecord.java
src/java/org/apache/poi/hssf/record/TextObjectRecord.java
src/java/org/apache/poi/hssf/record/TextRecord.java
src/java/org/apache/poi/hssf/record/TickRecord.java
src/java/org/apache/poi/hssf/record/TopMarginRecord.java
src/java/org/apache/poi/hssf/record/UncalcedRecord.java
src/java/org/apache/poi/hssf/record/UnicodeString.java
src/java/org/apache/poi/hssf/record/UnitsRecord.java
src/java/org/apache/poi/hssf/record/UnknownRecord.java
src/java/org/apache/poi/hssf/record/UseSelFSRecord.java
src/java/org/apache/poi/hssf/record/VCenterRecord.java
src/java/org/apache/poi/hssf/record/ValueRangeRecord.java
src/java/org/apache/poi/hssf/record/VerticalPageBreakRecord.java
src/java/org/apache/poi/hssf/record/WSBoolRecord.java
src/java/org/apache/poi/hssf/record/WindowOneRecord.java
src/java/org/apache/poi/hssf/record/WindowProtectRecord.java
src/java/org/apache/poi/hssf/record/WindowTwoRecord.java
src/java/org/apache/poi/hssf/record/WriteAccessRecord.java
src/java/org/apache/poi/hssf/record/WriteProtectRecord.java
src/java/org/apache/poi/hssf/usermodel/HSSFFormulaEvaluator.java
src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java
src/testcases/org/apache/poi/hssf/record/TestAreaFormatRecord.java
src/testcases/org/apache/poi/hssf/record/TestAreaRecord.java
src/testcases/org/apache/poi/hssf/record/TestAxisLineFormatRecord.java
src/testcases/org/apache/poi/hssf/record/TestAxisOptionsRecord.java
src/testcases/org/apache/poi/hssf/record/TestAxisParentRecord.java
src/testcases/org/apache/poi/hssf/record/TestAxisRecord.java
src/testcases/org/apache/poi/hssf/record/TestAxisUsedRecord.java
src/testcases/org/apache/poi/hssf/record/TestBarRecord.java
src/testcases/org/apache/poi/hssf/record/TestBoundSheetRecord.java
src/testcases/org/apache/poi/hssf/record/TestCategorySeriesAxisRecord.java
src/testcases/org/apache/poi/hssf/record/TestChartRecord.java
src/testcases/org/apache/poi/hssf/record/TestCommonObjectDataSubRecord.java
src/testcases/org/apache/poi/hssf/record/TestDatRecord.java
src/testcases/org/apache/poi/hssf/record/TestDataFormatRecord.java
src/testcases/org/apache/poi/hssf/record/TestDefaultDataLabelTextPropertiesRecord.java
src/testcases/org/apache/poi/hssf/record/TestEndSubRecord.java
src/testcases/org/apache/poi/hssf/record/TestExtendedFormatRecord.java
src/testcases/org/apache/poi/hssf/record/TestFontBasisRecord.java
src/testcases/org/apache/poi/hssf/record/TestFontIndexRecord.java
src/testcases/org/apache/poi/hssf/record/TestFontRecord.java
src/testcases/org/apache/poi/hssf/record/TestFrameRecord.java
src/testcases/org/apache/poi/hssf/record/TestLegendRecord.java
src/testcases/org/apache/poi/hssf/record/TestLineFormatRecord.java
src/testcases/org/apache/poi/hssf/record/TestLinkedDataRecord.java
src/testcases/org/apache/poi/hssf/record/TestNoteRecord.java
src/testcases/org/apache/poi/hssf/record/TestNoteStructureSubRecord.java
src/testcases/org/apache/poi/hssf/record/TestNumberFormatIndexRecord.java
src/testcases/org/apache/poi/hssf/record/TestObjectLinkRecord.java
src/testcases/org/apache/poi/hssf/record/TestPaneRecord.java
src/testcases/org/apache/poi/hssf/record/TestPlotAreaRecord.java
src/testcases/org/apache/poi/hssf/record/TestPlotGrowthRecord.java
src/testcases/org/apache/poi/hssf/record/TestSCLRecord.java
src/testcases/org/apache/poi/hssf/record/TestSeriesChartGroupIndexRecord.java
src/testcases/org/apache/poi/hssf/record/TestSeriesIndexRecord.java
src/testcases/org/apache/poi/hssf/record/TestSeriesLabelsRecord.java
src/testcases/org/apache/poi/hssf/record/TestSeriesListRecord.java
src/testcases/org/apache/poi/hssf/record/TestSeriesRecord.java
src/testcases/org/apache/poi/hssf/record/TestSeriesTextRecord.java
src/testcases/org/apache/poi/hssf/record/TestSeriesToChartGroupRecord.java
src/testcases/org/apache/poi/hssf/record/TestSheetPropertiesRecord.java
src/testcases/org/apache/poi/hssf/record/TestStringRecord.java
src/testcases/org/apache/poi/hssf/record/TestSupBookRecord.java
src/testcases/org/apache/poi/hssf/record/TestTableRecord.java
src/testcases/org/apache/poi/hssf/record/TestTextObjectBaseRecord.java
src/testcases/org/apache/poi/hssf/record/TestTextObjectRecord.java
src/testcases/org/apache/poi/hssf/record/TestTextRecord.java
src/testcases/org/apache/poi/hssf/record/TestTickRecord.java
src/testcases/org/apache/poi/hssf/record/TestUnitsRecord.java
src/testcases/org/apache/poi/hssf/record/TestValueRangeRecord.java
src/testcases/org/apache/poi/hssf/usermodel/TestHSSFSheet.java
src/testcases/org/apache/poi/hssf/usermodel/TestHSSFWorkbook.java

index e74bf9296f44cf6f2df0d4d08d29f03f68deb04b..36f01ceea58d5ff5595d8b28354e6e2f8d165cf2 100644 (file)
@@ -67,6 +67,7 @@
            <action dev="POI-DEVELOPERS" type="add">Created a common interface for handling Excel files, irrespective of if they are .xls or .xlsx</action>
         </release>
         <release version="3.2-alpha1" date="2008-??-??">
+           <action dev="POI-DEVELOPERS" type="fix">45876 - fixed BoundSheetRecord to allow sheet names longer than 31 chars</action>
            <action dev="POI-DEVELOPERS" type="add">45890 - fixed HSSFSheet.shiftRows to also update conditional formats</action>
            <action dev="POI-DEVELOPERS" type="add">45865 modified Formula Parser/Evaluator to handle cross-worksheet formulas</action>
            <action dev="POI-DEVELOPERS" type="add">Optimised the FormulaEvaluator to take cell dependencies into account</action>
index c331931b21fec94a1673e65d9c129324bc1a785b..bcf430664c73c4fa437af45c7612abd000d9f7a8 100644 (file)
@@ -64,6 +64,7 @@
            <action dev="POI-DEVELOPERS" type="add">Created a common interface for handling Excel files, irrespective of if they are .xls or .xlsx</action>
         </release>
         <release version="3.2-alpha1" date="2008-??-??">
+           <action dev="POI-DEVELOPERS" type="fix">45876 - fixed BoundSheetRecord to allow sheet names longer than 31 chars</action>
            <action dev="POI-DEVELOPERS" type="add">45890 - fixed HSSFSheet.shiftRows to also update conditional formats</action>
            <action dev="POI-DEVELOPERS" type="add">45865 modified Formula Parser/Evaluator to handle cross-worksheet formulas</action>
            <action dev="POI-DEVELOPERS" type="add">Optimised the FormulaEvaluator to take cell dependencies into account</action>
index b259788a85a0984212b86c2c1516cef7758617ec..670fa99d2a9784d0930636547f1c7eed83ba56c6 100644 (file)
@@ -18,7 +18,6 @@
 package org.apache.poi.hssf.eventusermodel.dummyrecord;
 
 import org.apache.poi.hssf.record.Record;
-import org.apache.poi.hssf.record.RecordInputStream;
 
 /**
  * A dummy record to indicate that we've now had the last
@@ -47,15 +46,10 @@ public class LastCellOfRowDummyRecord extends Record {
         */
        public int getLastColumnNumber() { return lastColumnNumber; }
        
-       protected void fillFields(RecordInputStream in) {
-       }
        public short getSid() {
                return -1;
        }
        public int serialize(int offset, byte[] data) {
                return -1;
        }
-       protected void validateSid(short id) {
-       }
-
 }
index 61dc7ac5bd1a7d3c80f910855d83694d41432c25..587ccb6596cfc81517ae2559bc9e97adcadf56c6 100644 (file)
@@ -18,7 +18,6 @@
 package org.apache.poi.hssf.eventusermodel.dummyrecord;
 
 import org.apache.poi.hssf.record.Record;
-import org.apache.poi.hssf.record.RecordInputStream;
 
 /**
  * A dummy record for when we're missing a cell in a row,
@@ -33,16 +32,12 @@ public class MissingCellDummyRecord extends Record {
                this.column = column;
        }
        
-       protected void fillFields(RecordInputStream in) {
-       }
        public short getSid() {
                return -1;
        }
        public int serialize(int offset, byte[] data) {
                return -1;
        }
-       protected void validateSid(short id) {
-       }
        
        public int getRow() { return row; }
        public int getColumn() { return column; }
index 68a86528ac2fbeefb642b7d364b1c93b92a6fd7f..46f76b1d348db665842b96ae78ecdaf5b4cc326a 100644 (file)
@@ -31,16 +31,12 @@ public class MissingRowDummyRecord extends Record {
                this.rowNumber = rowNumber;
        }
        
-       protected void fillFields(RecordInputStream in) {
-       }
        public short getSid() {
                return -1;
        }
        public int serialize(int offset, byte[] data) {
                return -1;
        }
-       protected void validateSid(short id) {
-       }
        
        public int getRowNumber() {
                return rowNumber;
index 0728d0e29ffdeb37d72cedcac65e7fa15ff494f3..54639adb182e114b8225cb0039e09cc8c2efb78c 100644 (file)
@@ -55,6 +55,12 @@ import org.apache.poi.util.POILogger;
  * @version 1.0-pre
  */
 public final class Workbook implements Model {
+    /**
+     * Excel silently truncates long sheet names to 31 chars.
+     * This constant is used to ensure uniqueness in the first 31 chars
+     */
+    private static final int MAX_SENSITIVE_SHEET_NAME_LEN = 31;
+
     private static final int   DEBUG       = POILogger.DEBUG;
 
     /**
@@ -488,32 +494,43 @@ public final class Workbook implements Model {
 
     /**
      * sets the name for a given sheet.  If the boundsheet record doesn't exist and
-     * its only one more than we have, go ahead and create it.  If its > 1 more than
+     * its only one more than we have, go ahead and create it.  If it's > 1 more than
      * we have, except
      *
      * @param sheetnum the sheet number (0 based)
      * @param sheetname the name for the sheet
      */
-    public void setSheetName(int sheetnum, String sheetname ) {
+    public void setSheetName(int sheetnum, String sheetname) {
         checkSheets(sheetnum);
         BoundSheetRecord sheet = (BoundSheetRecord)boundsheets.get( sheetnum );
         sheet.setSheetname(sheetname);
     }
 
     /**
-     * Determines whether a workbook contains the provided sheet name.
+     * Determines whether a workbook contains the provided sheet name.  For the purpose of 
+     * comparison, long names are truncated to 31 chars.
      *
      * @param name the name to test (case insensitive match)
      * @param excludeSheetIdx the sheet to exclude from the check or -1 to include all sheets in the check.
      * @return true if the sheet contains the name, false otherwise.
      */
-    public boolean doesContainsSheetName( String name, int excludeSheetIdx )
-    {
-        for ( int i = 0; i < boundsheets.size(); i++ )
-        {
+    public boolean doesContainsSheetName(String name, int excludeSheetIdx) {
+        String aName = name;
+        if (aName.length() > MAX_SENSITIVE_SHEET_NAME_LEN) {
+            aName = aName.substring(0, MAX_SENSITIVE_SHEET_NAME_LEN);
+        }
+        for (int i = 0; i < boundsheets.size(); i++) {
             BoundSheetRecord boundSheetRecord = getBoundSheetRec(i);
-            if (excludeSheetIdx != i && name.equalsIgnoreCase(boundSheetRecord.getSheetname()))
+            if (excludeSheetIdx == i) {
+                continue;
+            }
+            String bName = boundSheetRecord.getSheetname();
+            if (bName.length() > MAX_SENSITIVE_SHEET_NAME_LEN) {
+                bName = bName.substring(0, MAX_SENSITIVE_SHEET_NAME_LEN);
+            }
+            if (aName.equalsIgnoreCase(bName)) {
                 return true;
+            }
         }
         return false;
     }
@@ -1954,9 +1971,9 @@ public final class Workbook implements Model {
         return (short)getOrCreateLinkTable().checkExternSheet(sheetNumber);
     }
 
-       public int getExternalSheetIndex(String workbookName, String sheetName) {
-               return getOrCreateLinkTable().getExternalSheetIndex(workbookName, sheetName);
-       }
+    public int getExternalSheetIndex(String workbookName, String sheetName) {
+        return getOrCreateLinkTable().getExternalSheetIndex(workbookName, sheetName);
+    }
     
 
     /** gets the total number of names
index c5fae27ae37415af86d613ae58aa57e2b72c7f75..295a86a703d603f6571280e30d699cdbe019525b 100644 (file)
@@ -57,31 +57,7 @@ public abstract class AbstractEscherHolderRecord
         escherRecords = new ArrayList();
     }
 
-    /**
-     * Constructs a Bar record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
     public AbstractEscherHolderRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != getSid())
-        {
-            throw new RecordFormatException("Not an escher record! (sid was " + id + ", expecting " + getSid() + ")");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         escherRecords = new ArrayList();
         if (! DESERIALISE )
index 843128120efb22d8fcba5f09a0eec2411b19220b..65658ab3ed5d8ef1098aa1868d018a4ef5484bbd 100644 (file)
@@ -48,32 +48,7 @@ public final class AreaFormatRecord extends Record {
 
     }
 
-    /**
-     * Constructs a AreaFormat record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public AreaFormatRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a AreaFormat record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_foregroundColor        = in.readInt();
         field_2_backgroundColor        = in.readInt();
index fd652894606a81566ef223dbbf689f9b667dc793..de994427f586834452af88057cfdb97b2332c5dc 100644 (file)
@@ -42,32 +42,7 @@ public final class AreaRecord extends Record {
 
     }
 
-    /**
-     * Constructs a Area record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public AreaRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Area record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
 
         field_1_formatFlags            = in.readShort();
index e46ce1a5d2f214ee0b00b43c156f45de199dbdb2..64d2a071d96973b9b58802f655f8aa3c47b55c4f 100644 (file)
@@ -46,32 +46,7 @@ public class AxisLineFormatRecord
 
     }
 
-    /**
-     * Constructs a AxisLineFormat record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public AxisLineFormatRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a AxisLineFormat record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_axisType               = in.readShort();
     }
index 8be3f200be2ece1b9a10d1199cdc0117d6901bd0..54cd2b49bcc689c67b1e292987558d31d929d6d8 100644 (file)
@@ -57,32 +57,7 @@ public final class AxisOptionsRecord extends Record {
 
     }
 
-    /**
-     * Constructs a AxisOptions record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public AxisOptionsRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a AxisOptions record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_minimumCategory        = in.readShort();
         field_2_maximumCategory        = in.readShort();
index 3c4d5985459f30d3d6c2b2e0f60d5bfd8eed1bbb..281a0824f53d9f5c7eedf3a61240e2a7ef8ef350 100644 (file)
@@ -48,32 +48,7 @@ public class AxisParentRecord
 
     }
 
-    /**
-     * Constructs a AxisParent record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public AxisParentRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a AxisParent record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_axisType               = in.readShort();
         field_2_x                      = in.readInt();
index b18c9c32fdde381715331401f33036a39cd42930..b57ac1d4f93aff5fd638d1681bdfdf1bc7f08df1 100644 (file)
@@ -49,32 +49,7 @@ public class AxisRecord
 
     }
 
-    /**
-     * Constructs a Axis record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public AxisRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Axis record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_axisType               = in.readShort();
         field_2_reserved1              = in.readInt();
index 2b3937978a08368b78dc0a52f9850fcdbc3b15ce..361ae2304f76881190709696f59f8a919cd91ec3 100644 (file)
@@ -42,32 +42,7 @@ public class AxisUsedRecord
 
     }
 
-    /**
-     * Constructs a AxisUsed record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public AxisUsedRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a AxisUsed record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_numAxis                = in.readShort();
     }
index dd5335a420b2f3894396625ff2f25d95046efae0..672505f5b0f9f1e0f616c7ff8d20d1ee2115b58a 100644 (file)
@@ -86,27 +86,7 @@ public class BOFRecord
     {
     }
 
-    /**
-     * Constructs a BOFRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
-
     public BOFRecord(RecordInputStream in)
-    {
-        super(in);
-
-        // fillFields(data,size);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A BOF RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_version  = in.readShort();
         field_2_type     = in.readShort();
index 6ee6cccfb68f64a98b89483bebf5801a43c977e7..2ffbb5ba796e5700b3f61056ee6462fa131338bb 100644 (file)
@@ -40,25 +40,7 @@ public class BackupRecord
     {
     }
 
-    /**
-     * Constructs a BackupRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
-
     public BackupRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A BACKUP RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_backup = in.readShort();
     }
index 0c88385ba8e6cf72d2d04f6be65193b0e364a1cc..dbf41202ac404849e0fba0053e871a0a35b68921 100644 (file)
@@ -47,32 +47,7 @@ public final class BarRecord extends Record {
 
     }
 
-    /**
-     * Constructs a Bar record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public BarRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Bar record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_barSpace               = in.readShort();
         field_2_categorySpace          = in.readShort();
index b899d3a6a8d8f29b4e95f0be02668c0e5f317469..c983590bd64228fb275b5dd2b2c1da655fb97fbd 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -29,10 +27,7 @@ import org.apache.poi.util.LittleEndian;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-
-public class BeginRecord
-    extends Record
-{
+public class BeginRecord extends Record {
     public static final short sid = 0x1033;
 
     public BeginRecord()
@@ -40,24 +35,9 @@ public class BeginRecord
     }
 
     /**
-     * Constructs a BeginRecord record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
+     * @param in unused (since this record has no data)
      */
-
     public BeginRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A BEGIN RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
     }
 
@@ -72,9 +52,8 @@ public class BeginRecord
 
     public int serialize(int offset, byte [] data)
     {
-        LittleEndian.putShort(data, 0 + offset, sid);
-        LittleEndian.putShort(data, 2 + offset,
-                              (( short ) 0));   // no record info
+        LittleEndian.putUShort(data, 0 + offset, sid);
+        LittleEndian.putUShort(data, 2 + offset, 0);   // no record info
         return getRecordSize();
     }
 
index cf1b97f5a58a224ff9614c99d5b39f34c644e5bd..864cc53c3c9e666dbee76a5fe77a66a09a28d923 100644 (file)
@@ -39,36 +39,13 @@ public final class BlankRecord extends Record implements CellValueRecordInterfac
     {
     }
 
-    /**
-     * Constructs a BlankRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
     public BlankRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_row = in.readUShort();
         field_2_col = in.readShort();
         field_3_xf  = in.readShort();
     }
 
-    /**
-     * called by constructor, should throw runtime exception in the event of a
-     * record passed with a differing ID.
-     *
-     * @param id alleged id for this record
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A BLANKRECORD!");
-        }
-    }
-
     /**
      * set the row this cell occurs on
      * @param row the row this cell occurs within
index 6b5cf98836f5a61897fc3febd82f1f5dd2c8d311..0ad7a5ac6096e72a2cae766a9358b9f95c86b7d9 100644 (file)
@@ -40,25 +40,7 @@ public class BookBoolRecord
     {
     }
 
-    /**
-     * Constructs a BookBoolRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
-
     public BookBoolRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A BOOKBOOL RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_save_link_values = in.readShort();
     }
index 11f562d559eeca849f5e3c86852cc3ffdb0d9164..5595c4f0ddd5d78b71ff6e2b40a411699508aac6 100644 (file)
@@ -41,19 +41,9 @@ public final class BoolErrRecord extends Record implements CellValueRecordInterf
     }
 
     /**
-     * Constructs a BoolErr record and sets its fields appropriately.
-     *
      * @param in the RecordInputstream to read the record from
      */
     public BoolErrRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    /**
-     * @param in the RecordInputstream to read the record from
-     */
-    protected void fillFields(RecordInputStream in)
     {
         //field_1_row      = LittleEndian.getShort(data, 0 + offset);
         field_1_row      = in.readUShort();
@@ -221,20 +211,6 @@ public final class BoolErrRecord extends Record implements CellValueRecordInterf
         return 12;
     }
 
-    /**
-     * called by constructor, should throw runtime exception in the event of a
-     * record passed with a differing ID.
-     *
-     * @param id alleged id for this record
-     */
-    protected void validateSid(short id)
-    {
-        if (id != BoolErrRecord.sid)
-        {
-            throw new RecordFormatException("Not a valid BoolErrRecord");
-        }
-    }
-
     public short getSid()
     {
         return sid;
index fc535e06b9a6ced0137af7b65b9446a778b00c4b..c8f50ff1c14aa24ae4be2095446a4576e50a4a7f 100644 (file)
@@ -39,30 +39,7 @@ public class BottomMarginRecord
 
     }
 
-    /**
-     * Constructs a BottomMargin record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
     public BottomMarginRecord( RecordInputStream in )
-    {
-        super( in );
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id the expected sid.
-     */
-    protected void validateSid( short id )
-    {
-        if ( id != sid )
-        {
-            throw new RecordFormatException( "Not a BottomMargin record" );
-        }
-    }
-
-    protected void fillFields( RecordInputStream in )
     {
         field_1_margin = in.readDouble();
     }
@@ -121,4 +98,4 @@ public class BottomMarginRecord
         return rec;
     }
 
-}  // END OF CLASS
+}  // END OF CLAS
\ No newline at end of file
index 2c1d0220f504f46bcf13356d4819d9acfb18e17d..036de6b2dfe91d4586873b80cfb32d11c296eb3a 100644 (file)
@@ -52,21 +52,6 @@ public final class BoundSheetRecord extends Record {
                setSheetname(sheetname);
        }
 
-       /**
-        * Constructs a BoundSheetRecord and sets its fields appropriately
-        *
-        * @param in the RecordInputstream to read the record from
-        */
-       public BoundSheetRecord(RecordInputStream in) {
-               super(in);
-       }
-
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT A Bound Sheet RECORD");
-               }
-       }
-
        /**
         * UTF8: sid + len + bof + flags + len(str) + unicode + str 2 + 2 + 4 + 2 +
         * 1 + 1 + len(str)
@@ -75,7 +60,7 @@ public final class BoundSheetRecord extends Record {
         * 1 + 1 + 2 * len(str)
         * 
         */
-       protected void fillFields(RecordInputStream in) {
+       public BoundSheetRecord(RecordInputStream in) {
                field_1_position_of_BOF = in.readInt();
                field_2_option_flags = in.readUShort();
                int field_3_sheetname_length = in.readUByte();
@@ -116,9 +101,8 @@ public final class BoundSheetRecord extends Record {
                        throw new IllegalArgumentException("sheetName must not be null");
                }
                int len = sheetName.length();
-               if (len < 1 || len > 31) {
-                       throw new IllegalArgumentException("sheetName '" + sheetName 
-                                       + "' is invalid - must be 1-30 characters long");
+               if (len < 1) {
+                       throw new IllegalArgumentException("sheetName must not be empty string");
                }
                for (int i=0; i<len; i++) {
                        char ch = sheetName.charAt(i);
index 7ecc7f6f8fe87f3bfd3933bbd5c18b186863cd0c..708b20d857967cd299eecba8ef0f47d72f658f72 100644 (file)
@@ -48,11 +48,6 @@ public final class CFHeaderRecord extends Record {
        }
 
        public CFHeaderRecord(RecordInputStream in)
-       {
-               super(in);
-       }
-
-       protected void fillFields(RecordInputStream in)
        {
                field_1_numcf = in.readShort();
                field_2_need_recalculation = in.readShort();
@@ -160,22 +155,6 @@ public final class CFHeaderRecord extends Record {
                return 4 + getDataSize();
        }
 
-       /**
-        * called by constructor, should throw runtime exception in the event of a
-        * record passed with a differing ID.
-        *
-        * @param id alleged id for this record
-        */
-
-       protected void validateSid(short id)
-       {
-               if (id != sid)
-               {
-                       throw new RecordFormatException(
-                                       "NOT A ConditionalFormattingHeaderRecord RECORD");
-               }
-       }
-
        public short getSid()
        {
                return sid;
index e676d431205845e89bceb409feadb6ad312e352d..ef8e05c29d54998b6deac6f22f63e7d6ab89edb1 100644 (file)
@@ -159,10 +159,6 @@ public final class CFRuleRecord extends Record {
        }
 
        public CFRuleRecord(RecordInputStream in) {
-               super(in);
-       }
-
-       protected void fillFields(RecordInputStream in) {
                field_1_condition_type = in.readByte();
                field_2_comparison_operator = in.readByte();
                int field_3_formula1_len = in.readUShort();
@@ -446,21 +442,6 @@ public final class CFRuleRecord extends Record {
        public void setParsedExpression2(Ptg[] ptgs) {
                field_18_formula2 = safeClone(ptgs);
        }
-       
-       /**
-        * called by constructor, should throw runtime exception in the event of a
-        * record passed with a differing ID.
-        *
-        * @param id alleged id for this record
-        */
-
-       protected void validateSid(short id)
-       {
-               if (id != sid)
-               {
-                       throw new RecordFormatException("NOT A CFRULE RECORD");
-               }
-       }
 
        public short getSid()
        {
index 4c9e4425cd0b6f8f7eb4c9cbfbd2c55ea2411371..5df98bf34edcc7dcc85ab06d0822c530a834af95 100755 (executable)
@@ -38,22 +38,12 @@ public final class CRNCountRecord extends Record {
                throw new RuntimeException("incomplete code");
        }
 
-       public CRNCountRecord(RecordInputStream in) {
-               super(in);
-       }
-
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT An XCT RECORD");
-               }
-       }
-
        public int getNumberOfCRNs() {
                return field_1_number_crn_records;
        }
 
 
-       protected void fillFields(RecordInputStream in) {
+       public CRNCountRecord(RecordInputStream in) {
                field_1_number_crn_records = in.readShort();
                if(field_1_number_crn_records < 0) {
                        // TODO - seems like the sign bit of this field might be used for some other purpose
index fe66d638b76278c198e029c539901d74484e2d2c..417f2a559a3250c749facea04750cb82f51bffec 100755 (executable)
@@ -39,22 +39,12 @@ public final class CRNRecord extends Record {
                throw new RuntimeException("incomplete code");
        }
 
-       public CRNRecord(RecordInputStream in) {
-               super(in);
-       }
-
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT An XCT RECORD");
-               }
-       }
-
        public int getNumberOfCRNs() {
                return field_1_last_column_index;
        }
 
 
-       protected void fillFields(RecordInputStream in) {
+       public CRNRecord(RecordInputStream in) {
                field_1_last_column_index = in.readByte() & 0x00FF;
                field_2_first_column_index = in.readByte() & 0x00FF;
                field_3_row_index = in.readShort();
index 307dfa35ee39d7ed010028f513f97c35cbca2359..eb783f3857bf2313cebf6c5a56eb844b09370032 100644 (file)
@@ -45,26 +45,7 @@ public class CalcCountRecord
     {
     }
 
-    /**
-     * Constructs a CalcCountRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     *
-     */
-
     public CalcCountRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An Calc Count RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_iterations = in.readShort();
     }
index b36123638e6349084d38fcdd2ccc61157d33f623..de8d860fcec1d0700433b7be16d51743c5fc3588 100644 (file)
@@ -61,25 +61,7 @@ public class CalcModeRecord
     {
     }
 
-    /**
-     * Constructs a CalcModeRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
-
     public CalcModeRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An Calc Mode RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_calcmode = in.readShort();
     }
index 0521ea709cdfa53f0f5b703bed367105d52b727d..158b3e5cf6c53540bdd3efb494f42b037bdfa70f 100644 (file)
@@ -47,32 +47,7 @@ public final class CategorySeriesAxisRecord extends Record {
 
     }
 
-    /**
-     * Constructs a CategorySeriesAxis record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public CategorySeriesAxisRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a CategorySeriesAxis record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_crossingPoint          = in.readShort();
         field_2_labelFrequency         = in.readShort();
index f5fd1d9530d2ac056f2eaaf2dc3fd5aa411667a8..490cdd45551d5a5c140ce3116405c277236800c8 100644 (file)
@@ -44,25 +44,7 @@ public final class ChartFormatRecord extends Record {
     {
     }
 
-    /**
-     * Constructs a ChartFormatRecord record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ChartFormatRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A CHARTFORMAT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field1_x_position = in.readInt();
         field2_y_position = in.readInt();
index a4c6a5f4f4cf9a697de7a357485265b845a8c1ff..9ce93946aed9840e7c7cb91ef12ee7d42545722d 100644 (file)
@@ -45,32 +45,7 @@ public class ChartRecord
 
     }
 
-    /**
-     * Constructs a Chart record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ChartRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Chart record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_x                      = in.readInt();
         field_2_y                      = in.readInt();
index a9ec81e97a925dccd0bc8921d3b6083a94268795..fd22d5229b4a4734f6ab51aa43e15364a14790aa 100644 (file)
@@ -59,17 +59,6 @@ public class ChartTitleFormatRecord extends Record {
        }
 
        public ChartTitleFormatRecord(RecordInputStream in) {
-               super(in);
-       }
-
-       protected void validateSid(short id) {
-               if (id != sid)
-        {
-            throw new RecordFormatException("NOT A CHARTTITLEFORMAT RECORD");
-        }
-       }
-
-       protected void fillFields(RecordInputStream in) {
                m_recs = in.readUShort();
                int idx;
                CTFormat ctf;
index b2aa41039f980daef518bca006b957dd943fa696..4675fe473041d67bc3f95a510f502976cbe74cec 100644 (file)
@@ -47,25 +47,7 @@ public class CodepageRecord
     {
     }
 
-    /**
-     * Constructs a CodepageRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
-
     public CodepageRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A CODEPAGE RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_codepage = in.readShort();
     }
index bd04a636f2dbe503466f4afc197dc47d1c169b0a..38d05181c9f18ff37878995acad835157a47928d 100644 (file)
@@ -52,17 +52,7 @@ public final class ColumnInfoRecord extends Record {
         field_6_reserved = 2; // seems to be the most common value
     }
 
-    /**
-     * Constructs a ColumnInfo record and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ColumnInfoRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_first_col = in.readUShort();
         field_2_last_col  = in.readUShort();
@@ -83,14 +73,6 @@ public final class ColumnInfoRecord extends Record {
         }
     }
 
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A COLINFO RECORD!!");
-        }
-    }
-
     /**
      * set the first column this record defines formatting info for
      * @param fc - the first column index (0-based)
index dd29f0d65c807d9acaf86020f0a3d5d5d4a1b090..ed2372d82ffa867610e4460b027f43819edac8a8 100644 (file)
@@ -82,31 +82,7 @@ public final class CommonObjectDataSubRecord extends SubRecord {
 
     }
 
-    /**
-     * Constructs a CommonObjectData record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public CommonObjectDataSubRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a CommonObjectData record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_objectType             = in.readShort();
         field_2_objectId               = in.readShort();
index 46d15a3c1e53c77e7c16ecce66fa15c725536ef9..435c45471231430f2f90d71fb395a6b119472c4f 100644 (file)
@@ -45,17 +45,6 @@ public class ContinueRecord
     {
     }
 
-    /**
-     * Main constructor -- kinda dummy because we don't validate or fill fields
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
-    public ContinueRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
     /**
      * USE ONLY within "processContinue"
      */
@@ -97,20 +86,6 @@ public class ContinueRecord
         return field_1_data;
     }
 
-    /**
-     * Make sure we have a good id
-     *
-     * @param id the alleged id
-     */
-
-    protected void validateSid(short id)
-    {
-        if (id != ContinueRecord.sid)
-        {
-            throw new RecordFormatException("Not a Continue Record");
-        }
-    }
-
     /**
      * Debugging toString
      *
@@ -139,7 +114,7 @@ public class ContinueRecord
      * @param in the RecordInputstream to read the record from
      */
 
-    protected void fillFields(RecordInputStream in)
+    public ContinueRecord(RecordInputStream in)
     {
       field_1_data = in.readRemainder();
     }
index ca1af5531ca40452ef73f56651c092361bfe51a5..7d739a9b652c701d3650308517ba700d34ca8a06 100644 (file)
@@ -44,25 +44,7 @@ public class CountryRecord
     {
     }
 
-    /**
-     * Constructs a CountryRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
-
     public CountryRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A Country RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_default_country = in.readShort();
         field_2_current_country = in.readShort();
index 1aef298e1f6514ecd8bee7713714459f1e44a15e..4085027dd4b981ca4c6fc46e13e7af5b03647e8d 100644 (file)
@@ -38,24 +38,7 @@ public final class DBCellRecord extends Record {
         field_2_cell_offsets = new short[0];
     }
 
-    /**
-     * Constructs a DBCellRecord and sets its fields appropriately
-     * @param in the RecordInputstream to read the record from
-     */
     public DBCellRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A valid DBCell RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_row_offset   = in.readUShort();
         int size = in.remaining();        
index ea898c2f2306bf84d2ff93d0788f21c4429b210b..9d405b3dda59c9e4ecbc1d53573157627c6cedc3 100644 (file)
@@ -40,25 +40,7 @@ public class DSFRecord
     {
     }
 
-    /**
-     * Constructs a DBCellRecord and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DSFRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A DSF RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_dsf = in.readShort();
     }
index 011c0a435557efab9fc1ae33deb473f4ab3556a8..9235ef2c6cfb02dd4e4c30e453c0d3cdeca215e8 100644 (file)
@@ -48,26 +48,7 @@ public class DVALRecord extends Record
         field_5_dv_no = 0x00000000;
     }
 
-    /**
-     * Constructs a DVAL record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
-    public DVALRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A valid DVAL RECORD");
-        }
-    }
-
-       protected void fillFields(RecordInputStream in)
+       public DVALRecord(RecordInputStream in)
        {
                this.field_1_options = in.readShort();
                this.field_2_horiz_pos = in.readInt();
index 852e4dfae7cc08160b2f687f7b0367fdfc699ff8..da34e5425a9abe0410627bd4c52eb6b5cfd79aea 100644 (file)
@@ -75,15 +75,6 @@ public final class DVRecord extends Record {
        private static final BitField opt_show_error_on_invalid_value  = new BitField(0x00080000);
        private static final BitField opt_condition_operator           = new BitField(0x00700000);
 
-       /**
-        * Constructs a DV record and sets its fields appropriately.
-        *
-        * @param in the RecordInputstream to read the record from
-        */
-       public DVRecord(RecordInputStream in) {
-               super(in);
-       }
-
        public DVRecord(int validationType, int operator, int errorStyle, boolean emptyCellAllowed,
                        boolean suppressDropDownArrow, boolean isExplicitList,
                        boolean showPromptBox, String promptTitle, String promptText, 
@@ -110,13 +101,7 @@ public final class DVRecord extends Record {
                _regions = regions;
        }
 
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT a valid DV RECORD");
-               }
-       }
-
-       protected void fillFields(RecordInputStream in) {
+       public DVRecord(RecordInputStream in) {
                
           _option_flags = in.readInt();
           
index 476b13291b682e557bb5a165871c6eadbbe17940..a6c5040150835762deae37ff94eb74b2e2267875 100644 (file)
@@ -45,32 +45,7 @@ public final class DatRecord extends Record {
 
     }
 
-    /**
-     * Constructs a Dat record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DatRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Dat record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_options                = in.readShort();
     }
index f86834daaed8ee2377cda9f6b3206dbdce9b59b3..7d37704eb770defa62237b3f76e3cfa36d22e914 100644 (file)
@@ -45,32 +45,7 @@ public final class DataFormatRecord extends Record {
 
     }
 
-    /**
-     * Constructs a DataFormat record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DataFormatRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a DataFormat record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_pointNumber            = in.readShort();
         field_2_seriesIndex            = in.readShort();
index 3a692fd6890c49709adc1efb5966442cef3a1839..dd023fa803d2e95ee4f3919386f424f0520bd3d2 100644 (file)
@@ -40,25 +40,7 @@ public class DateWindow1904Record
     {
     }
 
-    /**
-     * Constructs a DateWindow1904 record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DateWindow1904Record(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A 1904 RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_window = in.readShort();
     }
index ff6270be017e5f122b5a04ad6850d72290b73422..8ceb88edfa558ee586d7b2c94fc0674644732300 100644 (file)
@@ -36,25 +36,7 @@ public final class DefaultColWidthRecord extends Record {
     {
     }
 
-    /**
-     * Constructs a DefaultColumnWidth record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DefaultColWidthRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A DefaultColWidth RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_col_width = in.readUShort();
     }
index 771d7c4cf797c49f4ad6fa05bf694cb33d4c384b..49120e75a7d64d8f8b86c4b14091200fb0386951 100644 (file)
@@ -45,32 +45,7 @@ public class DefaultDataLabelTextPropertiesRecord
 
     }
 
-    /**
-     * Constructs a DefaultDataLabelTextProperties record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DefaultDataLabelTextPropertiesRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a DefaultDataLabelTextProperties record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_categoryDataType       = in.readShort();
     }
index 8f049c1602dee4606c0b2e25764641dd93193d82..ae48ea4b00179bc66e706009d996d86bdca608fd 100644 (file)
@@ -42,25 +42,7 @@ public class DefaultRowHeightRecord
     {
     }
 
-    /**
-     * Constructs a DefaultRowHeight record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DefaultRowHeightRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A DefaultRowHeight RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_option_flags = in.readShort();
         field_2_row_height   = in.readShort();
index b7afe7486b77d7c0842c6c696ac079a364533b5d..d02eb2cef7169f37eb9474c6574f639f6c7fa27d 100644 (file)
@@ -44,25 +44,7 @@ public class DeltaRecord
     {
     }
 
-    /**
-     * Constructs a Delta record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DeltaRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A DELTA RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_max_change = in.readDouble();
     }
index feecd68653d9696451c42444b4bd3dc9bdcda06e..fa7477bc79c5dc2b6c6c258093ee298bdf2a6565 100644 (file)
@@ -45,25 +45,7 @@ public class DimensionsRecord
     {
     }
 
-    /**
-     * Constructs a Dimensions record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public DimensionsRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A valid DIMENSIONS RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_first_row = in.readInt();
         field_2_last_row  = in.readInt();
index 009bc31bce892e7911f4987bbb68f7db750b2a12..c8c54123c994f6ec79ef89cb7c61b852abd182af 100644 (file)
@@ -30,24 +30,6 @@ public class DrawingRecord extends Record
     }
 
     public DrawingRecord( RecordInputStream in )
-    {
-        super( in );
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a MSODRAWING record");
-        }
-    }
-
-    protected void fillFields( RecordInputStream in )
     {
       recordData = in.readRemainder();
     }
index d658c6d83dabf08ed0c5eb5b8c4e8a79e1d677ea..f7495179b44be6a035aab8ee48998f74216a00ec 100644 (file)
@@ -40,24 +40,9 @@ public final class EOFRecord extends Record {
     }
 
     /**
-     * Constructs a EOFRecord record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
+     * @param in unused (since this record has no data)
      */
-
     public EOFRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An EOF RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
     }
 
index 0e59282f5e22789ee809382c36aaedd081aa15ef..fb6bce5d3a03310b7e0f9aa47a46313bea1640db 100644 (file)
@@ -54,35 +54,12 @@ public class EmbeddedObjectRefSubRecord
         field_5_ole_classname = "";
     }
 
-    /**
-     * Constructs an EmbeddedObjectRef record and sets its fields appropriately.
-     *
-     * @param in the record input stream.
-     */
-    public EmbeddedObjectRefSubRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a EmbeddedObjectRef record");
-        }
-    }
-
     public short getSid()
     {
         return sid;
     }
 
-    protected void fillFields(RecordInputStream in)
+    public EmbeddedObjectRefSubRecord(RecordInputStream in)
     {
         field_1_stream_id_offset       = in.readShort();
         field_2_unknown                = in.readShortArray();
index 0c15e723f7298581c8b841525bf5fd6519fae66e..6f91dc3a2472b3847419841f3434391e28f92400 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -30,9 +28,7 @@ import org.apache.poi.util.LittleEndian;
  * @author Glen Stampoultzis (glens at apache.org)
  */
 
-public class EndRecord
-    extends Record
-{
+public final class EndRecord extends Record {
     public static final short sid = 0x1034;
 
     public EndRecord()
@@ -40,25 +36,9 @@ public class EndRecord
     }
 
     /**
-     * Constructs a EndRecord record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
+     * @param in unused (since this record has no data)
      */
-
     public EndRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An END RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
     }
 
@@ -73,9 +53,8 @@ public class EndRecord
 
     public int serialize(int offset, byte [] data)
     {
-        LittleEndian.putShort(data, 0 + offset, sid);
-        LittleEndian.putShort(data, 2 + offset,
-                              (( short ) 0));   // no record info
+        LittleEndian.putUShort(data, 0 + offset, sid);
+        LittleEndian.putUShort(data, 2 + offset, 0);   // no record info
         return getRecordSize();
     }
 
index 28478adbfaf057dff30ab35ee832144540df1ce1..e2d3808d84880e6fb6b291d9fa7cbc6fdc4972d9 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
    limitations under the License.
 ==================================================================== */
         
-
 package org.apache.poi.hssf.record;
 
-
-
-import org.apache.poi.util.*;
+import org.apache.poi.util.LittleEndian;
 
 /**
  * The end data record is used to denote the end of the subrecords.
@@ -30,10 +26,8 @@ import org.apache.poi.util.*;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class EndSubRecord
-    extends SubRecord
-{
-    public final static short      sid                             = 0x00;
+public final class EndSubRecord extends SubRecord {
+    public final static short      sid = 0x00;
 
 
     public EndSubRecord()
@@ -42,31 +36,9 @@ public class EndSubRecord
     }
 
     /**
-     * Constructs a End record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
+     * @param in unused (since this record has no data)
      */
-
     public EndSubRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a End record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
     }
 
index d62b59adbb33e9b4b3238f5a94a76310612f2527..bfeb44751b0280b59af85bc9729f66ce53df0474 100644 (file)
 
 package org.apache.poi.hssf.record;
 
-import org.apache.poi.ddf.*;
-import org.apache.poi.hssf.usermodel.*;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+
+import org.apache.poi.ddf.DefaultEscherRecordFactory;
+import org.apache.poi.ddf.EscherBoolProperty;
+import org.apache.poi.ddf.EscherClientAnchorRecord;
+import org.apache.poi.ddf.EscherClientDataRecord;
+import org.apache.poi.ddf.EscherContainerRecord;
+import org.apache.poi.ddf.EscherDgRecord;
+import org.apache.poi.ddf.EscherDggRecord;
+import org.apache.poi.ddf.EscherOptRecord;
+import org.apache.poi.ddf.EscherProperties;
+import org.apache.poi.ddf.EscherRecord;
+import org.apache.poi.ddf.EscherRecordFactory;
+import org.apache.poi.ddf.EscherSerializationListener;
+import org.apache.poi.ddf.EscherSpRecord;
+import org.apache.poi.ddf.EscherSpgrRecord;
+import org.apache.poi.ddf.EscherTextboxRecord;
 import org.apache.poi.hssf.model.AbstractShape;
-import org.apache.poi.hssf.model.TextboxShape;
-import org.apache.poi.hssf.model.DrawingManager2;
-import org.apache.poi.hssf.model.ConvertAnchor;
 import org.apache.poi.hssf.model.CommentShape;
+import org.apache.poi.hssf.model.ConvertAnchor;
+import org.apache.poi.hssf.model.DrawingManager2;
+import org.apache.poi.hssf.model.TextboxShape;
+import org.apache.poi.hssf.usermodel.HSSFClientAnchor;
+import org.apache.poi.hssf.usermodel.HSSFPatriarch;
+import org.apache.poi.hssf.usermodel.HSSFShape;
+import org.apache.poi.hssf.usermodel.HSSFShapeContainer;
+import org.apache.poi.hssf.usermodel.HSSFShapeGroup;
+import org.apache.poi.hssf.usermodel.HSSFTextbox;
 import org.apache.poi.util.POILogFactory;
 import org.apache.poi.util.POILogger;
 
-import java.util.*;
-
 /**
  * This class is used to aggregate the MSODRAWING and OBJ record
  * combinations.  This is necessary due to the bizare way in which
@@ -282,16 +305,6 @@ public class EscherAggregate extends AbstractEscherHolderRecord
         return sid;
     }
 
-    /**
-     * Unused since this is an aggregate record.  Use createAggregate().
-     *
-     * @see #createAggregate
-     */
-    protected void fillFields( byte[] data, short size, int offset )
-    {
-        throw new IllegalStateException( "Should not reach here" );
-    }
-
     /**
      * Calculates the string representation of this record.  This is
      * simply a dump of all the records.
@@ -539,8 +552,7 @@ public class EscherAggregate extends AbstractEscherHolderRecord
        // The top level container ought to have
        //  the DgRecord and the container of one container
        //  per shape group (patriach overall first)
-       EscherContainerRecord topContainer =
-               (EscherContainerRecord)getEscherContainer();
+       EscherContainerRecord topContainer = getEscherContainer();
        if(topContainer == null) {
                return;
        }
index bd2bc365665c92c1590f472d2f7fa9b255dacc39..278bfd78166b4d947209ebd6292d0ed616c777a4 100644 (file)
@@ -51,17 +51,6 @@ public class ExtSSTInfoSubRecord
     }
 
     public ExtSSTInfoSubRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-
-        // do nothing
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_stream_pos        = in.readInt();
         field_2_bucket_sst_offset = in.readShort();
index cb31fb59b41329c29143100e77dd599bfebe4ba3..582668f331751ca73a088d81aaef38f16417dd56 100644 (file)
@@ -53,25 +53,7 @@ public class ExtSSTRecord
         field_2_sst_info = new ArrayList();
     }
 
-    /**
-     * Constructs a EOFRecord record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ExtSSTRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An EXTSST RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_2_sst_info           = new ArrayList();
         field_1_strings_per_bucket = in.readShort();
index ae983876711ef44ebb5d8efebcc5ca04769b6539..0bc50ca89e9a39934283a8d96eb44b097c1b4850 100644 (file)
@@ -195,25 +195,7 @@ public class ExtendedFormatRecord
     {
     }
 
-    /**
-     * Constructs an ExtendedFormat record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ExtendedFormatRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A EXTENDED FORMAT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_font_index           = in.readShort();
         field_2_format_index         = in.readShort();
index 64f86c3f124f0ca97407d2990107714fc0e1b17e..f6266cfa3511f1d2181685de68aa70b7f00336c8 100644 (file)
@@ -95,34 +95,13 @@ public class ExternSheetRecord extends Record {
                _list = new ArrayList();
        }
        
-       /**
-        * Constructs a Extern Sheet record and sets its fields appropriately.
-        * @param in the RecordInputstream to read the record from
-        */
-       
-       public ExternSheetRecord(RecordInputStream in) {
-               super(in);
-       }
-       
-       /**
-        * called by constructor, should throw runtime exception in the event of a
-        * record passed with a differing ID.
-        *
-        * @param id alleged id for this record
-        */
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT An ExternSheet RECORD");
-               }
-       }
-       
        /**
         * called by the constructor, should set class level fields.  Should throw
         * runtime exception for bad/icomplete data.
         *
         * @param in the RecordInputstream to read the record from
         */
-       protected void fillFields(RecordInputStream in) {
+       public ExternSheetRecord(RecordInputStream in) {
                _list              = new ArrayList();
                
                int nItems  = in.readShort();
index 45960e91af5f582960c79830db0bdaab938151ab..d2135c085ee1e3a307229a29b8b5c9d50d6630e1 100755 (executable)
@@ -47,11 +47,7 @@ public final class ExternalNameRecord extends Record {
        private short  field_2_index;
        private short  field_3_not_used;
        private String field_4_name;
-       private Ptg[]  field_5_name_definition; // TODO - junits for name definition field
-
-       public ExternalNameRecord(RecordInputStream in) {
-               super(in);
-       }
+       private Ptg[]  field_5_name_definition;
 
        /**
         * Convenience Function to determine if the name is a built-in name
@@ -90,19 +86,6 @@ public final class ExternalNameRecord extends Record {
                return field_4_name;
        }
 
-
-       /**
-        * called by constructor, should throw runtime exception in the event of a
-        * record passed with a differing ID.
-        *
-        * @param id alleged id for this record
-        */
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT A valid ExternalName RECORD");
-               }
-       }
-
        private int getDataSize(){
                int result = 3 * 2  // 3 short fields
                        + 2 + field_4_name.length(); // nameLen and name
@@ -129,13 +112,13 @@ public final class ExternalNameRecord extends Record {
                LittleEndian.putShort( data, 4 + offset, field_1_option_flag );
                LittleEndian.putShort( data, 6 + offset, field_2_index );
                LittleEndian.putShort( data, 8 + offset, field_3_not_used );
-               short nameLen = (short) field_4_name.length();
-               LittleEndian.putShort( data, 10 + offset, nameLen );
+               int nameLen = field_4_name.length();
+               LittleEndian.putUShort( data, 10 + offset, nameLen );
                StringUtil.putCompressedUnicode( field_4_name, data, 12 + offset );
                if(hasFormula()) {
-                       short defLen = (short) getNameDefinitionSize();
-                       LittleEndian.putShort( data, 12 + nameLen + offset, defLen );
-                       Ptg.serializePtgStack(toStack(field_5_name_definition), data, 14 + nameLen + offset );
+                       int defLen = getNameDefinitionSize();
+                       LittleEndian.putUShort( data, 12 + nameLen + offset, defLen );
+                       Ptg.serializePtgs(field_5_name_definition, data, 14 + nameLen + offset );
                }
                return dataSize + 4;
        }
@@ -154,7 +137,7 @@ public final class ExternalNameRecord extends Record {
        }
 
 
-       protected void fillFields(RecordInputStream in) {
+       public ExternalNameRecord(RecordInputStream in) {
                field_1_option_flag = in.readShort();
                field_2_index       = in.readShort();
                field_3_not_used    = in.readShort();
@@ -171,7 +154,7 @@ public final class ExternalNameRecord extends Record {
                        throw readFail("Ran out of record data trying to read formula.");
                }
                short formulaLen = in.readShort();
-               field_5_name_definition = toPtgArray(Ptg.createParsedExpressionTokens(formulaLen, in));
+               field_5_name_definition = Ptg.readTokens(formulaLen, in);
        }
        /*
         * Makes better error messages (while hasFormula() is not reliable) 
@@ -209,19 +192,6 @@ public final class ExternalNameRecord extends Record {
                return true;
        }
 
-       private static Ptg[] toPtgArray(Stack s) {
-               Ptg[] result = new Ptg[s.size()];
-               s.toArray(result);
-               return result;
-       }
-       private static Stack toStack(Ptg[] ptgs) {
-               Stack result = new Stack();
-               for (int i = 0; i < ptgs.length; i++) {
-                       result.push(ptgs[i]);
-               }
-               return result;
-       }
-
        public short getSid() {
                return sid;
        }
index a3f29c765487904022382a7f2bf3399f5583c079..3af87278e23abac5ec152304a3f0ce58b6436cb5 100644 (file)
@@ -40,25 +40,7 @@ public class FilePassRecord
     {
     }
 
-    /**
-     * Constructs a FILEPASS record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public FilePassRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A FILEPASS RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_encryptedpassword = in.readInt();
         
index e375af365720a87b64a015d8195638ae98cf688c..3631251995bf5dad76ec2b0efd3a309cbab45dc8 100644 (file)
@@ -36,24 +36,8 @@ public final class FileSharingRecord extends Record {
     private String            field_3_username_value;
 
     public FileSharingRecord() {}
-    
-
-    /**
-     * Constructs a FileSharing record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
 
     public FileSharingRecord(RecordInputStream in) {
-        super(in);
-    }
-
-    protected void validateSid(short id) {
-        if (id != sid) {
-            throw new RecordFormatException("NOT A FILESHARING RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in) {
         field_1_readonly = in.readShort();
         field_2_password = in.readShort();
         
index 41bb07d8f49cc8ad4275cb6b9f4a0650f40d3700..23de9135af495057f6b7d0f1a99be1d514029833 100644 (file)
@@ -46,25 +46,7 @@ public class FnGroupCountRecord
     {
     }
 
-    /**
-     * Constructs a FnGroupCount record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public FnGroupCountRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A FNGROUPCOUNT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_count = in.readShort();
     }
index a2a45e23c1a75380dd9e7feeae618d5fe45f0e80..645db716c0ed263b4cf6a45d6393d16bdee47f94 100644 (file)
@@ -46,32 +46,7 @@ public class FontBasisRecord
 
     }
 
-    /**
-     * Constructs a FontBasis record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public FontBasisRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a FontBasis record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_xBasis                 = in.readShort();
         field_2_yBasis                 = in.readShort();
index 12e2f66691175b38c94e02031f5b21d762263b76..b2ece531757e22a0e66935d679c352ece4d789e4 100644 (file)
@@ -42,32 +42,7 @@ public class FontIndexRecord
 
     }
 
-    /**
-     * Constructs a FontIndex record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public FontIndexRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a FontIndex record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_fontIndex              = in.readShort();
     }
index 11ba3aaa897d0ec0dd1fed71acebbbd0fcff2f31..e1d421977402492e4e9b8a73baed70b542d0aba6 100644 (file)
@@ -76,26 +76,7 @@ public class FontRecord
     {
     }
 
-    /**
-     * Constructs a Font record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public FontRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A FONT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_font_height         = in.readShort();
         field_2_attributes          = in.readShort();
index 6ccc082801aa3d8b0153e20736a4e3d3b4a7247c..8434792172d15829448bcb8f2001b69d42cbb771 100644 (file)
@@ -45,25 +45,7 @@ public class FooterRecord
     {
     }
 
-    /**
-     * Constructs a FooterRecord record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public FooterRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A FooterRECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         if (in.remaining() > 0)
         {
index d71c401f8891087780fe48cc832dc742744b6140..b8f4ea22ad44c52c4e6e158c405fbb65ec798d17 100644 (file)
@@ -46,25 +46,7 @@ public class FormatRecord
     {
     }
 
-    /**
-     * Constructs a Format record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public FormatRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A FORMAT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_index_code       = in.readShort();
         field_3_unicode_len      = in.readShort();
index e4b599a0e9eae2ca2f4c60d99226e79481d2d8c0..a6c97d46efd85ec4b98e93df859409de210d535e 100644 (file)
@@ -186,20 +186,7 @@ public final class FormulaRecord extends Record implements CellValueRecordInterf
                field_8_parsed_expr = Ptg.EMPTY_PTG_ARRAY;
        }
 
-       /**
-        * Constructs a Formula record and sets its fields appropriately.
-        * Note - id must be 0x06 (NOT 0x406 see MSKB #Q184647 for an
-        * "explanation of this bug in the documentation) or an exception
-        *  will be throw upon validation
-        *
-        * @param in the RecordInputstream to read the record from
-        */
-
        public FormulaRecord(RecordInputStream in) {
-               super(in);
-       }
-
-       protected void fillFields(RecordInputStream in) {
                field_1_row      = in.readUShort();
                field_2_column  = in.readShort();
                field_3_xf        = in.readShort();
@@ -356,18 +343,6 @@ public final class FormulaRecord extends Record implements CellValueRecordInterf
                field_8_parsed_expr = ptgs;
        }
 
-       /**
-        * called by constructor, should throw runtime exception in the event of a
-        * record passed with a differing ID.
-        *
-        * @param id alleged id for this record
-        */
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT A FORMULA RECORD");
-               }
-       }
-
        public short getSid() {
                return sid;
        }
index e4a2bf6c3a1ac813232ac9610e3ae68edf4a099b..4425b0817d6fd218afb786ba1acefc519ed85d69 100644 (file)
@@ -46,32 +46,7 @@ public final class FrameRecord extends Record {
 
     }
 
-    /**
-     * Constructs a Frame record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public FrameRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Frame record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_borderType             = in.readShort();
         field_2_options                = in.readShort();
index 5f5896d5a1fde8a8fb3d2c647167a5a7d17129a5..ad652ba8245c066924813418fa6d37a5547f4bbd 100644 (file)
@@ -44,25 +44,7 @@ public class GridsetRecord
     {
     }
 
-    /**
-     * Constructs a GridSet record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public GridsetRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A Gridset RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_gridset_flag = in.readShort();
     }
index cba7544e93263ea0a2d677f483ee440dd5d8add6..0618ea6f4fc12e4abdca961fe261d5edfbb354b2 100644 (file)
@@ -41,32 +41,7 @@ public class GroupMarkerSubRecord
 
     }
 
-    /**
-     * Constructs a group marker record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public GroupMarkerSubRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Group Marker record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
 //        int pos = 0;
         reserved = in.readRemainder();
index 2dc5c4a6cca56f1aa467246c778c0f8ec8967364..0073066ec1e98ea674c9c96170868485c6db642e 100644 (file)
@@ -43,25 +43,7 @@ public class GutsRecord
     {
     }
 
-    /**
-     * Constructs a Guts record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public GutsRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A Guts RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_left_row_gutter = in.readShort();
         field_2_top_col_gutter  = in.readShort();
index 5232e17dea577d72905e1a811371f8e5528770e9..261431e7167df837b966089c0982c70853ab8025 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
 import org.apache.poi.util.LittleEndian;
 
 /**
- * Title:        HCenter record<P>
+ * Title:        HCenter record (0x0083)<P>
  * Description:  whether to center between horizontal margins<P>
  * REFERENCE:  PG 320 Microsoft Excel 97 Developer's Kit (ISBN: 1-57231-498-2)<P>
  * @author Andrew C. Oliver (acoliver at apache dot org)
  * @author Jason Height (jheight at chariot dot net dot au)
  * @version 2.0-pre
  */
-
-public class HCenterRecord
-    extends Record
-{
-    public final static short sid = 0x83;
+public final class HCenterRecord extends Record {
+    public final static short sid = 0x0083;
     private short             field_1_hcenter;
 
     public HCenterRecord()
     {
     }
 
-    /**
-     * Constructs an HCenter record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public HCenterRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A HCenter RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_hcenter = in.readShort();
     }
@@ -104,8 +81,8 @@ public class HCenterRecord
     public int serialize(int offset, byte [] data)
     {
         LittleEndian.putShort(data, 0 + offset, sid);
-        LittleEndian.putShort(data, 2 + offset, ( short ) 0x2);
-        LittleEndian.putShort(data, 4 + offset, ( short ) field_1_hcenter);
+        LittleEndian.putUShort(data, 2 + offset, 0x2);
+        LittleEndian.putUShort(data, 4 + offset, field_1_hcenter);
         return getRecordSize();
     }
 
index e8190364a8f67a7f57d1288c89c98dbd66431f40..61ff851306a14279aca8ffe570c5fcd6b976eae6 100644 (file)
@@ -45,25 +45,7 @@ public class HeaderRecord
     {
     }
 
-    /**
-     * Constructs an Header record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public HeaderRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A HEADERRECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         if (in.remaining() > 0)
         {
index 1496720d4f142fcc343ad5a09bf8f166359697f0..496002618ace17a9f2e770a83833375c1013ab6c 100644 (file)
@@ -42,25 +42,7 @@ public class HideObjRecord
     {
     }
 
-    /**
-     * Constructs an HideObj record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public HideObjRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A HIDEOBJ RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_hide_obj = in.readShort();
     }
index a6846300cb9de30167f89a8c1568de30bb3e490e..63e757f8df2e97da021c01a74386b19605a6dde7 100644 (file)
@@ -44,13 +44,6 @@ public final class HorizontalPageBreakRecord extends PageBreakRecord {
                super(in);
        }
 
-       protected void validateSid(short id) {
-               if (id != getSid()) {
-                       throw new RecordFormatException(
-                                       "NOT A HorizontalPageBreak or VerticalPageBreak RECORD!! " + id);
-               }
-       }
-
        public short getSid() {
                return sid;
        }
index e28cf5bf864b36e4204cd43f289ddbd096682e69..6fc5b97163d5e69debbb7f0f29d7cb93333e80da 100644 (file)
@@ -130,16 +130,6 @@ public class HyperlinkRecord extends Record {
 
     }
 
-    /**
-     * Read hyperlink from input stream
-     *
-     * @param in the stream to read from
-     */
-    public HyperlinkRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
     /**
      * Return the column of the first cell that contains the hyperlink
      *
@@ -315,7 +305,7 @@ public class HyperlinkRecord extends Record {
     /**
      * @param in the RecordInputstream to read the record from
      */
-    protected void fillFields(RecordInputStream in)
+    public HyperlinkRecord(RecordInputStream in)
     {
         try {
             rwFirst = in.readShort();
@@ -372,14 +362,6 @@ public class HyperlinkRecord extends Record {
         return HyperlinkRecord.sid;
     }
 
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A HYPERLINKRECORD!");
-        }
-    }
-
     public int serialize(int offset, byte[] data)
     {
         int pos = offset;
index 957a047eaaa5d94492d18a76d35a17c4e70b2531..c7c3cfbb5afea00e19c75a9cc11eefb7164fa747 100644 (file)
@@ -48,25 +48,7 @@ public class IndexRecord
     {
     }
 
-    /**
-     * Constructs an Index record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public IndexRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An Index RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_5_dbcells       =
             new IntList(DBCELL_CAPACITY);   // initial capacity of 30
index bb92c974cae672d4e0c6ecf9dc4731210c300ac2..2a9de8e5636e3b143c4c3e40ce27b35d8688790a 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
 import org.apache.poi.util.LittleEndian;
 
 /**
- * Title: Interface End Record<P>
+ * Title: Interface End Record (0x00E2)<P>
  * Description: Shows where the Interface Records end (MMS)
  *  (has no fields)<P>
  * REFERENCE:  PG 324 Microsoft Excel 97 Developer's Kit (ISBN: 1-57231-498-2)<P>
  * @author Andrew C. Oliver (acoliver at apache dot org)
  * @version 2.0-pre
  */
-
-public class InterfaceEndRecord
-    extends Record
-{
-    public final static short sid = 0xe2;
+public final class InterfaceEndRecord extends Record {
+    public final static short sid = 0x00E2;
 
     public InterfaceEndRecord()
     {
     }
 
     /**
-     * Constructs an InterfaceEnd record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
+     * @param in unused (since this record has no data)
      */
-
     public InterfaceEndRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A INTERFACEEND RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
     }
 
index 637e2c679431c4f6ad5059e85575c34ad240cfca..617e95855fcbee3220c3af50e14cac827c16a6e3 100644 (file)
@@ -45,25 +45,7 @@ public class InterfaceHdrRecord
     {
     }
 
-    /**
-     * Constructs an Codepage record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public InterfaceHdrRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A INTERFACEHDR RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_codepage = in.readShort();
     }
index 16ab1d0741fad97a77e42ceebf1f3e72d2aa7b19..400ee0a8c5eb40bf623b1f7d3b5097cda6680b7f 100644 (file)
@@ -43,25 +43,7 @@ public class IterationRecord
     {
     }
 
-    /**
-     * Constructs an Iteration record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public IterationRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An ITERATION RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_iteration = in.readShort();
     }
index 844d86a0c6c68469ba21d8880e99687d5225d516..f4bbfb85dadd195323e71dd1b2db9244b0dfae85 100644 (file)
@@ -44,33 +44,9 @@ public final class LabelRecord extends Record implements CellValueRecordInterfac
     }
 
     /**
-     * Constructs an Label record and sets its fields appropriately.
-     *
      * @param in the RecordInputstream to read the record from
      */
     public LabelRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    /**
-     * called by constructor, should throw runtime exception in the event of a
-     * record passed with a differing ID.
-     *
-     * @param id alleged id for this record
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a valid LabelRecord");
-        }
-    }
-
-    /**
-     * @param in the RecordInputstream to read the record from
-     */
-    protected void fillFields(RecordInputStream in)
     {
         field_1_row          = in.readUShort();
         field_2_column       = in.readShort();
index 92b3436d3657d3fa60e205b75f9267f86fc2985f..f2eae83debe95bd1032de991a0a549ebc0c4f245 100644 (file)
@@ -40,25 +40,7 @@ public final class LabelSSTRecord extends Record implements CellValueRecordInter
     {
     }
 
-    /**
-     * Constructs an LabelSST record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
     public LabelSSTRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A valid LabelSST RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_row       = in.readUShort();
         field_2_column    = in.readUShort();
index 9e5883c5a05aa95a08dcb03fdec28dd03f1d45de..92da85ceaa567ddc28d4327ed20deceade1af757 100644 (file)
@@ -31,28 +31,7 @@ public class LeftMarginRecord extends Record implements Margin
 
     public LeftMarginRecord()    {    }
 
-    /**
-     * Constructs a LeftMargin record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
     public LeftMarginRecord(RecordInputStream in)
-    {        super(in);    }
-
-    /**
-     * Checks the sid matches the expected side for this record     
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid( short id )
-    {
-        if ( id != sid )
-        {
-            throw new RecordFormatException( "Not a LeftMargin record" );
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_margin = in.readDouble();
     }
@@ -106,4 +85,4 @@ public class LeftMarginRecord extends Record implements Margin
         rec.field_1_margin = this.field_1_margin;
         return rec;
     }
-}  // END OF CLASS
+}  // END OF CLAS
\ No newline at end of file
index e9c529ea3c49d4deda8b869d2c5240ba729202a8..08ff6513fc8a69731a24766e4e7de391f0802e94 100644 (file)
@@ -62,32 +62,7 @@ public final class LegendRecord extends Record {
 
     }
 
-    /**
-     * Constructs a Legend record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public LegendRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Legend record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_xAxisUpperLeft         = in.readInt();
         field_2_yAxisUpperLeft         = in.readInt();
index 8caa8e4a4b0b053da13c7f23fb54fc959a555f44..c3f0197f389b5aca47f862ed668cef8958d10441 100644 (file)
@@ -61,32 +61,7 @@ public final class LineFormatRecord extends Record {
 
     }
 
-    /**
-     * Constructs a LineFormat record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public LineFormatRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a LineFormat record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_lineColor              = in.readInt();
         field_2_linePattern            = in.readShort();
index be5d1fd8416af6d5af3facca72b6a7627c3f2f84..9e82adec6eea16cbcf9556c7c0ce133747fbf440 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +29,7 @@ import java.util.Iterator;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class LinkedDataFormulaField
-        implements CustomField
-{
+public final class LinkedDataFormulaField implements CustomField {
     Stack formulaTokens = new Stack();
 
     public int getSize()
index 093c1a86cbf48a0f65be6203c918e010944336a3..673fc85f4be25481b8c2e5d1e68479b085fa2cec 100644 (file)
@@ -54,32 +54,7 @@ public final class LinkedDataRecord extends Record {
 
     }
 
-    /**
-     * Constructs a LinkedData record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public LinkedDataRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a LinkedData record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_linkType               = in.readByte();
         field_2_referenceType          = in.readByte();
@@ -155,7 +130,7 @@ public final class LinkedDataRecord extends Record {
         rec.field_2_referenceType = field_2_referenceType;
         rec.field_3_options = field_3_options;
         rec.field_4_indexNumberFmtRecord = field_4_indexNumberFmtRecord;
-        rec.field_5_formulaOfLink = ((LinkedDataFormulaField)field_5_formulaOfLink.clone());;
+        rec.field_5_formulaOfLink = ((LinkedDataFormulaField)field_5_formulaOfLink.clone());
         return rec;
     }
 
index c38e9df2449c1089a4d70071bd830e496d68a360..c11e2d510d16bb4ae2e3be27f995358802e7bacf 100644 (file)
@@ -41,25 +41,7 @@ public class MMSRecord
     {
     }
 
-    /**
-     * Constructs a MMS record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public MMSRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A MMS RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_addMenuCount = in.readByte();
         field_2_delMenuCount = in.readByte();
index 525749dffa02b35ad3cfe4e7cb06766e5e63fad6..7683c6b863dde0eac2472fd862c3087f63badd37 100644 (file)
@@ -56,9 +56,6 @@ public final class MergeCellsRecord extends Record {
        _startIndex = 0;
        _regions = cras;
     }
-    protected void fillFields(RecordInputStream in) {
-       throw new RuntimeException("obsolete");
-    }
     /**
      * get the number of merged areas.  If this drops down to 0 you should just go
      * ahead and delete the record.
@@ -119,13 +116,6 @@ public final class MergeCellsRecord extends Record {
         return retval.toString();
     }
 
-    protected void validateSid(short id) {
-        if (id != sid) {
-            throw new RecordFormatException("NOT A MERGEDCELLS RECORD!! "
-                                            + id);
-        }
-    }
-
     public Object clone() {
        int nRegions = _numberOfRegions;
        CellRangeAddress[] clonedRegions = new CellRangeAddress[nRegions];
index dffdb2fe80673f010b40c648a8efe47c1aa9bae3..8d461fe975bdaf7639d3a721f51fe87b2fdc09da 100644 (file)
@@ -52,17 +52,6 @@ public class MulBlankRecord
     {
     }
 
-    /**
-     * Constructs a MulBlank record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
-    public MulBlankRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
     /**
      * get the row number of the cells this represents
      *
@@ -119,7 +108,7 @@ public class MulBlankRecord
     /**
      * @param in the RecordInputstream to read the record from
      */
-    protected void fillFields(RecordInputStream in)
+    public MulBlankRecord(RecordInputStream in)
     {
         //field_1_row       = LittleEndian.getShort(data, 0 + offset);
         field_1_row       = in.readUShort();
@@ -159,21 +148,6 @@ public class MulBlankRecord
         return buffer.toString();
     }
 
-    /**
-     * called by constructor, should throw runtime exception in the event of a
-     * record passed with a differing ID.
-     *
-     * @param id alleged id for this record
-     */
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a MulBlankRecord!");
-        }
-    }
-
     public short getSid()
     {
         return sid;
index c301d52a8afa3549d9566c9bf0b25ccc66a0ce85..0419f4921dcdff15f7d8dd595a546b1d76b9d86b 100644 (file)
@@ -37,15 +37,6 @@ public final class MulRKRecord extends Record {
        private RkRec[] field_3_rks;
        private short   field_4_last_col;
 
-       /**
-        * Constructs a MulRK record and sets its fields appropriately.
-        *
-        * @param in the RecordInputstream to read the record from
-        */
-       public MulRKRecord(RecordInputStream in) {
-               super(in);
-       }
-
        public int getRow() {
                return field_1_row;
        }
@@ -93,7 +84,7 @@ public final class MulRKRecord extends Record {
        /**
         * @param in the RecordInputstream to read the record from
         */
-       protected void fillFields(RecordInputStream in) {
+       public MulRKRecord(RecordInputStream in) {
                field_1_row = in.readUShort();
                field_2_first_col = in.readShort();
                field_3_rks = RkRec.parseRKs(in);
@@ -117,21 +108,6 @@ public final class MulRKRecord extends Record {
                return buffer.toString();
        }
 
-       /**
-        * called by constructor, should throw runtime exception in the event of a
-        * record passed with a differing ID.
-        *
-        * @param id alleged id for this record
-        */
-
-       protected void validateSid(short id)
-       {
-               if (id != sid)
-               {
-                       throw new RecordFormatException("Not a MulRKRecord!");
-               }
-       }
-
        public short getSid()
        {
                return sid;
index 29834b9d1622dba33bc44e07094ef6df3e43cc76..512e784c0d2c2b27d546d26e4afc8c153b133e89 100644 (file)
@@ -107,15 +107,6 @@ public final class NameRecord extends Record {
                field_17_status_bar_text = "";
        }
 
-       /**
-        * Constructs a Name record and sets its fields appropriately.
-        *
-        * @param in the RecordInputstream to read the record from
-        */
-       public NameRecord(RecordInputStream in) {
-               super(in);
-       }
-
        /**
         * Constructor to create a built-in named region
         * @param builtin Built-in byte representation for the name record, use the public constants
@@ -340,18 +331,6 @@ public final class NameRecord extends Record {
                return field_17_status_bar_text;
        }
 
-       /**
-        * called by constructor, should throw runtime exception in the event of a
-        * record passed with a differing ID.
-        *
-        * @param id alleged id for this record
-        */
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT A valid Name RECORD");
-               }
-       }
-
 
        /**
         * called by the class that is responsible for writing this sucker.
@@ -551,7 +530,7 @@ public final class NameRecord extends Record {
         *
         * @param in the RecordInputstream to read the record from
         */
-       protected void fillFields(RecordInputStream in) {
+       public NameRecord(RecordInputStream in) {
                field_1_option_flag                 = in.readShort();
                field_2_keyboard_shortcut           = in.readByte();
                int field_3_length_name_text        = in.readByte();
index a9e83806f85d24ecba2319c8db42f0ad7290ba9d..2f506f115d7212061cde4e53058ebdab5384950e 100644 (file)
@@ -53,18 +53,6 @@ public class NoteRecord extends Record {
         field_3_flags = 0;
     }
 
-    /**
-     * Constructs a <code>NoteRecord</code> and fills its fields
-     * from the supplied <code>RecordInputStream</code>.
-     *
-     * @param in the stream to read from
-     */
-    public NoteRecord(RecordInputStream in)
-    {
-        super(in);
-
-    }
-
     /**
      * @return id of this record.
      */
@@ -73,23 +61,10 @@ public class NoteRecord extends Record {
         return sid;
     }
 
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a NoteRecord record");
-        }
-    }
-
     /**
      * Read the record data from the supplied <code>RecordInputStream</code>
      */
-    protected void fillFields(RecordInputStream in)
+    public NoteRecord(RecordInputStream in)
     {
         field_1_row = in.readShort();
         field_2_col = in.readShort();
index c99f29be354942ab2008b927b589fcd8b15932a3..4b96a0ee4524e189c5ede5b68b5f67d39bc379af 100644 (file)
@@ -45,33 +45,10 @@ public class NoteStructureSubRecord
         reserved = new byte[22];
     }
 
-    /**
-     * Constructs a NoteStructureSubRecord and sets its fields appropriately.
-     *
-     */
-    public NoteStructureSubRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Note Structure record");
-        }
-    }
-
     /**
      * Read the record data from the supplied <code>RecordInputStream</code>
      */
-    protected void fillFields(RecordInputStream in)
+    public NoteStructureSubRecord(RecordInputStream in)
     {
         //just grab the raw data
         reserved = in.readRemainder();
index 15f9c367d837c24a562fb6afffe9346abbaca4f7..00307e141f902d83122a42d026ab176140326d42 100644 (file)
@@ -42,32 +42,7 @@ public class NumberFormatIndexRecord
 
     }
 
-    /**
-     * Constructs a NumberFormatIndex record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public NumberFormatIndexRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a NumberFormatIndex record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_formatIndex            = in.readShort();
     }
index f3d906155c164c12b12f602fb74340ba1d33257b..e4283f0eac722acbde6fcb258e592a8cd6b9d6a9 100644 (file)
@@ -41,19 +41,9 @@ public final class NumberRecord extends Record implements CellValueRecordInterfa
     }
 
     /**
-     * Constructs a Number record and sets its fields appropriately.
-     *
      * @param in the RecordInputstream to read the record from
      */
     public NumberRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    /**
-     * @param in the RecordInputstream to read the record from
-     */
-    protected void fillFields(RecordInputStream in)
     {
         field_1_row   = in.readUShort();
         field_2_col   = in.readUShort();
@@ -157,21 +147,6 @@ public final class NumberRecord extends Record implements CellValueRecordInterfa
         return 18;
     }
 
-    /**
-     * called by constructor, should throw runtime exception in the event of a
-     * record passed with a differing ID.
-     *
-     * @param id alleged id for this record
-     */
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A Number RECORD");
-        }
-    }
-
     public short getSid()
     {
         return sid;
index 0e424eed2f3d5e101cc2b8be940fd255a33bb2b7..03ea7d1f5581d0c9e6ef5c35e214d1bd7a6e98ea 100644 (file)
@@ -43,31 +43,7 @@ public final class ObjRecord extends Record {
         // TODO - ensure 2 sub-records (ftCmo  15h, and ftEnd  00h) are always created
     }
 
-    /**
-     * Constructs a OBJ record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ObjRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not an OBJ record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
        // TODO - problems with OBJ sub-records stream
        // MS spec says first sub-records is always CommonObjectDataSubRecord, and last is 
index 0d64ee3d6bb113387ddcbb3f5a14d46b0e638068..7a86fe2ec44a0cf037b949a27d1026782631d8f0 100644 (file)
@@ -49,32 +49,7 @@ public class ObjectLinkRecord
 
     }
 
-    /**
-     * Constructs a ObjectLink record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ObjectLinkRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a ObjectLink record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_anchorId               = in.readShort();
         field_2_link1                  = in.readShort();
index 64d50cb3399db647a3bc0be728412b916d3686eb..cdbb56d8731900e234cd34577b03543d9ead194a 100644 (file)
@@ -41,25 +41,7 @@ public class ObjectProtectRecord
     {
     }
 
-    /**
-     * Constructs a Protect record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ObjectProtectRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT AN OBJECTPROTECT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_protect = in.readShort();
     }
index f11e3395cdaf27c722258ae4286436bd39b3a552..654439f31609cfbe5a6e9e97ddbff265ff96dcc2 100644 (file)
@@ -83,11 +83,7 @@ public abstract class PageBreakRecord extends Record {
         _breakMap = new HashMap();
     }
 
-    protected PageBreakRecord(RecordInputStream in) {
-        super(in);
-    }
-
-    protected void fillFields(RecordInputStream in)
+    public PageBreakRecord(RecordInputStream in)
     {
         int nBreaks = in.readShort();
         _breaks = new ArrayList(nBreaks + 2);
index d626baa2d3897fcf6a11b8052a31783ebc8eb5f1..0cf78563619d9193bd47fe833b46191956daeba4 100644 (file)
@@ -48,25 +48,7 @@ public class PaletteRecord
       createDefaultPalette();
     }
 
-    /**
-     * Constructs a PaletteRecord record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PaletteRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An Palette RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
        field_1_numcolors = in.readShort();
        field_2_colors    = new ArrayList(field_1_numcolors);
index f02e4116555bf9253c9e769f47b51d2c09e1da05..f02b0795b916a0db26e6df7311f43daef13cfe26 100644 (file)
@@ -49,32 +49,7 @@ public final class PaneRecord extends Record {
 
     }
 
-    /**
-     * Constructs a Pane record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PaneRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Pane record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_x                      = in.readShort();
         field_2_y                      = in.readShort();
index 67f210da5b14209b5d0b51ced15ea76dd6dcd49d..b7b9d075ab300fe0c05249b608bc72ce61d7cd79 100644 (file)
@@ -36,22 +36,7 @@ public class PasswordRecord extends Record {
     public PasswordRecord() {
     }
 
-    /**
-     * Constructs a Password record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PasswordRecord(RecordInputStream in) {
-        super(in);
-    }
-
-    protected void validateSid(short id) {
-        if (id != sid) {
-            throw new RecordFormatException("NOT A PASSWORD RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in) {
         field_1_password = in.readShort();
     }
 
index 8d31860af401b5fe1018d31f3338feb84258ae79..5c5567a86ba8a772b7a2b09673ca6faa4dfdf2c7 100644 (file)
@@ -40,25 +40,7 @@ public class PasswordRev4Record
     {
     }
 
-    /**
-     * Constructs a PasswordRev4 (PROT4REVPASS) record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PasswordRev4Record(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A PROT4REVPASSWORD RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_password = in.readShort();
     }
index e149c39ebb32851bcd93c14db5039c3d8aa6acad..e678eac782948c01bc92ed71159f2e24ae3cc51e 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
-
-
-import org.apache.poi.util.*;
+import org.apache.poi.util.LittleEndian;
 
 /**
  * preceeds and identifies a frame as belonging to the plot area.
@@ -30,9 +26,7 @@ import org.apache.poi.util.*;
 
  * @author Andrew C. Oliver (acoliver at apache.org)
  */
-public class PlotAreaRecord
-    extends Record
-{
+public final class PlotAreaRecord extends Record {
     public final static short      sid                             = 0x1035;
 
 
@@ -42,31 +36,9 @@ public class PlotAreaRecord
     }
 
     /**
-     * Constructs a PlotArea record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
+     * @param in unused (since this record has no data)
      */
-
     public PlotAreaRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a PlotArea record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
 
     }
index 29c3259923349100a539fb5063a5ab47d2aa431b..9df6811e2ab0a7130ed5aab69f60f3fda29fea92 100644 (file)
@@ -43,32 +43,7 @@ public class PlotGrowthRecord
 
     }
 
-    /**
-     * Constructs a PlotGrowth record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PlotGrowthRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a PlotGrowth record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_horizontalScale        = in.readInt();
         field_2_verticalScale          = in.readInt();
index dd7b719221037a2dca609ac21d4a326aee408cc8..a147cfff18727d4875466b31289d624cdec3c0d4 100644 (file)
@@ -40,25 +40,7 @@ public class PrecisionRecord
     {
     }
 
-    /**
-     * Constructs a Precision record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PrecisionRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A PRECISION RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_precision = in.readShort();
     }
index ff0eaac18d3f9ace916be7d3428ae082e304218e..7b4de7c68e78d5952df97c983465d89d5317004d 100644 (file)
@@ -40,25 +40,7 @@ public class PrintGridlinesRecord
     {
     }
 
-    /**
-     * Constructs a PrintGridlines record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PrintGridlinesRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A PrintGridlines RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_print_gridlines = in.readShort();
     }
index d3a69e513671ca7acc6456d8c56ddd924c406372..61c1cce86eabb7d27af548345ac323133ae6aabe 100644 (file)
@@ -41,25 +41,7 @@ public class PrintHeadersRecord
     {
     }
 
-    /**
-     * Constructs a PrintHeaders record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PrintHeadersRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A PrintHeaders RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_print_headers = in.readShort();
     }
index a8518c7b316ba00530d3aaa528a3bbf603f0830d..e41f278d5411e26a2f7dde266561d7303f509530 100644 (file)
@@ -66,26 +66,7 @@ public class PrintSetupRecord extends Record {
     {
     }
 
-    /**
-     * Constructs a PrintSetup (SETUP) record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public PrintSetupRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException(
-                "NOT A valid PrintSetup record RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_paper_size    = in.readShort();
         field_2_scale         = in.readShort();
index 94868e0a6055916e21fc0b9afaff5d0caaa830bb..d5425c55234823e9674da19410cd15409cc45f21 100644 (file)
@@ -42,25 +42,7 @@ public class ProtectRecord
     {
     }
 
-    /**
-     * Constructs a Protect record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ProtectRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A PROTECT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_protect = in.readShort();
     }
index 68678799dcf4bdf6127fccf5a99cc92090789723..b7d2f4323f37a68b93482ee0c74eb3539edda7e9 100644 (file)
@@ -40,25 +40,7 @@ public class ProtectionRev4Record
     {
     }
 
-    /**
-     * Constructs a ProtectionRev4 record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ProtectionRev4Record(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A PROTECTION REV 4 RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_protect = in.readShort();
     }
index ce9c84c6ec71877e08f485c7ad80e53266e52cd2..c4b25a5bead8f9b71c512f0e5f3785349932cbfd 100644 (file)
@@ -52,24 +52,7 @@ public final class RKRecord extends Record implements CellValueRecordInterface {
     {
     }
 
-    /**
-     * Constructs a RK record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
     public RKRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A valid RK RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_row       = in.readUShort();
         field_2_col       = in.readUShort();
index e076b2bab536f1bdb96e80d50af2da04f07aa63e..c0cab053c9a0584238ed0175ebf4670c2da78bbc 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -37,9 +35,7 @@ import org.apache.poi.util.LittleEndian;
  * @see org.apache.poi.hssf.model.Workbook
  */
 
-public class RecalcIdRecord
-    extends Record
-{
+public final class RecalcIdRecord extends Record {
     public final static short sid = 0x1c1;
     public short[]            field_1_recalcids;
 
@@ -49,25 +45,7 @@ public class RecalcIdRecord
     {
     }
 
-    /**
-     * Constructs a RECALCID record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public RecalcIdRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A RECALCID RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_recalcids = new short[ in.remaining() / 2 ];
         for (int k = 0; k < field_1_recalcids.length; k++)
@@ -126,9 +104,8 @@ public class RecalcIdRecord
         short   length     = ( short ) (tabids.length * 2);
         int     byteoffset = 4;
 
-        LittleEndian.putShort(data, 0 + offset, sid);
-        LittleEndian.putShort(data, 2 + offset,
-                              (( short ) length));
+        LittleEndian.putUShort(data, 0 + offset, sid);
+        LittleEndian.putUShort(data, 2 + offset, length);
 
         // 2 (num bytes in a short)
         for (int k = 0; k < (length / 2); k++)
index c33da57deb3848acf1e6731c0f5b28e2b563bf50..29c37d7047b056c628c810b5730fbe16a2d3b385 100644 (file)
@@ -40,35 +40,6 @@ public abstract class Record extends RecordBase {
     {
     }
 
-    /**
-     * Constructor Record
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-    protected Record(RecordInputStream in)
-    {
-        validateSid(in.getSid());
-        fillFields(in);
-    }
-
-    /**
-     * called by constructor, should throw runtime exception in the event of a
-     * record passed with a differing ID.
-     *
-     * @param id alleged id for this record
-     */
-
-    protected abstract void validateSid(short id);
-
-    /**
-     * called by the constructor, should set class level fields.  Should throw
-     * runtime exception for bad/icomplete data.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
-    protected abstract void fillFields(RecordInputStream in);
-
     /**
      * called by the class that is responsible for writing this sucker.
      * Subclasses should implement this so that their data is passed back in a
index c99d02e5c87c074803bff97d9b31d25201ce0ba5..f0567509bfa57fc40d72de6ec5ddd32501c0012c 100644 (file)
@@ -42,25 +42,7 @@ public class RefModeRecord
     {
     }
 
-    /**
-     * Constructs a RefMode record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public RefModeRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT An RefMode RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_mode = in.readShort();
     }
index eae7a532ed402c704ce122e46ca5b1cd13b29ea9..021a34c69be081ee891b29bf2d276b8785de5da1 100644 (file)
@@ -40,25 +40,7 @@ public class RefreshAllRecord
     {
     }
 
-    /**
-     * Constructs a RefreshAll record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public RefreshAllRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A REFRESHALL RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_refreshall = in.readShort();
     }
index 34c7d8e123540205f1b5371562c5033713944172..75cdb2c2f3a97de36d771e2206b56f7cf836ee41 100644 (file)
@@ -30,24 +30,7 @@ public class RightMarginRecord extends Record implements Margin
 
     public RightMarginRecord()    {    }
 
-    /**
-     * Constructs a RightMargin record and sets its fields appropriately.     *     * @param id    id must be 0x27 or an exception     *              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 RightMarginRecord( RecordInputStream in )
-    {        super( in );    }
-
-    /**
-     * Checks the sid matches the expected side for this record     *     * @param id   the expected sid.
-     */
-    protected void validateSid( short id )
-    {
-        if ( id != sid )
-        {
-            throw new RecordFormatException( "Not a RightMargin record" );
-        }
-    }
-
-    protected void fillFields( RecordInputStream in )
     {
         field_1_margin = in.readDouble();
     }
@@ -93,4 +76,4 @@ public class RightMarginRecord extends Record implements Margin
         rec.field_1_margin = this.field_1_margin;
         return rec;
     }
-}  // END OF CLASS
\ No newline at end of file
+}  // END OF CLA
\ No newline at end of file
index 8b49523a86a0f316b5908bb20ff952a5c85ead86..f893bf2831ec533a0fc6d8646fbbfe7557458b4e 100644 (file)
@@ -74,24 +74,7 @@ public final class RowRecord extends Record {
         field_8_xf_index = ( short ) 0xf;
     }
 
-    /**
-     * Constructs a Row record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
     public RowRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A valid ROW RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_row_number   = in.readUShort();
         field_2_first_col    = in.readShort();
index f06694b8658dd67ecbaf344eff47e357bd3f0d4d..7295ad61edc1a0bbee03d2a4ea71d79cae9ef04c 100644 (file)
@@ -43,32 +43,7 @@ public class SCLRecord
 
     }
 
-    /**
-     * Constructs a SCL record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SCLRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a SCL record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_numerator              = in.readShort();
         field_2_denominator            = in.readShort();
index 9c67c8dca45b3cf74c1d9c6cb833d986e04a2d4f..ca75af00d4c64afc28582670290db5c4bdb012bd 100644 (file)
@@ -90,17 +90,6 @@ public class SSTRecord
         deserializer = new SSTDeserializer(field_3_strings);
     }
 
-    /**
-     * Constructs an SST record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
-    public SSTRecord( RecordInputStream in )
-    {
-        super( in );
-    }
-
     /**
      * Add a string.
      *
@@ -256,23 +245,6 @@ public class SSTRecord
                 .equals( other.field_3_strings ) );
     }
 
-    /**
-     * validate SID
-     *
-     * @param id the alleged SID
-     *
-     * @exception RecordFormatException if validation fails
-     */
-
-    protected void validateSid( final short id )
-            throws RecordFormatException
-    {
-        if ( id != sid )
-        {
-            throw new RecordFormatException( "NOT An SST RECORD" );
-        }
-    }
-
     /**
      * Fill the fields from the data
      * <P>
@@ -352,7 +324,7 @@ public class SSTRecord
      * @param in the RecordInputstream to read the record from
      */
 
-    protected void fillFields( RecordInputStream in )
+    public SSTRecord( RecordInputStream in )
     {
         // this method is ALWAYS called after construction -- using
         // the nontrivial constructor, of course -- so this is where
index 96ff53af24d7af959cd79b132f9fd4338ec42b25..9cf008fe18cc55639c942459813981c9cad46847 100644 (file)
@@ -40,25 +40,7 @@ public class SaveRecalcRecord
     {
     }
 
-    /**
-     * Constructs an SaveRecalc record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SaveRecalcRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A Save Recalc RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_recalc = in.readShort();
     }
index f84316362cbc45091552fd818e119ebb6e4d1fc4..037eed02067412361c21b20121013e754b731685 100644 (file)
@@ -42,25 +42,7 @@ public class ScenarioProtectRecord
     {
     }
 
-    /**
-     * Constructs a Protect record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ScenarioProtectRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A SCENARIOPROTECT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_protect = in.readShort();
     }
index e1849a7d75c0514a1d6fb83e9768228601058873..ca2bf22fc14e734ec46a25401b2bed8539d1d555 100644 (file)
@@ -51,21 +51,7 @@ public final class SelectionRecord extends Record {
         };
     }
 
-    /**
-     * Constructs a Selection record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
     public SelectionRecord(RecordInputStream in) {
-        super(in);
-    }
-
-    protected void validateSid(short id) {
-        if (id != sid) {
-            throw new RecordFormatException("NOT A valid Selection RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in) {
         field_1_pane            = in.readByte();
         field_2_row_active_cell = in.readUShort();
         field_3_col_active_cell = in.readShort();
index 076687d69f514a02319cb0ce17ab598b4a25e55a..acc9665ae3490c5e31ded5404b0a01b2cdeda765 100644 (file)
@@ -42,32 +42,7 @@ public class SeriesChartGroupIndexRecord
 
     }
 
-    /**
-     * Constructs a SeriesChartGroupIndex record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SeriesChartGroupIndexRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a SeriesChartGroupIndex record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_chartGroupIndex        = in.readShort();
     }
index dcc2ebe4a3bef5372c89fbe28d5237715d51afd3..6896a0a217f147b888d1408fc46ee8f34405f6a8 100644 (file)
@@ -42,32 +42,7 @@ public class SeriesIndexRecord
 
     }
 
-    /**
-     * Constructs a SeriesIndex record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SeriesIndexRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a SeriesIndex record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_index                  = in.readShort();
     }
index e18ae4be83023eb1a54c524ac3d78d4d8bb9c772..a3e8089bfb6b8d2e133b1e0618cbf426b8b154ec 100644 (file)
@@ -46,32 +46,7 @@ public final class SeriesLabelsRecord extends Record {
 
     }
 
-    /**
-     * Constructs a SeriesLabels record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SeriesLabelsRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a SeriesLabels record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_formatFlags            = in.readShort();
     }
index 0825e68e6cdb246f8a3c0d27e9a67d4180f420dd..2894f158777a77d0108902aac251bcda82a9942f 100644 (file)
@@ -42,32 +42,7 @@ public class SeriesListRecord
 
     }
 
-    /**
-     * Constructs a SeriesList record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SeriesListRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a SeriesList record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_seriesNumbers          = in.readShortArray();
     }
index c13ff35e7c9223f813e4bcb4b79d30b34222f837..34b3c94a662b3987f4cf2ed301fbc74920620d3f 100644 (file)
@@ -59,32 +59,7 @@ public class SeriesRecord
 
     }
 
-    /**
-     * Constructs a Series record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SeriesRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Series record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_categoryDataType       = in.readShort();
         field_2_valuesDataType         = in.readShort();
index 66c26b32148285cb15bb3e6ae958263fdac12ad1..a445dc6166ace42bae1295e48be6a38797495cd2 100644 (file)
@@ -45,32 +45,7 @@ public class SeriesTextRecord
 
     }
 
-    /**
-     * Constructs a SeriesText record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SeriesTextRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a SeriesText record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_id                     = in.readShort();
         field_2_textLength             = in.readByte();
index 96e882e7c5a4a78aa2845714348246263755ffa7..ff69ec54692c764c8c018683935940b8d28cf34f 100644 (file)
@@ -42,32 +42,7 @@ public class SeriesToChartGroupRecord
 
     }
 
-    /**
-     * Constructs a SeriesToChartGroup record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SeriesToChartGroupRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a SeriesToChartGroup record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_chartGroupIndex        = in.readShort();
     }
index 16a73d0a81116b248b3c2cd7531441de6d5a5e3a..fc51de8465de19aa0dadaf954568ca053c51a63e 100644 (file)
@@ -45,12 +45,6 @@ public abstract class SharedValueRecordBase extends Record {
                _range = new CellRangeAddress8Bit(in);
        }
 
-       protected final void validateSid(short id) {
-               if (id != getSid()) {
-                       throw new RecordFormatException("Not a valid SharedFormula");
-               }
-       }
-
        public final CellRangeAddress8Bit getRange() {
                return _range;
        }
@@ -92,10 +86,6 @@ public abstract class SharedValueRecordBase extends Record {
                return dataSize + 4;
        }
 
-       protected final void fillFields(RecordInputStream in) {
-               throw new RuntimeException("Should not be called.  Fields are filled in constructor");
-       }
-
        /**
         * @return <code>true</code> if (rowIx, colIx) is within the range ({@link #getRange()})
         * of this shared value object.
index c919278b4286b5c545bb463cb65cc0eaf0fff880..9ae777e1e664624948a7c258eb310896ec75d7d0 100644 (file)
@@ -50,32 +50,7 @@ public final class SheetPropertiesRecord extends Record {
 
     }
 
-    /**
-     * Constructs a SheetProperties record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public SheetPropertiesRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a SheetProperties record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_flags                  = in.readShort();
         field_2_empty                  = in.readByte();
index d7eb11efcfa671262cbf5cafef896dd411799b0e..e4e71f7db8e99fc9f060aec6e19a1c09a8c36b56 100644 (file)
@@ -36,35 +36,10 @@ public class StringRecord extends Record {
     {
     }
 
-    /**
-     * Constructs a String record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-    public StringRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-
-    /**
-     * Throw a runtime exception in the event of a
-     * record passed with a differing ID.
-     *
-     * @param id alleged id for this record
-     */
-    protected void validateSid( short id )
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a valid StringRecord");
-        }
-    }
-
     /**
      * @param in the RecordInputstream to read the record from
      */
-    protected void fillFields( RecordInputStream in)
+    public StringRecord( RecordInputStream in)
     {
         field_1_string_length           = in.readShort();
         field_2_unicode_flag            = in.readByte();
index c6591f550de4e83762391b8ead794a61a1a69408..5b746e8d4b7dba7f0e4d0f7d181c3e2c6d136555 100644 (file)
@@ -54,25 +54,7 @@ public final class StyleRecord extends Record {
     {
     }
 
-    /**
-     * Constructs a Style record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public StyleRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A STYLE RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_xf_index = in.readShort();
         if (getType() == STYLE_BUILT_IN)
index 6b836c6ca6c18fa2d912916ccb8fc2d850587f63..1333ef19105b03a452e855774f4106b55a45b123 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
 /**
  * Subrecords are part of the OBJ class.
  */
-abstract public class SubRecord
-        extends Record
-{
-    public SubRecord()
-    {
-    }
-
-    public SubRecord( RecordInputStream in )
-    {
-        super( in );
+abstract public class SubRecord extends Record {
+    protected SubRecord() {
     }
 
     public static Record createSubRecord(RecordInputStream in)
index b4e46c6fdc8356024ea2499b94e8d2445d515afe..30e337e2d53a8e13c0b0d42873040732b4d67401 100644 (file)
@@ -66,23 +66,6 @@ public final class SupBookRecord extends Record {
         _isAddInFunctions = false;
     }
 
-    /**
-     * Constructs a Extern Sheet record and sets its fields appropriately.
-     *
-     * @param id     id must be 0x16 or an exception will be throw upon validation
-     * @param size  the size of the data area of the record
-     * @param data  data of the record (should not contain sid/len)
-     */
-    public SupBookRecord(RecordInputStream in) {
-        super(in);
-    }
-
-    protected void validateSid(short id) {
-        if (id != sid) {
-            throw new RecordFormatException("NOT An ExternSheet RECORD");
-        }
-    }
-
     public boolean isExternalReferences() {
         return field_3_sheet_names != null;
     }
@@ -100,7 +83,7 @@ public final class SupBookRecord extends Record {
      * @param size size of data
      * @param offset of the record's data (provided a big array of the file)
      */
-    protected void fillFields(RecordInputStream in) {
+    public SupBookRecord(RecordInputStream in) {
         field_1_number_of_sheets = in.readShort();
         
         if(in.getLength() > SMALL_RECORD_SIZE) {
index 7e924c33730c18b48698c4d127d62e4303112302..d5aca3e045315dc328549da6ca50365a9234658e 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -29,10 +27,7 @@ import org.apache.poi.util.LittleEndian;
  * @author Andrew C. Oliver (acoliver at apache dot org)
  * @version 2.0-pre
  */
-
-public class TabIdRecord
-    extends Record
-{
+public final class TabIdRecord extends Record {
     public final static short sid = 0x13d;
     public short[]            field_1_tabids;
 
@@ -40,25 +35,7 @@ public class TabIdRecord
     {
     }
 
-    /**
-     * Constructs a TabID record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public TabIdRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A TABID RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_tabids = new short[ in.remaining() / 2 ];
         for (int k = 0; k < field_1_tabids.length; k++)
@@ -105,13 +82,12 @@ public class TabIdRecord
 
     public int serialize(int offset, byte [] data)
     {
-        short[] tabids     = getTabIdArray();
-        short   length     = ( short ) (tabids.length * 2);
-        int     byteoffset = 4;
+        short[] tabids = getTabIdArray();
+        int length = tabids.length * 2;
+        int byteoffset = 4;
 
-        LittleEndian.putShort(data, 0 + offset, sid);
-        LittleEndian.putShort(data, 2 + offset,
-                              (( short ) length));   // nubmer tabids *
+        LittleEndian.putUShort(data, 0 + offset, sid);
+        LittleEndian.putUShort(data, 2 + offset, length);   // nubmer tabids *
 
         // 2 (num bytes in a short)
         for (int k = 0; k < (length / 2); k++)
index 2d649247b5fc9777b582bfa8a0faaecb28db7c21..8a27d163dac90718ce93e0dc57bae213ba022d8e 100644 (file)
@@ -69,32 +69,7 @@ public class TextObjectBaseRecord extends Record {
 
     }
 
-    /**
-     * Constructs a TextObjectBase record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public TextObjectBaseRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a TextObjectBase record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_options                = in.readShort();
         field_2_textOrientation        = in.readShort();
index 5201a7292b926cfc16c94c9f2df7a608b75f9e1c..600aaf1542c6d63e74c20aa85ea4beef28d8e12e 100644 (file)
 
 package org.apache.poi.hssf.record;
 
+import java.io.UnsupportedEncodingException;
+
 import org.apache.poi.hssf.usermodel.HSSFRichTextString;
-import org.apache.poi.util.LittleEndian;
 import org.apache.poi.util.HexDump;
-import java.io.UnsupportedEncodingException;
-import java.io.ByteArrayOutputStream;
+import org.apache.poi.util.LittleEndian;
 
 public class TextObjectRecord
         extends TextObjectBaseRecord
@@ -35,13 +35,7 @@ public class TextObjectRecord
     public TextObjectRecord( RecordInputStream in )
     {
         super( in );
-        if (str == null)
-               str = new HSSFRichTextString("");
-    }
 
-    protected void fillFields(RecordInputStream in)
-    {
-        super.fillFields(in);
         if (getTextLength() > 0) {
         if (in.isContinueNext() && in.remaining() == 0) {
             //1st Continue
@@ -56,6 +50,8 @@ public class TextObjectRecord
                 processFontRuns(in);
             } else throw new RecordFormatException("Expected Continue Record to hold font runs for TextObjectRecord");
         }
+        if (str == null)
+               str = new HSSFRichTextString("");
     }
 
 
index 0840fe19324c275f91f7d8ac5f20bd5ffabbf1dd..60c05e6d0a71403fab09be8afd0fad54bd5a8527 100644 (file)
@@ -93,32 +93,7 @@ public final class TextRecord extends Record {
 
     }
 
-    /**
-     * Constructs a Text record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public TextRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Text record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_horizontalAlignment    = in.readByte();
         field_2_verticalAlignment      = in.readByte();
index 1b950e3ed428c02eea6e3627efce6de933dbcd1b..7e216156c67ff7eaa929cc05b7ff7cccdb9f90b3 100644 (file)
@@ -56,32 +56,7 @@ public final class TickRecord extends Record {
 
     }
 
-    /**
-     * Constructs a Tick record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public TickRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Tick record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
 
         field_1_majorTickType          = in.readByte();
index 5c507a211ba7cbb4b6a6e3a939fcfc0a9d8178de..d67509e6c6faa3f9f8ceb1dcd71447da1842163a 100644 (file)
@@ -33,30 +33,9 @@ public class TopMarginRecord extends Record implements Margin
     public TopMarginRecord()    {    }
 
     /**
-     * Constructs a TopMargin record and sets its fields appropriately.
-     *
      * @param in the RecordInputstream to read the record from
      */
     public TopMarginRecord( RecordInputStream in )
-    {        super( in );    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid( short id )
-    {
-        if ( id != sid )
-        {
-            throw new RecordFormatException( "Not a TopMargin record" );
-        }
-    }
-
-    /**
-     * @param in the RecordInputstream to read the record from
-     */
-    protected void fillFields( RecordInputStream in )
     {
         field_1_margin = in.readDouble();
     }
@@ -102,4 +81,4 @@ public class TopMarginRecord extends Record implements Margin
         rec.field_1_margin = this.field_1_margin;
         return rec;
     }
-}  // END OF CLASS
+}  // END OF CLAS
\ No newline at end of file
index a67b0b5af4adb9b2eed6868bd0ef87778869b7b1..adc2cdeb907bf80067fcda358f125521726f7c90 100644 (file)
@@ -37,25 +37,13 @@ public class UncalcedRecord extends Record
         */
        public UncalcedRecord() {
        }
-       /**
-        * read constructor
-        */
-       public UncalcedRecord(RecordInputStream in) {
-               super(in);
-       }
 
        public short getSid() {
                return sid;
        }
 
-       protected void validateSid(short id) {
-               if (id != sid) {
-                       throw new RecordFormatException("NOT AN UNCALCED RECORD");
-               }
-       }
-
-       protected void fillFields(RecordInputStream in) {
-               short unused = in.readShort();
+       public UncalcedRecord(RecordInputStream in) {
+               in.readShort(); // unused
        }
 
        public String toString() {
@@ -66,9 +54,9 @@ public class UncalcedRecord extends Record
        }
 
        public int serialize(int offset, byte[] data) {
-               LittleEndian.putShort(data, 0 + offset, sid);
-               LittleEndian.putShort(data, 2 + offset, (short) 2);
-               LittleEndian.putShort(data, 4 + offset, (short) 0); // unused
+               LittleEndian.putUShort(data, 0 + offset, sid);
+               LittleEndian.putUShort(data, 2 + offset, 2);
+               LittleEndian.putUShort(data, 4 + offset, 0); // unused
                return getRecordSize();
        }
 
index 9919d52c3d5ee83c79c03a07101f97b314617eb5..4258d0a6d5164e2cc4000c3ef579e520a128181c 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -111,8 +109,7 @@ public class UnicodeString
 
     public UnicodeString(RecordInputStream in)
     {
-      validateSid(in.getSid());
-      fillFields(in);
+      fillFields(in); // TODO - inline
     }
 
 
@@ -188,16 +185,6 @@ public class UnicodeString
         return true;
     }
 
-    /**
-     * NO OP
-     */
-
-    protected void validateSid(short id)
-    {
-
-        // included only for interface compliance
-    }
-
     /**
      * @param in the RecordInputstream to read the record from
      */
@@ -236,9 +223,7 @@ public class UnicodeString
               throw new RecordFormatException("Expected continue record.");
           }
           if (isCompressed) {
-            //Typecast direct to char from byte with high bit set causes all ones
-            //in the high byte of the char (which is of course incorrect)
-            char ch = (char)( (short)0xff & (short)in.readByte() );
+            char ch = (char)in.readUByte(); // avoid sex
             tmpString.append(ch);
           } else {
             char ch = (char) in.readShort();
index a427a6c73950a12ae565a3d3163fc4c0713d8fee..4295cfe79ba27877b1768148f72880a629d8d109 100644 (file)
@@ -42,31 +42,7 @@ public class UnitsRecord
 
     }
 
-    /**
-     * Constructs a Units record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
     public UnitsRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a Units record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_units                  = in.readShort();
 
index 851dbfbf55d339ac3eb8e72921c899f1ec633294..77de66d9b32fdfaf434b36ba39842e867cfc588c 100644 (file)
@@ -91,13 +91,6 @@ public final class UnknownRecord extends Record {
                return 4 + _rawData.length;
        }
 
-       /**
-        * NO OP!
-        */
-       protected void validateSid(short id) {
-               // if we had a valid sid we wouldn't be using the "Unknown Record" record now would we?
-       }
-
        /**
         * print a sort of string representation ([UNKNOWN RECORD] id = x [/UNKNOWN RECORD])
         */
@@ -276,11 +269,6 @@ public final class UnknownRecord extends Record {
                return false;
        }
 
-       protected final void fillFields(RecordInputStream in) {
-               throw new RecordFormatException(
-                               "Unknown record cannot be constructed via offset -- we need a copy of the data");
-       }
-
        public final Object clone() {
                // immutable - ok to return this
                return this;
index 107792eb3e3940c9143346a4af6ea94ee21656cc..8e64ef3d4a6fe86af3372add461427b12a9ad022 100644 (file)
@@ -42,25 +42,7 @@ public class UseSelFSRecord
     {
     }
 
-    /**
-     * Constructs a UseSelFS record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public UseSelFSRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A UseSelFS RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_flag = in.readShort();
     }
index e221b35f1c4edae9796f15cdf4916a17740c0b81..7326a3292903015e16d79f165142bac0a3ea647f 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -30,35 +28,15 @@ import org.apache.poi.util.LittleEndian;
  * @version 2.0-pre
  */
 
-public class VCenterRecord
-    extends Record
-{
+public final class VCenterRecord extends Record {
     public final static short sid = 0x84;
-    private short             field_1_vcenter;
+    private int field_1_vcenter;
 
     public VCenterRecord()
     {
     }
 
-    /**
-     * Constructs a VCENTER record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public VCenterRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A VCenter RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_vcenter = in.readShort();
     }
@@ -70,14 +48,7 @@ public class VCenterRecord
 
     public void setVCenter(boolean hc)
     {
-        if (hc == true)
-        {
-            field_1_vcenter = 1;
-        }
-        else
-        {
-            field_1_vcenter = 0;
-        }
+       field_1_vcenter = hc ? 1 : 0;
     }
 
     /**
@@ -103,9 +74,9 @@ public class VCenterRecord
 
     public int serialize(int offset, byte [] data)
     {
-        LittleEndian.putShort(data, 0 + offset, sid);
-        LittleEndian.putShort(data, 2 + offset, ( short ) 0x2);
-        LittleEndian.putShort(data, 4 + offset, ( short ) field_1_vcenter);
+        LittleEndian.putUShort(data, 0 + offset, sid);
+        LittleEndian.putUShort(data, 2 + offset, 0x2);
+        LittleEndian.putUShort(data, 4 + offset, field_1_vcenter);
         return getRecordSize();
     }
 
index 6c94c5c54cca35191e05792bbbdc90a71af2f8e2..bc5023212c09683ea26aa0fa934d4d0e436e0dd3 100644 (file)
@@ -55,32 +55,7 @@ public final class ValueRangeRecord extends Record {
 
     }
 
-    /**
-     * Constructs a ValueRange record and sets its fields appropriately.
-     *
-     * @param in the RecordInputstream to read the record from
-     */
-
     public ValueRangeRecord(RecordInputStream in)
-    {
-        super(in);
-    
-    }
-
-    /**
-     * Checks the sid matches the expected side for this record
-     *
-     * @param id   the expected sid.
-     */
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("Not a ValueRange record");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_minimumAxisValue       = in.readDouble();
         field_2_maximumAxisValue       = in.readDouble();
index fccb7ccdaac0271ccf58fbe36266c228d915bf2e..3dc5348f971c83ec16dfe3fa22c1d5dc013541d1 100644 (file)
@@ -43,13 +43,6 @@ public final class VerticalPageBreakRecord extends PageBreakRecord {
                super(in);
        }
 
-       protected void validateSid(short id) {
-               if (id != getSid()) {
-                       throw new RecordFormatException(
-                                       "NOT A HorizontalPageBreak or VerticalPageBreak RECORD!! " + id);
-               }
-       }
-
        public short getSid() {
                return sid;
        }
index 480b7f240e03f762730ae7448d3acad125dafcf7..bdb1cb4847868bac8212b3b8eabb218b7042fc3c 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -33,63 +31,33 @@ import org.apache.poi.util.LittleEndian;
  * @author Jason Height (jheight at chariot dot net dot au)
  * @version 2.0-pre
  */
-
-public class WSBoolRecord
-    extends Record
-{
+public final class WSBoolRecord extends Record {
     public final static short     sid = 0x81;
     private byte                  field_1_wsbool;         // crappy names are because this is really one big short field (2byte)
-    private byte                  field_2_wsbool;         // but the docs inconsistantly use it as 2 seperate bytes
+    private byte                  field_2_wsbool;         // but the docs inconsistently use it as 2 separate bytes
 
-    // I decided to be consistant in this way.
-    static final private BitField autobreaks          =
-        BitFieldFactory.getInstance(0x01);                               // are automatic page breaks visible
+    // I decided to be consistent in this way.
+    private static final BitField autobreaks    = BitFieldFactory.getInstance(0x01); // are automatic page breaks visible
 
     // bits 1 to 3 unused
-    static final private BitField dialog              =
-        BitFieldFactory.getInstance(0x10);                               // is sheet dialog sheet
-    static final private BitField applystyles         =
-        BitFieldFactory.getInstance(0x20);                               // whether to apply automatic styles to outlines
-    static final private BitField rowsumsbelow        = BitFieldFactory.getInstance(
-        0x40);                                            // whether summary rows will appear below detail in outlines
-    static final private BitField rowsumsright        = BitFieldFactory.getInstance(
-        0x80);                                            // whether summary rows will appear right of the detail in outlines
-    static final private BitField fittopage           =
-        BitFieldFactory.getInstance(0x01);                               // whether to fit stuff to the page
+    private static final BitField dialog        = BitFieldFactory.getInstance(0x10); // is sheet dialog sheet
+    private static final BitField applystyles   = BitFieldFactory.getInstance(0x20); // whether to apply automatic styles to outlines
+    private static final BitField rowsumsbelow  = BitFieldFactory.getInstance(0x40); // whether summary rows will appear below detail in outlines
+    private static final BitField rowsumsright  = BitFieldFactory.getInstance(0x80); // whether summary rows will appear right of the detail in outlines
+    private static final BitField fittopage     = BitFieldFactory.getInstance(0x01); // whether to fit stuff to the page
 
     // bit 2 reserved
-    static final private BitField displayguts         = BitFieldFactory.getInstance(
-        0x06);                                            // whether to display outline symbols (in the gutters)
+    private static final BitField displayguts   = BitFieldFactory.getInstance(0x06); // whether to display outline symbols (in the gutters)
 
     // bits 4-5 reserved
-    static final private BitField alternateexpression =   // whether to use alternate expression eval
-        BitFieldFactory.getInstance(0x40);
-    static final private BitField alternateformula    =   // whether to use alternate formula entry
-        BitFieldFactory.getInstance(0x80);
+    private static final BitField alternateexpression = BitFieldFactory.getInstance(0x40); // whether to use alternate expression eval
+    private static final BitField alternateformula    = BitFieldFactory.getInstance(0x80); // whether to use alternate formula entry
 
     public WSBoolRecord()
     {
     }
 
-    /**
-     * Constructs a WSBool record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public WSBoolRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A WSBoolRECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
        byte data[] = in.readRemainder();
         field_1_wsbool =
index 4c836f2ee83e731d15ba44c6b2d1780cc3826fbb..7afc5fa1f893ab5ac66a793f999f0a01b57413eb 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,7 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
@@ -32,10 +30,7 @@ import org.apache.poi.util.LittleEndian;
  * @author Andrew C. Oliver (acoliver at apache dot org)
  * @version 2.0-pre
  */
-
-public class WindowOneRecord
-    extends Record
-{
+public final class WindowOneRecord extends Record {
     public final static short     sid = 0x3d;
 
     // our variable names stolen from old TV sets.
@@ -66,25 +61,7 @@ public class WindowOneRecord
     {
     }
 
-    /**
-     * Constructs a WindowOne record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public WindowOneRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A WINDOW1 RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_h_hold            = in.readShort();
         field_2_v_hold            = in.readShort();
index 9a5bc45dcee8faf916b4c2b2980a6fd2d9c2fe51..13d54e9b5e3dc3e6816b59e19a9a97352182cb55 100644 (file)
@@ -39,25 +39,7 @@ public class WindowProtectRecord
     {
     }
 
-    /**
-     * Constructs a WindowProtect record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public WindowProtectRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A WINDOWPROTECT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         field_1_protect = in.readShort();
     }
index 6e60ef41a6974e0898ba08d41f24ad3a36f624c9..9023d2d6920534268fc834781f63291ce0a3f7c0 100644 (file)
@@ -61,25 +61,7 @@ public final class WindowTwoRecord extends Record {
     {
     }
 
-    /**
-     * Constructs a WindowTwo record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public WindowTwoRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A valid WindowTwo RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
       int size = in.remaining();
         field_1_options      = in.readShort();
index af5f06e04f91a20eb39632bc4acdd6da27a65f3b..d73687f1866a07f441cf4c72bdd3a3c7656c7da3 100644 (file)
@@ -42,25 +42,7 @@ public class WriteAccessRecord
     {
     }
 
-    /**
-     * Constructs a WriteAccess record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
-     */
-
     public WriteAccessRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A WRITEACCESS RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
         byte[] data = in.readRemainder();
         //The string is always 112 characters (padded with spaces), therefore
index 9b3daf2a161e6c9519c9a926021a2c6610d4cbf6..eb27e0ec129dc3ec245c3bc60009e1183d582be5 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
 
 package org.apache.poi.hssf.record;
 
 import org.apache.poi.util.LittleEndian;
-import org.apache.poi.util.StringUtil;
 
 /**
  * Title:        Write Protect Record<P>
@@ -28,10 +25,7 @@ import org.apache.poi.util.StringUtil;
  * REFERENCE:  PG 425 Microsoft Excel 97 Developer's Kit (ISBN: 1-57231-498-2)<P>
  * @version 3.0-pre
  */
-
-public class WriteProtectRecord
-    extends Record
-{
+public final class WriteProtectRecord extends Record {
     public final static short sid = 0x86;
 
     public WriteProtectRecord()
@@ -39,24 +33,9 @@ public class WriteProtectRecord
     }
 
     /**
-     * Constructs a WriteAccess record and sets its fields appropriately.
-     * @param in the RecordInputstream to read the record from
+     * @param in unused (since this record has no data)
      */
-
     public WriteProtectRecord(RecordInputStream in)
-    {
-        super(in);
-    }
-
-    protected void validateSid(short id)
-    {
-        if (id != sid)
-        {
-            throw new RecordFormatException("NOT A WRITEPROTECT RECORD");
-        }
-    }
-
-    protected void fillFields(RecordInputStream in)
     {
     }
 
@@ -71,8 +50,8 @@ public class WriteProtectRecord
 
     public int serialize(int offset, byte [] data)
     {
-        LittleEndian.putShort(data, 0 + offset, sid);
-        LittleEndian.putShort(data, 2 + offset, (short)0);
+        LittleEndian.putUShort(data, 0 + offset, sid);
+        LittleEndian.putUShort(data, 2 + offset, 0);
 
         return getRecordSize();
     }
index 2f30d15c81bd8cf316e380e2b4cc90d5042c451a..0c666847af3b722adefb51afd23655c440453137 100644 (file)
@@ -147,22 +147,19 @@ public class HSSFFormulaEvaluator /* almost implements FormulaEvaluator */ {
 \r
 \r
        /**\r
-        * If cell contains formula, it evaluates the formula,\r
-        *  and saves the result of the formula. The cell\r
-        *  remains as a formula cell.\r
-        * Else if cell does not contain formula, this method leaves\r
-        *  the cell unchanged.\r
-        * Note that the type of the formula result is returned,\r
-        *  so you know what kind of value is also stored with\r
-        *  the formula.\r
+        * If cell contains formula, it evaluates the formula, and saves the result of the formula. The\r
+        * cell remains as a formula cell. If the cell does not contain formula, this method returns -1\r
+        * and leaves the cell unchanged.\r
+        * \r
+        * Note that the type of the <em>formula result</em> is returned, so you know what kind of \r
+        * cached formula result is also stored with  the formula.\r
         * <pre>\r
         * int evaluatedCellType = evaluator.evaluateFormulaCell(cell);\r
         * </pre>\r
-        * Be aware that your cell will hold both the formula,\r
-        *  and the result. If you want the cell replaced with\r
-        *  the result of the formula, use {@link #evaluateInCell(HSSFCell)}\r
+        * Be aware that your cell will hold both the formula, and the result. If you want the cell \r
+        * replaced with the result of the formula, use {@link #evaluateInCell(HSSFCell)}\r
         * @param cell The cell to evaluate\r
-        * @return The type of the formula result (the cell's type remains as HSSFCell.CELL_TYPE_FORMULA however)\r
+        * @return -1 for non-formula cells, or the type of the <em>formula result</em>\r
         */\r
        public int evaluateFormulaCell(Cell cell) {\r
                if (cell == null || cell.getCellType() != HSSFCell.CELL_TYPE_FORMULA) {\r
index aa0b72013267eb9b06455d491d5f0401a4992339..aacf6638632d482cfdee2449ceb543bdb9fac24e 100644 (file)
@@ -516,15 +516,17 @@ public class HSSFWorkbook extends POIDocument implements org.apache.poi.ss.userm
     }
 
     /**
-     * set the sheet name.
-     * Will throw IllegalArgumentException if the name is greater than 31 chars
-     * or contains /\?*[]
+     * Sets the sheet name.
+     * Will throw IllegalArgumentException if the name is duplicated or contains /\?*[]
+     * Note - Excel allows sheet names up to 31 chars in length but other applications allow more.
+     * Excel does not crash with names longer than 31 chars, but silently truncates such names to 
+     * 31 chars.  POI enforces uniqueness on the first 31 chars.
+     * 
      * @param sheetIx number (0 based)
      */
-    public void setSheetName(int sheetIx, String name)
-    {
-        if (workbook.doesContainsSheetName( name, sheetIx )) {
-            throw new IllegalArgumentException( "The workbook already contains a sheet with this name" );
+    public void setSheetName(int sheetIx, String name) {
+        if (workbook.doesContainsSheetName(name, sheetIx)) {
+            throw new IllegalArgumentException("The workbook already contains a sheet with this name");
         }
         validateSheetIndex(sheetIx);
         workbook.setSheetName(sheetIx, name);
@@ -764,14 +766,14 @@ public class HSSFWorkbook extends POIDocument implements org.apache.poi.ss.userm
      * create an HSSFSheet for this HSSFWorkbook, adds it to the sheets and
      * returns the high level representation. Use this to create new sheets.
      *
-     * @param sheetname
-     *            sheetname to set for the sheet.
+     * @param sheetname the name for the new sheet. Note - certain length limits
+     * apply. See {@link #setSheetName(int, String)}.
+     *
      * @return HSSFSheet representing the new sheet.
      * @throws IllegalArgumentException
      *             if there is already a sheet present with a case-insensitive
      *             match for the specified name.
      */
-
     public HSSFSheet createSheet(String sheetname)
     {
         if (workbook.doesContainsSheetName( sheetname, _sheets.size() ))
index 7d543c451f4d51c9c0c6bb31026b913783c10e66..de834be95eed8fbc06908d2fa7ac0495b24da110 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestAreaFormatRecord
-        extends TestCase
-{
+public final class TestAreaFormatRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0xFF,(byte)0xFF,(byte)0xFF,(byte)0x00,    // forecolor
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,    // backcolor
@@ -44,14 +39,7 @@ public class TestAreaFormatRecord
 
     };
 
-    public TestAreaFormatRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         AreaFormatRecord record = new AreaFormatRecord(new TestcaseRecordInputStream((short)0x100a, (short)data.length, data));
         assertEquals( 0xFFFFFF, record.getForegroundColor());
@@ -65,8 +53,6 @@ public class TestAreaFormatRecord
 
 
         assertEquals( 20, record.getRecordSize() );
-
-        record.validateSid((short)0x100a);
     }
 
     public void testStore()
index 23b50a5ffeb435e0b27ccfe9447ba8fb17c1fd7d..2131e7319284e173e535e9c069e6b4d4d99ccf9d 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,21 +28,12 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestAreaRecord
-        extends TestCase
-{
+public final class TestAreaRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x02,(byte)0x00       // format flags
     };
 
-    public TestAreaRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         AreaRecord record = new AreaRecord(new TestcaseRecordInputStream((short)0x101A, (short)data.length, data));
         assertEquals( 2, record.getFormatFlags());
@@ -55,8 +43,6 @@ public class TestAreaRecord
 
 
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x101A);
     }
 
     public void testStore()
index 5340e20d98c394b4c4a6f133175c037f1397e556..2146a31cfb58cb23558d3f7eee80ebc6221a986b 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,28 +28,16 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestAxisLineFormatRecord
-        extends TestCase
-{
+public final class TestAxisLineFormatRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x01,(byte)0x00
     };
 
-    public TestAxisLineFormatRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         AxisLineFormatRecord record = new AxisLineFormatRecord(new TestcaseRecordInputStream((short)0x1021, (short)data.length, data));
         assertEquals( AxisLineFormatRecord.AXIS_TYPE_MAJOR_GRID_LINE, record.getAxisType());
 
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1021);
     }
 
     public void testStore()
index 498f896a7610c33327faf4e2f68c252955730eb1..4d88e18a69552d4e62c3b6ac98783e9f5d0e4551 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver(acoliver at apache.org)
  */
-public class TestAxisOptionsRecord
-        extends TestCase
-{
+public final class TestAxisOptionsRecord extends TestCase {
     byte[] data = new byte[] {        
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,
@@ -41,14 +36,7 @@ public class TestAxisOptionsRecord
         (byte)0x00,(byte)0xEF,(byte)0x00
     };
 
-    public TestAxisOptionsRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         AxisOptionsRecord record = new AxisOptionsRecord(new TestcaseRecordInputStream((short)0x1062, (short)data.length, data));
         assertEquals( 0, record.getMinimumCategory());
         assertEquals( 0, record.getMaximumCategory());
@@ -68,10 +56,7 @@ public class TestAxisOptionsRecord
         assertEquals( true, record.isDefaultCross() );
         assertEquals( true, record.isDefaultDateSettings() );
 
-
         assertEquals( 22, record.getRecordSize() );
-
-        record.validateSid((short)0x1062);
     }
 
     public void testStore()
index 0d5cdb1719be8ef179e928618a0c3f02e1a9b894..a9787a2e8b4584f87110f266ea14b76bdacb3efe 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -30,9 +27,7 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestAxisParentRecord
-        extends TestCase
-{
+public final class TestAxisParentRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00,                                   // axis type
         (byte)0x1D,(byte)0x02,(byte)0x00,(byte)0x00,             // x
@@ -41,14 +36,7 @@ public class TestAxisParentRecord
         (byte)0x56,(byte)0x0B,(byte)0x00,(byte)0x00              // height
     };
 
-    public TestAxisParentRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         AxisParentRecord record = new AxisParentRecord(new TestcaseRecordInputStream((short)0x1041, (short)data.length, data));
         assertEquals( AxisParentRecord.AXIS_TYPE_MAIN, record.getAxisType());
         assertEquals( 0x021d, record.getX());
@@ -56,10 +44,7 @@ public class TestAxisParentRecord
         assertEquals( 0x0b31, record.getWidth());
         assertEquals( 0x0b56, record.getHeight());
 
-
         assertEquals( 22, record.getRecordSize() );
-
-        record.validateSid((short)0x1041);
     }
 
     public void testStore()
index 76c53bd164952b0061f2a3575238801252e1055e..719e930d2545e0ec35d94b29b8f925fc6043bc64 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,26 +28,16 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestAxisRecord
-        extends TestCase
-{
+public final class TestAxisRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00,                               // type
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00
-
     };
 
-    public TestAxisRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         AxisRecord record = new AxisRecord(new TestcaseRecordInputStream((short)0x101d, (short)data.length, data));
         assertEquals( AxisRecord.AXIS_TYPE_CATEGORY_OR_X_AXIS, record.getAxisType());
@@ -59,10 +46,7 @@ public class TestAxisRecord
         assertEquals( 0, record.getReserved3());
         assertEquals( 0, record.getReserved4());
 
-
         assertEquals( 4 + 18, record.getRecordSize() );
-
-        record.validateSid((short)0x101d);
     }
 
     public void testStore()
index 24516e095aaea51fc39c055f8023260ab4db7664..98812ddd6f8f8abf22f890a7790b03182559dd63 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,28 +28,16 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestAxisUsedRecord
-        extends TestCase
-{
+public final class TestAxisUsedRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x01,(byte)0x00,
     };
 
-    public TestAxisUsedRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         AxisUsedRecord record = new AxisUsedRecord(new TestcaseRecordInputStream((short)0x1046, (short)data.length, data));
         assertEquals( 1, record.getNumAxis());
 
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1046);
     }
 
     public void testStore()
index 99cf3678872b6136462a71c0e5456ce3eb5fd657..1da17aae926e97b5d5c9e2f9f26c905d95601b54 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,23 +28,14 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestBarRecord
-        extends TestCase
-{
+public final class TestBarRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00,   // bar space
         (byte)0x96,(byte)0x00,   // category space
         (byte)0x00,(byte)0x00    // format flags
     };
 
-    public TestBarRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         BarRecord record = new BarRecord(new TestcaseRecordInputStream((short)0x1017, (short)data.length, data));
         assertEquals( 0, record.getBarSpace());
@@ -58,10 +46,7 @@ public class TestBarRecord
         assertEquals( false, record.isDisplayAsPercentage() );
         assertEquals( false, record.isShadow() );
 
-
         assertEquals( 10, record.getRecordSize() );
-
-        record.validateSid((short)0x1017);
     }
 
     public void testStore()
index e80afe973b07a82ebf5165369dbe876cc95b19a3..f973da37dbc682e3266365ed060cd6a6f000dedf 100644 (file)
@@ -45,13 +45,6 @@ public final class TestBoundSheetRecord extends TestCase {
     public void testName() {
         BoundSheetRecord record = new BoundSheetRecord("1234567890223456789032345678904");
 
-        try {
-            record.setSheetname("12345678902234567890323456789042");
-            throw new AssertionFailedError("Should have thrown IllegalArgumentException, but didnt");
-        } catch (IllegalArgumentException e) {
-            // expected
-        }
-
         try {
             record.setSheetname("s//*s");
             throw new AssertionFailedError("Should have thrown IllegalArgumentException, but didnt");
index d0baf9b95bf74bd8833410376988ba262a26818c..415b302e87508480fccc288150644f54af2265d1 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestCategorySeriesAxisRecord
-        extends TestCase
-{
+public final class TestCategorySeriesAxisRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x01,(byte)0x00,    // crossing point
         (byte)0x01,(byte)0x00,    // label frequency
@@ -41,14 +36,7 @@ public class TestCategorySeriesAxisRecord
         (byte)0x01,(byte)0x00     // options
     };
 
-    public TestCategorySeriesAxisRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         CategorySeriesAxisRecord record = new CategorySeriesAxisRecord(new TestcaseRecordInputStream((short)0x1020, (short)data.length, data));
         assertEquals( 1, record.getCrossingPoint());
@@ -59,10 +47,7 @@ public class TestCategorySeriesAxisRecord
         assertEquals( false, record.isCrossesFarRight() );
         assertEquals( false, record.isReversed() );
 
-
         assertEquals( 4 + 8, record.getRecordSize() );
-
-        record.validateSid((short)0x1020);
     }
 
     public void testStore()
index 16591a209bee67341c0349942efe7864180d833c..996fb1102b12d171aab12ff4685882ef255ff694 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestChartRecord
-        extends TestCase
-{
+public final class TestChartRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,
@@ -41,14 +36,7 @@ public class TestChartRecord
         (byte)0xC8,(byte)0xCC,(byte)0xE5,(byte)0x00
     };
 
-    public TestChartRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         ChartRecord record = new ChartRecord(new TestcaseRecordInputStream((short)0x1002, (short)data.length, data));
         assertEquals( 0, record.getX());
@@ -56,10 +44,7 @@ public class TestChartRecord
         assertEquals( 30474216, record.getWidth());
         assertEquals( 15060168, record.getHeight());
 
-
         assertEquals( 20, record.getRecordSize() );
-
-        record.validateSid((short)0x1002);
     }
 
     public void testStore()
index 0f7ba7296927b889638f977bf447f50969d75ab4..e68b5026d85b640a79b304c5254e05cfb06f2c1c 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestCommonObjectDataSubRecord
-        extends TestCase
-{
+public final class TestCommonObjectDataSubRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x12,(byte)0x00,(byte)0x01,(byte)0x00,
         (byte)0x01,(byte)0x00,(byte)0x11,(byte)0x60,
@@ -42,16 +37,8 @@ public class TestCommonObjectDataSubRecord
         (byte)0x00,(byte)0x00,
     };
 
-    public TestCommonObjectDataSubRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         CommonObjectDataSubRecord record = new CommonObjectDataSubRecord(new TestcaseRecordInputStream((short)0x15, (short)data.length, data));
-        
 
         assertEquals( CommonObjectDataSubRecord.OBJECT_TYPE_LIST_BOX, record.getObjectType());
         assertEquals( (short)1, record.getObjectId());
@@ -64,8 +51,6 @@ public class TestCommonObjectDataSubRecord
         assertEquals( (int)218103808, record.getReserved2());
         assertEquals( (int)294, record.getReserved3());
         assertEquals( 22 , record.getRecordSize() );
-
-        record.validateSid((short)0x15);
     }
 
     public void testStore()
index 3cd7a0603043c086e10275b4ed62405852e2cec7..8efbe70ecbccf3b9a8663ac4c8a116943c73a3bb 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,21 +28,12 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestDatRecord
-        extends TestCase
-{
+public final class TestDatRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x0D,(byte)0x00   // options
     };
 
-    public TestDatRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         DatRecord record = new DatRecord(new TestcaseRecordInputStream((short)0x1063, (short)data.length, data));
         assertEquals( 0xD, record.getOptions());
@@ -54,10 +42,7 @@ public class TestDatRecord
         assertEquals( true, record.isBorder() );
         assertEquals( true, record.isShowSeriesKey() );
 
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1063);
     }
 
     public void testStore()
index ff8111ced568156734c55dfa21b4cebb745c8172..8de2f24dac549b165402ae69fe1dcce9018b7473 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestDataFormatRecord
-        extends TestCase
-{
+public final class TestDataFormatRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0xFF,(byte)0xFF,      // point number
         (byte)0x00,(byte)0x00,      // series index
@@ -41,14 +36,7 @@ public class TestDataFormatRecord
         (byte)0x00,(byte)0x00       // format flags
     };
 
-    public TestDataFormatRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         DataFormatRecord record = new DataFormatRecord(new TestcaseRecordInputStream((short)0x1006, (short)data.length, data));
         assertEquals( (short)0xFFFF, record.getPointNumber());
@@ -57,10 +45,7 @@ public class TestDataFormatRecord
         assertEquals( 0, record.getFormatFlags());
         assertEquals( false, record.isUseExcel4Colors() );
 
-
         assertEquals( 12, record.getRecordSize() );
-
-        record.validateSid((short)0x1006);
     }
 
     public void testStore()
index 61a3dd5926e99e987260153b0589187a004a0dd2..f96dca90bfdfe486e1e85adf440034bd5040ac33 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,29 +28,17 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestDefaultDataLabelTextPropertiesRecord
-        extends TestCase
-{
+public final class TestDefaultDataLabelTextPropertiesRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x02,(byte)0x00
     };
 
-    public TestDefaultDataLabelTextPropertiesRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         DefaultDataLabelTextPropertiesRecord record = new DefaultDataLabelTextPropertiesRecord(new TestcaseRecordInputStream((short)0x1024, (short)data.length, data));
         assertEquals( 2, record.getCategoryDataType());
 
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1024);
     }
 
     public void testStore()
index 0e48fcb4cadcebf39b62232742e2abba6137cffb..d809392420fc0e70425e28375292e3d459da3294 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -30,28 +27,15 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestEndSubRecord
-        extends TestCase
-{
+public final class TestEndSubRecord extends TestCase {
     byte[] data = new byte[] {
        
     };
 
-    public TestEndSubRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         EndSubRecord record = new EndSubRecord(new TestcaseRecordInputStream((short)0x00, (short)data.length, data));
-        
-
 
         assertEquals( 4, record.getRecordSize() );
-
-        record.validateSid((short)0x00);
     }
 
     public void testStore()
index 5bbf05be77342b604d0afd1289239ff7c6208141..3faa69ed6a8d901024d01f9b2bcc51b1eb008d53 100644 (file)
@@ -36,14 +36,7 @@ public final class TestExtendedFormatRecord extends TestCase {
                0xC0-256, 0x20  // Fill Palette 20c0
     };
 
-    public TestExtendedFormatRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         ExtendedFormatRecord record = new ExtendedFormatRecord(new TestcaseRecordInputStream((short)0xe0, (short)data.length, data));
         assertEquals(0, record.getFontIndex());
         assertEquals(0, record.getFormatIndex());
@@ -56,7 +49,6 @@ public final class TestExtendedFormatRecord extends TestCase {
         assertEquals(0x20c0, record.getFillPaletteOptions());
 
         assertEquals( 20 + 4, record.getRecordSize() );
-        record.validateSid((short)0xe0);
     }
 
     public void testStore()
index 7f59031d9e12ae17fbc46382fdc8056ffcd8541d..acd5a361ad454f211d77c8cdc84ced7c3b2735a7 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -29,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestFontBasisRecord
-        extends TestCase
-{
+public final class TestFontBasisRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x28,(byte)0x1A,   // x basis
         (byte)0x9C,(byte)0x0F,   // y basis
@@ -40,14 +37,7 @@ public class TestFontBasisRecord
         (byte)0x05,(byte)0x00    // index to font table
     };
 
-    public TestFontBasisRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         FontBasisRecord record = new FontBasisRecord(new TestcaseRecordInputStream((short)0x1060, (short)data.length, data));
         assertEquals( 0x1a28, record.getXBasis());
@@ -56,10 +46,7 @@ public class TestFontBasisRecord
         assertEquals( 0x00, record.getScale());
         assertEquals( 0x05, record.getIndexToFontTable());
 
-
         assertEquals( 14, record.getRecordSize() );
-
-        record.validateSid((short)0x1060);
     }
 
     public void testStore()
index 9509f2662b4f5baa52920d8731a1ba76bc590a0d..49cfe4d65019b256f4290c270efb3d2ab71af23e 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,29 +28,17 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestFontIndexRecord
-        extends TestCase
-{
+public final class TestFontIndexRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x05,(byte)0x00
     };
 
-    public TestFontIndexRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         FontIndexRecord record = new FontIndexRecord(new TestcaseRecordInputStream((short)0x1026, (short)data.length, data));
         assertEquals( 5, record.getFontIndex());
 
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1026);
     }
 
     public void testStore()
index d5051e99150d101dcb88998edc892570734f9fb6..19449c2f56bbecf4de4909fabb18b19038b822eb 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -26,9 +25,7 @@ import junit.framework.TestCase;
  * class works correctly.  Test data taken directly from a real
  * Excel file.
  */
-public class TestFontRecord
-        extends TestCase
-{
+public final class TestFontRecord extends TestCase {
        byte[] header = new byte[] {
                0x31, 00, 0x1a, 00, // sid=31, 26 bytes long
        };
@@ -45,14 +42,7 @@ public class TestFontRecord
                0x00, 0x61, 0x00, 0x6C, 0x00
     };
 
-    public TestFontRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         FontRecord record = new FontRecord(new TestcaseRecordInputStream((short)0x31, (short)data.length, data));
         assertEquals( 0xc8, record.getFontHeight());
@@ -72,7 +62,6 @@ public class TestFontRecord
 
 
         assertEquals( 26 + 4, record.getRecordSize() );
-        record.validateSid((short)0x31);
     }
 
     public void testStore()
index 1ad27bdc30bd2b1b0f1853cc37ea6c2fe184c360..f35a04b48836df92a9e4bc3d5f15b9d90b727b30 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,22 +28,13 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestFrameRecord
-        extends TestCase
-{
+public final class TestFrameRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00,      // border type
         (byte)0x02,(byte)0x00       // options
     };
 
-    public TestFrameRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         FrameRecord record = new FrameRecord(new TestcaseRecordInputStream((short)0x1032, (short)data.length, data));
         assertEquals( FrameRecord.BORDER_TYPE_REGULAR, record.getBorderType());
@@ -54,10 +42,7 @@ public class TestFrameRecord
         assertEquals( false, record.isAutoSize() );
         assertEquals( true, record.isAutoPosition() );
 
-
         assertEquals( 8, record.getRecordSize() );
-
-        record.validateSid((short)0x1032);
     }
 
     public void testStore()
index be4ab5731f27cb50623b3d710551ad10b1682dca..aaee94fe8215cd79505f539d8ad5562afa442c38 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,21 +28,12 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver (acoliver at apache.org)
  */
-public class TestLegendRecord
-        extends TestCase
-{
+public class TestLegendRecord extends TestCase {
     byte[] data = new byte[] {
        (byte)0x76,(byte)0x0E,(byte)0x00,(byte)0x00,(byte)0x86,(byte)0x07,(byte)0x00,(byte)0x00,(byte)0x19,(byte)0x01,(byte)0x00,(byte)0x00,(byte)0x8B,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x03,(byte)0x01,(byte)0x1F,(byte)0x00
     };
 
-    public TestLegendRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         LegendRecord record = new LegendRecord(new TestcaseRecordInputStream((short)0x1015, (short)data.length, data));
         
 
@@ -71,8 +59,6 @@ public class TestLegendRecord
 
 
         assertEquals( 24, record.getRecordSize() );
-
-        record.validateSid((short)0x1015);
     }
 
     public void testStore()
index d54f8622059fb3a0fc5e27f5161057505c9fd9f0..f4e833dd7c11d6a8646fc5a0f9456fba1c482b87 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestLineFormatRecord
-        extends TestCase
-{
+public class TestLineFormatRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,    // colour
         (byte)0x00,(byte)0x00,                          // pattern
@@ -42,14 +37,7 @@ public class TestLineFormatRecord
         (byte)0x4D,(byte)0x00                           // index
     };
 
-    public TestLineFormatRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         LineFormatRecord record = new LineFormatRecord(new TestcaseRecordInputStream((short)0x1007, (short)data.length, data));
         assertEquals( 0, record.getLineColor());
         assertEquals( 0, record.getLinePattern());
@@ -59,10 +47,7 @@ public class TestLineFormatRecord
         assertEquals( false, record.isDrawTicks() );
         assertEquals( 0x4d, record.getColourPaletteIndex());
 
-
         assertEquals( 16, record.getRecordSize() );
-
-        record.validateSid((short)0x1007);
     }
 
     public void testStore()
index c7ef2acc1c8791344d049df6c6cdeb8781af63b7..641f1e8c54a50a1843187ca2c1fc43df17b49c1c 100644 (file)
@@ -171,9 +171,6 @@ recordid = 0x1051, size =8
         assertEquals(ptgExpected.toString(),  ptgActual.toString());
 
         assertEquals( data.length + 4, record.getRecordSize() );
-
-        record.validateSid((short)0x1051);
-
     }
 
     public void testStore() {
index 2f967b66e65729c23d1670f44160b762993e8124..063f34e0c7155f32292ffd2eb5446aca7b62e73b 100644 (file)
@@ -39,19 +39,11 @@ public class TestNoteRecord
         0x6E, 0x64, 0x61, 0x74, 0x69, 0x6F, 0x6E, 0x00
     };
 
-    public TestNoteRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testRead()
-            throws Exception
-    {
+    public void testRead() {
 
         NoteRecord record = new NoteRecord(new TestcaseRecordInputStream(NoteRecord.sid, (short)data.length, data));
 
         assertEquals(NoteRecord.sid, record.getSid());
-        record.validateSid(NoteRecord.sid);
         assertEquals(6, record.getRow());
         assertEquals(1, record.getColumn());
         assertEquals(NoteRecord.NOTE_VISIBLE, record.getFlags());
@@ -60,11 +52,9 @@ public class TestNoteRecord
 
     }
 
-    public void testWrite()
-    {
+    public void testWrite() {
         NoteRecord record = new NoteRecord();
         assertEquals(NoteRecord.sid, record.getSid());
-        record.validateSid(NoteRecord.sid);
         
         record.setRow((short)6);
         record.setColumn((short)1);
index d6b5a64766c3b71461e18a469726616b55a5db2b..014a5be2baec66f14f409155ff2b25a068309e48 100644 (file)
@@ -14,6 +14,7 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
+
 package org.apache.poi.hssf.record;
 
 
@@ -28,37 +29,24 @@ import java.util.Arrays;
  *
  * @author Yegor Kozlov
  */
-public class TestNoteStructureSubRecord
-        extends TestCase
-{
+public final class TestNoteStructureSubRecord extends TestCase {
     private byte[] data = new byte[] {
         0x00, 0x00, 0x00, 0x00, 0x00, 0x00, (byte)0x80, 0x00, 0x00, 0x00,
         0x00, 0x00, (byte)0xBF, 0x00, 0x00, 0x00, 0x00, 0x00, (byte)0x81, 0x01,
         (byte)0xCC, (byte)0xEC
     };
 
-    public TestNoteStructureSubRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testRead()
-            throws Exception
-    {
+    public void testRead() {
 
         NoteStructureSubRecord record = new NoteStructureSubRecord(new TestcaseRecordInputStream(NoteStructureSubRecord.sid, (short)data.length, data));
 
         assertEquals(NoteStructureSubRecord.sid, record.getSid());
-        record.validateSid(NoteStructureSubRecord.sid);
         assertEquals(data.length + 4, record.getRecordSize());
-
     }
 
-    public void testWrite()
-    {
+    public void testWrite() {
         NoteStructureSubRecord record = new NoteStructureSubRecord();
         assertEquals(NoteStructureSubRecord.sid, record.getSid());
-        record.validateSid(NoteStructureSubRecord.sid);
         assertEquals(data.length + 4, record.getRecordSize());
 
         byte [] ser = record.serialize();
index e891a184131e2e6f31b4c74124e8ed4df7df9dfc..1ee74f2cc70b49dba0506ba0d38cfc828fb70719 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,29 +28,17 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestNumberFormatIndexRecord
-        extends TestCase
-{
+public final class TestNumberFormatIndexRecord extends TestCase {
     byte[] data = new byte[] {
         0x05,0x00
     };
 
-    public TestNumberFormatIndexRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         NumberFormatIndexRecord record = new NumberFormatIndexRecord(new TestcaseRecordInputStream((short)0x104e, (short)data.length, data));
         assertEquals( 5, record.getFormatIndex());
 
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x104e);
     }
 
     public void testStore()
index deaef1a040e0dfbee456e854bfad5a61d1b4d4e9..e9d536df0ac656f633713ad603729eb4a45bc170 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,46 +28,26 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver (acoliver at apache.org)
  */
-public class TestObjectLinkRecord
-        extends TestCase
-{
+public final class TestObjectLinkRecord extends TestCase {
     byte[] data = new byte[] {
        (byte)0x03,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00
     };
 
-    public TestObjectLinkRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         ObjectLinkRecord record = new ObjectLinkRecord(new TestcaseRecordInputStream((short)0x1027, (short)data.length, data));
         
-
         assertEquals( (short)3, record.getAnchorId());
-
         assertEquals( (short)0x00, record.getLink1());
-
         assertEquals( (short)0x00, record.getLink2());
 
-
         assertEquals( 10, record.getRecordSize() );
-
-        record.validateSid((short)0x1027);
     }
 
-    public void testStore()
-    {
+    public void testStore() {
         ObjectLinkRecord record = new ObjectLinkRecord();
 
-
-
         record.setAnchorId( (short)3 );
-
         record.setLink1( (short)0x00 );
-
         record.setLink2( (short)0x00 );
 
 
index a0d0cf841f0511144880b0e7b433e3b2b71245ce..aa8004791302ad485b7008b9ff771c56214d4037 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -30,9 +27,7 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestPaneRecord
-        extends TestCase
-{
+public final class TestPaneRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x01, (byte)0x00,
         (byte)0x02, (byte)0x00,
@@ -41,16 +36,8 @@ public class TestPaneRecord
         (byte)0x02, (byte)0x00,
     };
 
-    public TestPaneRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         PaneRecord record = new PaneRecord(new TestcaseRecordInputStream((short)0x41, (short)data.length, data));
-        
 
         assertEquals( (short)1, record.getX());
         assertEquals( (short)2, record.getY());
@@ -59,8 +46,6 @@ public class TestPaneRecord
         assertEquals( PaneRecord.ACTIVE_PANE_LOWER_LEFT, record.getActivePane());
 
         assertEquals( 14, record.getRecordSize() );
-
-        record.validateSid((short)0x41);
     }
 
     public void testStore()
index 01848870d32bc2c149e55078c4844ef1bd056ba6..aa5941a49a64d29e291e0677cdebd72cdd07bf28 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,37 +28,20 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver (acoliver at apache.org)
  */
-public class TestPlotAreaRecord
-        extends TestCase
-{
+public final class TestPlotAreaRecord extends TestCase {
     byte[] data = new byte[] {
        
     };
 
-    public TestPlotAreaRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         PlotAreaRecord record = new PlotAreaRecord(new TestcaseRecordInputStream((short)0x1035, (short)data.length, data));
-        
-
 
         assertEquals( 4, record.getRecordSize() );
-
-        record.validateSid((short)0x1035);
     }
 
-    public void testStore()
-    {
+    public void testStore() {
         PlotAreaRecord record = new PlotAreaRecord();
 
-
-
-
         byte [] recordBytes = record.serialize();
         assertEquals(recordBytes.length - 4, data.length);
         for (int i = 0; i < data.length; i++)
index e01e5bca54ffbf5820040e6ddad2da64b3444d0a..2897d8c33252e47e7bf54f2684fb99f3dab50b32 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -30,31 +27,19 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestPlotGrowthRecord
-        extends TestCase
-{
+public final class TestPlotGrowthRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00,    // horizontal
         (byte)0x00,(byte)0x00,(byte)0x01,(byte)0x00     // vertical
     };
 
-    public TestPlotGrowthRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         PlotGrowthRecord record = new PlotGrowthRecord(new TestcaseRecordInputStream((short)0x1064, (short)data.length, data));
         assertEquals( 65536, record.getHorizontalScale());
         assertEquals( 65536, record.getVerticalScale());
 
-
         assertEquals( 12, record.getRecordSize() );
-
-        record.validateSid((short)0x1064);
     }
 
     public void testStore()
index 9f2a579774d885a9cccd62b86fb28467493be5d6..9b98dab1f658231ab091685f4c606e94d0f60055 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,29 +28,17 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver (acoliver at apache.org)
  */
-public class TestSCLRecord
-        extends TestCase
-{
+public final class TestSCLRecord extends TestCase {
     byte[] data = new byte[] {
       (byte)0x3,(byte)0x0,(byte)0x4,(byte)0x0
     };
 
-    public TestSCLRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         SCLRecord record = new SCLRecord(new TestcaseRecordInputStream((short)0xa0, (short)data.length, data));
         assertEquals( 3, record.getNumerator());
         assertEquals( 4, record.getDenominator());
 
-
         assertEquals( 8, record.getRecordSize() );
-
-        record.validateSid((short)0xa0);
     }
 
     public void testStore()
index ff541fa9d2d7c9cc2a9dcd1e4c50d6f107f2ba16..1f30f26ec0c8bf069d751e94f3d24b9980af55a4 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -30,29 +27,16 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestSeriesChartGroupIndexRecord
-        extends TestCase
-{
+public final class TestSeriesChartGroupIndexRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00
     };
 
-    public TestSeriesChartGroupIndexRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         SeriesChartGroupIndexRecord record = new SeriesChartGroupIndexRecord(new TestcaseRecordInputStream((short)0x1045, (short)data.length, data));
         assertEquals( 0, record.getChartGroupIndex());
 
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1045);
-
     }
 
     public void testStore()
index f671ed9b51f4b088684b9ff195099ca74f7a3394..b971128c998a16a8109a22c2627609d2a63bce5d 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,30 +28,16 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver (acoliver at apache.org)
  */
-public class TestSeriesIndexRecord
-        extends TestCase
-{
+public final class TestSeriesIndexRecord extends TestCase {
     byte[] data = new byte[] {
        (byte)0x03,(byte)0x00
     };
 
-    public TestSeriesIndexRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         SeriesIndexRecord record = new SeriesIndexRecord(new TestcaseRecordInputStream((short)0x1065, (short)data.length, data));
         
-
         assertEquals( (short)3, record.getIndex());
-
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1065);
     }
 
     public void testStore()
index 03550cd3d868b540b50c8ee28744f97c80046179..265583655da475341c90908b22f0a5bf42eccc2d 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,21 +28,12 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestSeriesLabelsRecord
-        extends TestCase
-{
+public final class TestSeriesLabelsRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x03,(byte)0x00
     };
 
-    public TestSeriesLabelsRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         SeriesLabelsRecord record = new SeriesLabelsRecord(new TestcaseRecordInputStream((short)0x100c, (short)data.length, data));
         assertEquals( 3, record.getFormatFlags());
         assertEquals( true, record.isShowActual() );
@@ -57,8 +45,6 @@ public class TestSeriesLabelsRecord
 
 
         assertEquals( 2+4, record.getRecordSize() );
-
-        record.validateSid((short)0x100c);
     }
 
     public void testStore()
index 73be41bdc50b2b62bf700b7663079ce4803355c4..7e81f2928061e3e854406ebf4c109e3295aa331f 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,21 +28,12 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestSeriesListRecord
-        extends TestCase
-{
+public final class TestSeriesListRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x02,(byte)0x00,(byte)0x01,(byte)0x20,(byte)0xff,(byte)0xf0
     };
 
-    public TestSeriesListRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         SeriesListRecord record = new SeriesListRecord(new TestcaseRecordInputStream((short)0x1016, (short)data.length, data));
         assertEquals( (short)0x2001, record.getSeriesNumbers()[0]);
@@ -53,8 +41,6 @@ public class TestSeriesListRecord
         assertEquals( 2, record.getSeriesNumbers().length);
 
         assertEquals( 4 + 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1016);
     }
 
     public void testStore()
index a01f95966b5f824c8b808b79732bf6af41f06561..69232754fb8e857ce7d58cc7f697db01d7e592c7 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -30,9 +27,7 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestSeriesRecord
-        extends TestCase
-{
+public final class TestSeriesRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x01,(byte)0x00,      // category data type
         (byte)0x01,(byte)0x00,      // values data type
@@ -42,14 +37,7 @@ public class TestSeriesRecord
         (byte)0x00,(byte)0x00       // num bubble values
     };
 
-    public TestSeriesRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         SeriesRecord record = new SeriesRecord(new TestcaseRecordInputStream((short)0x1003, (short)data.length, data));
         assertEquals( SeriesRecord.CATEGORY_DATA_TYPE_NUMERIC, record.getCategoryDataType());
@@ -61,8 +49,6 @@ public class TestSeriesRecord
 
 
         assertEquals( 16, record.getRecordSize() );
-
-        record.validateSid((short)0x1003);
     }
 
     public void testStore()
index 5ad41870aad49dcdbe4298a94428c930975afdc1..08e4189afbc6f52ebdbf5051fcd204fddc720e87 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,50 +28,29 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver (acoliver at apache.org)
  */
-public class TestSeriesTextRecord
-        extends TestCase
-{
+public final class TestSeriesTextRecord extends TestCase {
     byte[] data = new byte[] {
        (byte)0x00,(byte)0x00,(byte)0x0C,(byte)0x01,(byte)0x56,(byte)0x00,(byte)0x61,(byte)0x00,(byte)0x6C,(byte)0x00,(byte)0x75,(byte)0x00,(byte)0x65,(byte)0x00,(byte)0x20,(byte)0x00,(byte)0x4E,(byte)0x00,(byte)0x75,(byte)0x00,(byte)0x6D,(byte)0x00,(byte)0x62,(byte)0x00,(byte)0x65,(byte)0x00,(byte)0x72,(byte)0x00
     };
 
-    public TestSeriesTextRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         SeriesTextRecord record = new SeriesTextRecord(new TestcaseRecordInputStream((short)0x100d, (short)data.length, data));
-        
 
         assertEquals( (short)0, record.getId());
-
         assertEquals( (byte)0x0C, record.getTextLength());
-
         assertEquals( (byte)0x01, record.getUndocumented());
-
         assertEquals( "Value Number", record.getText());
 
-
         assertEquals( 32, record.getRecordSize() );
-
-        record.validateSid((short)0x100d);
     }
 
     public void testStore()
     {
         SeriesTextRecord record = new SeriesTextRecord();
 
-
-
         record.setId( (short)0 );
-
         record.setTextLength( (byte)0x0C );
-
         record.setUndocumented( (byte)0x01 );
-
         record.setText( "Value Number" );
 
 
index 55e0b9467495efd35121dd32df058522500f5b65..4f20e0e696ab6ee16ec685662c7b4075fb1ae84b 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,28 +28,16 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver (acoliver at apache.org)
  */
-public class TestSeriesToChartGroupRecord
-        extends TestCase
-{
+public final class TestSeriesToChartGroupRecord extends TestCase {
     byte[] data = new byte[] {
        (byte)0x0, (byte)0x0
     };
 
-    public TestSeriesToChartGroupRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         SeriesToChartGroupRecord record = new SeriesToChartGroupRecord(new TestcaseRecordInputStream((short)0x1045, (short)data.length, data));
         assertEquals( 0x0, record.getChartGroupIndex());
 
-
         assertEquals( 0x6, record.getRecordSize() );
-
-        record.validateSid((short)0x1045);
     }
 
     public void testStore()
index 9fd260c6a407767923fcabb907dbc082e8cec749..932ff6eace66206e19131f8065e6c76b1cee093a 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,23 +28,14 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestSheetPropertiesRecord
-        extends TestCase
-{
+public final class TestSheetPropertiesRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x0A,(byte)0x00,
         (byte)0x00
         //,(byte)0x00       // not sure where that last byte comes from
     };
 
-    public TestSheetPropertiesRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         SheetPropertiesRecord record = new SheetPropertiesRecord(new TestcaseRecordInputStream((short)0x1044, (short)data.length, data));
         assertEquals( 10, record.getFlags());
         assertEquals( false, record.isChartTypeManuallyFormatted() );
@@ -59,8 +47,6 @@ public class TestSheetPropertiesRecord
 
 
         assertEquals( 7, record.getRecordSize() );
-
-        record.validateSid((short)0x1044);
     }
 
     public void testStore()
index b383c77e86f0f1053623da239a3754d1feb25d67..7f06c54ab3210b0d1ed397681d9e05678dce5592 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -30,9 +27,7 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestStringRecord
-        extends TestCase
-{
+public class TestStringRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x0B,(byte)0x00,   // length
         (byte)0x00,              // option
@@ -40,21 +35,12 @@ public class TestStringRecord
         (byte)0x46,(byte)0x61,(byte)0x68,(byte)0x72,(byte)0x7A,(byte)0x65,(byte)0x75,(byte)0x67,(byte)0x74,(byte)0x79,(byte)0x70
     };
 
-    public TestStringRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         StringRecord record = new StringRecord(new TestcaseRecordInputStream((short)0x207, (short)data.length, data));
         assertEquals( "Fahrzeugtyp", record.getString());
 
         assertEquals( 18, record.getRecordSize() );
-
-        record.validateSid((short)0x207);
     }
 
     public void testStore()
index e2a61dde79b32adc19fef584b5f2526903483adc..5506f6abdf67dde521d7ac864c7cf1f38e12cdd4 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -49,11 +46,6 @@ public final class TestSupBookRecord extends TestCase {
                 (byte)'S', (byte)'h', (byte)'e', (byte)'e', (byte)'t', (byte)'2', 
    };
 
-    public TestSupBookRecord(String name)
-    {
-        super(name);
-    }
-
     /**
      * tests that we can load the record
      */
@@ -64,8 +56,6 @@ public final class TestSupBookRecord extends TestCase {
         assertEquals( 0x4, record.getNumberOfSheets() );    //expected # of sheets
 
         assertEquals( 8, record.getRecordSize() );  //sid+size+data
-
-        record.validateSid((short)0x01AE);
     }
     /**
      * tests that we can load the record
@@ -83,8 +73,6 @@ public final class TestSupBookRecord extends TestCase {
         assertEquals(2, sheetNames.length);
         assertEquals("Sheet1", sheetNames[0].getString());
         assertEquals("Sheet2", sheetNames[1].getString());
-
-        record.validateSid((short)0x01AE);
     }
     
     /**
@@ -96,7 +84,6 @@ public final class TestSupBookRecord extends TestCase {
         assertTrue( record.isAddInFunctions() );             //expected flag
         assertEquals( 0x1, record.getNumberOfSheets() );    //expected # of sheets
         assertEquals( 8, record.getRecordSize() );  //sid+size+data
-        record.validateSid((short)0x01AE);
     }
    
     /**
@@ -119,10 +106,4 @@ public final class TestSupBookRecord extends TestCase {
 
         TestcaseRecordInputStream.confirmRecordEncoding(0x01AE, dataER, record.serialize());
     }    
-    
-    public static void main(String [] args) {
-        System.out
-        .println("Testing org.apache.poi.hssf.record.SupBookRecord");
-        junit.textui.TestRunner.run(TestSupBookRecord.class);
-    }
 }
index 0f562d1456bddfbb2ecd83472c4fab02362c9681..19c91bbacb014bd2fe4fc5903cab1da98d9291c2 100644 (file)
@@ -58,7 +58,6 @@ public final class TestTableRecord extends TestCase {
                assertEquals(0, record.getColInputCol());
 
                assertEquals( 16 + 4, record.getRecordSize() );
-               record.validateSid((short)0x236);
        }
 
     public void testStore()
index b7c6367cd253f4620675421098881b8a14e94572..b9d80dbad50fdf10c67b7aca345e4510f87ce4f6 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,23 +28,14 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestTextObjectBaseRecord
-        extends TestCase
-{
+public class TestTextObjectBaseRecord extends TestCase {
     byte[] data = new byte[] {
            0x44, 0x02, 0x02, 0x00, 0x00, 0x00, 0x00, 0x00,
            0x00, 0x00, 0x02, 0x00, 0x02, 0x00, 0x00, 0x00,
         0x00, 0x00,
     };
 
-    public TestTextObjectBaseRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         TextObjectBaseRecord record = new TextObjectBaseRecord(new TestcaseRecordInputStream((short)0x1B6, (short)data.length, data));
 
 
@@ -68,8 +56,6 @@ public class TestTextObjectBaseRecord
 
 
         assertEquals( 22, record.getRecordSize() );
-
-        record.validateSid((short)0x1B6);
     }
 
     public void testStore()
index 8f16516ca97fb5a990333ab89a94840a3bded155..91458e9ddb720426c16c04063590d5468cd6f314 100644 (file)
@@ -1,4 +1,3 @@
-\r
 /* ====================================================================\r
    Licensed to the Apache Software Foundation (ASF) under one or more\r
    contributor license agreements.  See the NOTICE file distributed with\r
 \r
 package org.apache.poi.hssf.record;\r
 \r
-import junit.framework.*;\r
-\r
-import java.util.Arrays;\r
-import java.util.List;\r
 import java.io.ByteArrayInputStream;\r
+import java.util.Arrays;\r
+\r
+import junit.framework.TestCase;\r
 \r
 import org.apache.poi.hssf.usermodel.HSSFRichTextString;\r
 \r
@@ -32,7 +30,7 @@ import org.apache.poi.hssf.usermodel.HSSFRichTextString;
  *\r
  * @author Yegor Kozlov\r
  */\r
-public class TestTextObjectRecord extends TestCase {\r
+public final class TestTextObjectRecord extends TestCase {\r
 \r
     byte[] data = {(byte)0xB6, 0x01, 0x12, 0x00, 0x12, 0x02, 0x00, 0x00, 0x00, 0x00,\r
                    0x00, 0x00, 0x00, 0x00, 0x0D, 0x00, 0x08, 0x00, 0x00, 0x00, 0x00,\r
@@ -42,17 +40,13 @@ public class TestTextObjectRecord extends TestCase {
                    0x00, 0x08, 0x00, 0x0D, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 };\r
 \r
 \r
-    public void testRead()\r
-            throws Exception\r
-    {\r
-\r
+    public void testRead() {\r
 \r
         RecordInputStream is = new RecordInputStream(new ByteArrayInputStream(data));\r
         is.nextRecord();\r
         TextObjectRecord record = new TextObjectRecord(is);\r
 \r
         assertEquals(TextObjectRecord.sid, record.getSid());\r
-        record.validateSid(TextObjectRecord.sid);\r
         assertEquals(TextObjectRecord.HORIZONTAL_TEXT_ALIGNMENT_LEFT_ALIGNED, record.getHorizontalTextAlignment());\r
         assertEquals(TextObjectRecord.VERTICAL_TEXT_ALIGNMENT_TOP, record.getVerticalTextAlignment());\r
         assertEquals(TextObjectRecord.TEXT_ORIENTATION_NONE, record.getTextOrientation());\r
index 72f2dea8a9661354bca8fd50efdff093b656ce3f..40e98dff93fe4f5cb43800707961a41432f08827 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -30,9 +27,7 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestTextRecord
-        extends TestCase
-{
+public class TestTextRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x02,                                          // horiz align
         (byte)0x02,                                          // vert align
@@ -48,14 +43,7 @@ public class TestTextRecord
         (byte)0x00,(byte)0x00                                // text rotation
     };
 
-    public TestTextRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         TextRecord record = new TextRecord(new TestcaseRecordInputStream((short)0x1025, (short)data.length, data));
         assertEquals( TextRecord.HORIZONTAL_ALIGNMENT_CENTER, record.getHorizontalAlignment());
@@ -87,9 +75,6 @@ public class TestTextRecord
 
 
         assertEquals( 36, record.getRecordSize() );
-
-        record.validateSid((short)0x1025);
-
     }
 
     public void testStore()
index e34feba4f7c5d2a29b40a3d84dfff8f5c9f68763..4f2d9efd360462a8eefd007e9b93abc4483cb8b1 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
 
  * @author Andrew C. Oliver(acoliver at apache.org)
  */
-public class TestTickRecord
-        extends TestCase
-{
+public final class TestTickRecord extends TestCase {
     byte[] data = new byte[] {
        (byte)0x02, (byte)0x00, (byte)0x03, (byte)0x01, 
         (byte)0x00, (byte)0x00, (byte)0x00, (byte)0x00, (byte)0x00,
@@ -44,14 +39,7 @@ public class TestTickRecord
        (byte)0x4D, (byte)0x00, (byte)0x00, (byte)0x00
     };
 
-    public TestTickRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
         TickRecord record = new TickRecord(new TestcaseRecordInputStream((short)0x101e, (short)data.length, data));
         assertEquals( (byte)2, record.getMajorTickType());
         assertEquals( (byte)0, record.getMinorTickType());
@@ -70,8 +58,6 @@ public class TestTickRecord
 
 
         assertEquals( 34, record.getRecordSize() );
-
-        record.validateSid((short)0x101e);
     }
 
     public void testStore()
index bcf08ea54d62c12a8cbe959861780a26676a7145..c43828af485e961c36a4f0f7fb08262aa51d64f1 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,29 +28,17 @@ import junit.framework.TestCase;
 
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestUnitsRecord
-        extends TestCase
-{
+public final class TestUnitsRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00, (byte)0x00
     };
 
-    public TestUnitsRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         UnitsRecord record = new UnitsRecord(new TestcaseRecordInputStream((short)0x1001, (short)data.length, data));
         assertEquals( 0, record.getUnits());
 
-
         assertEquals( 6, record.getRecordSize() );
-
-        record.validateSid((short)0x1001);
     }
 
     public void testStore()
index ad84b7fcf86f87c4637214409fbce266cefb6f17..b974bf9759b2fa645f3deedf86600dbd97cfb451 100644 (file)
@@ -1,4 +1,3 @@
-
 /* ====================================================================
    Licensed to the Apache Software Foundation (ASF) under one or more
    contributor license agreements.  See the NOTICE file distributed with
@@ -15,8 +14,6 @@
    See the License for the specific language governing permissions and
    limitations under the License.
 ==================================================================== */
-        
-
 
 package org.apache.poi.hssf.record;
 
@@ -31,9 +28,7 @@ import junit.framework.TestCase;
  *
  * @author Glen Stampoultzis (glens at apache.org)
  */
-public class TestValueRangeRecord
-        extends TestCase
-{
+public final class TestValueRangeRecord extends TestCase {
     byte[] data = new byte[] {
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,     // min axis value
         (byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,(byte)0x00,     // max axis value
@@ -43,14 +38,7 @@ public class TestValueRangeRecord
         (byte)0x1F,(byte)0x01                                    // options
     };
 
-    public TestValueRangeRecord(String name)
-    {
-        super(name);
-    }
-
-    public void testLoad()
-            throws Exception
-    {
+    public void testLoad() {
 
         ValueRangeRecord record = new ValueRangeRecord(new TestcaseRecordInputStream((short)0x101f, (short)data.length, data));
         assertEquals( 0.0, record.getMinimumAxisValue(), 0.001);
@@ -70,8 +58,6 @@ public class TestValueRangeRecord
         assertEquals( true, record.isReserved() );
 
         assertEquals( 42+4, record.getRecordSize() );
-
-        record.validateSid((short)0x101f);
     }
 
     public void testStore()
index 38b048f71996ff26d8f3a3482bf2ee271d47679b..4757223354a93da3c48f384db57a2c6714f36537 100644 (file)
@@ -892,4 +892,23 @@ public final class TestHSSFSheet extends TestCase {
 
         //TODO: check shapeId in the cloned sheet
     }
+    
+    /**
+     * POI now (Sep 2008) allows sheet names longer than 31 chars (for other apps besides Excel).
+     * Since Excel silently truncates to 31, make sure that POI enforces uniqueness on the first
+     * 31 chars. 
+     */
+    public void testLongSheetNames() {
+        HSSFWorkbook wb = new HSSFWorkbook();
+        final String SAME_PREFIX = "A123456789B123456789C123456789"; // 30 chars
+        
+        wb.createSheet(SAME_PREFIX + "Dxxxx");
+        try {
+            wb.createSheet(SAME_PREFIX + "Dyyyy"); // identical up to the 32nd char
+            throw new AssertionFailedError("Expected exception not thrown");
+        } catch (IllegalArgumentException e) {
+            assertEquals("The workbook already contains a sheet of this name", e.getMessage());
+        }
+        wb.createSheet(SAME_PREFIX + "Exxxx"); // OK - differs in the 31st char
+    }
 }
index 6fb08f4ee30857a7000c72005e08c321f615a202..4357c57e3b2e028afc1effec77422265411521ac 100644 (file)
@@ -498,18 +498,12 @@ public final class TestHSSFWorkbook extends TestCase {
         public BadlyBehavedRecord() {
             // 
         }
-        protected void fillFields(RecordInputStream in) {
-            throw new RuntimeException("Should not be called");
-        }
         public short getSid() {
             return 0x777;
         }
         public int serialize(int offset, byte[] data) {
             return 4;
         }
-        protected void validateSid(short id) {
-            throw new RuntimeException("Should not be called");
-        }
         public int getRecordSize() {
             return 8;
         }