[Libreoffice-commits] .: 7 commits - xmerge/source

Caolán McNamara caolan at kemper.freedesktop.org
Tue Nov 2 13:38:45 PDT 2010


 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BeginningOfFile.java |    3 
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BlankCell.java       |   51 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoolErrCell.java     |   49 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoundSheet.java      |   48 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/CellValue.java       |   54 +--
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ColInfo.java         |   54 +--
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefColWidth.java     |   15 
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefRowHeight.java    |    5 
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefinedName.java     |   94 ++---
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ExtendedFormat.java  |    4 
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FloatNumber.java     |   40 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FontDescription.java |   81 ++--
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Formula.java         |   84 ++--
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/LabelCell.java       |   48 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Pane.java            |    6 
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Row.java             |   49 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Selection.java       |   55 +--
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/StringValue.java     |   37 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window1.java         |   43 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window2.java         |   41 +-
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Workbook.java        |  175 +++++-----
 xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Worksheet.java       |  110 +++---
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/dom/DOMDocument.java                        |    2 
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/PalmDocument.java                      |    2 
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java                            |    4 
 xmerge/source/xmerge/java/org/openoffice/xmerge/test/ConverterInfoList.java                           |    2 
 xmerge/source/xmerge/java/org/openoffice/xmerge/util/ColourConverter.java                             |    4 
 xmerge/source/xmerge/java/org/openoffice/xmerge/util/Debug.java                                       |    4 
 xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java                   |   52 +-
 29 files changed, 596 insertions(+), 620 deletions(-)

New commits:
commit dde74cc65dba504fef86e3df0ced3481d6d0656a
Author: Nigel Hawkins <n.hawkins at gmx.com>
Date:   Tue Nov 2 13:20:32 2010 +0000

    Fix javadoc comment in filters/xmerge - DOMDocument.java

diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/dom/DOMDocument.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/dom/DOMDocument.java
index e023584..fe417fd 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/dom/DOMDocument.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/dom/DOMDocument.java
@@ -157,7 +157,7 @@ public class DOMDocument
      *  Sets the Content of the <code>Document</code> to the contents of the 
      *  supplied <code>Node</code> list. 
      *
-     *  @return  DOM <code>Document</code> object.
+     *  @param newDom DOM <code>Document</code> object.
      */
     public void setContentDOM( Node newDom) {
     contentDoc=(Document)newDom;
commit e3e06c3b82cf093b48a3dfec327565d5c1120803
Author: Nigel Hawkins <n.hawkins at gmx.com>
Date:   Tue Nov 2 13:18:53 2010 +0000

    Fix javadoc comments in filters/xmerge/palm

diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/PalmDocument.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/PalmDocument.java
index c60d1dc..8712753 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/PalmDocument.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/PalmDocument.java
@@ -120,7 +120,7 @@ public class PalmDocument
     /**
      *  Writes the <code>PalmDocument</code> to an <code>OutputStream</code>.
      *
-     *  @param  is  The <code>OutputStream</code> to write the content.
+     *  @param  os  The <code>OutputStream</code> to write the content.
      *
      *  @throws  IOException  If any I/O error occurs.
      */
diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java
index 4e2b126..b85caf3 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/palm/Record.java
@@ -138,7 +138,7 @@ public final class Record {
      *  <code>Record</code> length followed by the data in this
      *  <code>Record</code> object.
      *
-     *  @param  out  The <code>OutputStream</code> to write the object.
+     *  @param  outs  The <code>OutputStream</code> to write the object.
      *
      *  @throws  IOException  If any I/O error occurs.
      */
@@ -155,7 +155,7 @@ public final class Record {
      *  Read the necessary data to create a PDB from
      *  the <code>InputStream</code>.
      *
-     *  @param  in  The <code>InputStream</code> to read data from
+     *  @param  ins  The <code>InputStream</code> to read data from
      *              in order to restore the <code>object</code>.
      *
      *  @throws  IOException  If any I/O error occurs.
commit 0642d23e6367130b365221a5e871a82bdfc6aa36
Author: Nigel Hawkins <n.hawkins at gmx.com>
Date:   Tue Nov 2 13:16:13 2010 +0000

    Fix javadoc comments in filters/xmerge - Debug.java

diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/Debug.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/Debug.java
index ff89544..273b659 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/Debug.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/Debug.java
@@ -269,7 +269,7 @@ public final class Debug {
      *  <p>This logs the message during runtime if debug.info or debug.trace
      *  in the properties file is set to true.</p>
      *
-     *  @param   int   Log type, one of the Debug constants
+     *  @param   flag  Log type, one of the Debug constants
      *                 {@link #INFO}, {@link #TRACE}, {@link #ERROR}
      *                 or a combination of which or'ed together.
      *  @param   msg   The message.
@@ -291,7 +291,7 @@ public final class Debug {
      *  of the exception passed in.  Refer to the other log method
      *  for description.
      *
-     *  @param   int   Log type, one of the Debug constants
+     *  @param   flag  Log type, one of the Debug constants
      *                 {@link #INFO}, {@link #TRACE}, {@link #ERROR}
      *                 or a combination of which or'ed together.
      *  @param   msg   The message.
commit 394ff0b79404d6268cdd6a87c159cd9d25bdfbd0
Author: Nigel Hawkins <n.hawkins at gmx.com>
Date:   Tue Nov 2 13:14:34 2010 +0000

    Fix javadoc comments in filters/xmerge - ColourConverter.java

diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/ColourConverter.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/ColourConverter.java
index 525ecb4..ab72f2e 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/ColourConverter.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/ColourConverter.java
@@ -112,7 +112,7 @@ public class ColourConverter {
      * cases where the 16 colour values are something other than there default
      * values (e.g. in the case of pocket Excel)
      * 
-     * @param short[] a 16 bit array mapping the 16 colours to there values
+     * @param lookup[] a 16 bit array mapping the 16 colours to their values
      */
     public ColourConverter(short lookup[]) {
 
@@ -155,7 +155,7 @@ public class ColourConverter {
      * This method maps a Pocket Word colour index value to an RGB value as
      * used by OpenOffice.
      *
-     * @param   index   The index into Pocket Word's colour table.
+     * @param   colour   The index into Pocket Word's colour table.
      *
      * @return  A Color object representing the RGB value of the Pocket Word 
      *          colour.
commit e1a72fc8491bb7f48880d151123d207bf52a5c4d
Author: Nigel Hawkins <n.hawkins at gmx.com>
Date:   Tue Nov 2 12:59:17 2010 +0000

    Fix javadoc comments in filters/xmerge - ConverterinfoMgr.java

diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java
index 66fede9..b94a708 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -37,7 +37,7 @@ import java.io.*;
  *  manager can exist at a time.  It is final, so it may not be
  *  subclassed.
  *
- *  @author: Brian Cameron
+ *  @author Brian Cameron
  */
 public final class ConverterInfoMgr {
 
@@ -50,15 +50,13 @@ public final class ConverterInfoMgr {
        converterInfoList = new Vector();
     }
 
-
    /**
     *  Adds a converter plug-in to the registry.  The
     *  <code>ConverterInfo</code> must have a unique DisplayName
     *  and must have non-null values for DisplayName, ClassImpl,
-    *  OfficeMime, and DeviceMime.  
+    *  OfficeMime, and DeviceMime.
     *
-    *  @param  ConverterInfo  A <code>ConverterInfo</code> object
-    *                         describing a plug-in.
+    *  @param  ci  A <code>ConverterInfo</code> object describing a plug-in.
     *
     *  @throws  RegistryException  If the <code>ConverterInfo</code>
     *                              is not valid.
@@ -102,7 +100,7 @@ public final class ConverterInfoMgr {
                 throw re;
             }
         }
-       
+
         // Since this is a adding to a static Vector, make sure this
         // add method call is synchronized.
         //
@@ -110,15 +108,14 @@ public final class ConverterInfoMgr {
             converterInfoList.add(ci);
         }
     }
-   
 
    /**
-    *  Adds a <code>Vector</code> of converter plug-ins to the registry.
-    *  Each <code>ConverterInfo</code> in the <code>Vector</code> must have 
+    *  Adds a list of converter plug-ins to the registry.
+    *  Each <code>ConverterInfo</code> in the list must have
     *  a unique DisplayName and must have non-null values for DisplayName,
-    *  ClassImpl, OfficeMime, and DeviceMime.  
+    *  ClassImpl, OfficeMime, and DeviceMime.
     *
-    *  @param  ciVectory  A <code>Vector</code> of <code>ConverterInfo</code> 
+    *  @param  jarEnum  An <code>Enumeration</code> of <code>ConverterInfo</code>
     *                     objects describing one or more plug-in(s).
     *
     *  @throws  RegistryException  If a <code>ConverterInfo</code> in the
@@ -132,7 +129,6 @@ public final class ConverterInfoMgr {
         }
     }
 
-
    /**
     *  Returns an <code>Enumeration</code> of registered
     *  <code>ConverterInfo</code> objects.
@@ -145,14 +141,13 @@ public final class ConverterInfoMgr {
       return (converterInfoList.elements());
    }
 
-
    /**
     *  Removes any <code>ConverterInfo</code> object from the registry
     *  that have the specified jar name value.
     *
     *  @param  jar  The name of the jarfile.
     *
-    *  @return  True if a <code>ConverterInfo</code> object was 
+    *  @return  True if a <code>ConverterInfo</code> object was
     *           removed, false otherwise.
     */
    public static boolean removeByJar(String jar) {
@@ -183,14 +178,13 @@ public final class ConverterInfoMgr {
         return rc;
     }
 
-
    /**
     *  Removes any <code>ConverterInfo</code> object from the registry
     *  that have the specified display name value.
     *
     *  @param  name  The display name.
     *
-    *  @return  True if a <code>ConverterInfo</code> object was 
+    *  @return  True if a <code>ConverterInfo</code> object was
     *           removed, false otherwise.
     */
    public static boolean removeByName(String name) {
@@ -210,7 +204,6 @@ public final class ConverterInfoMgr {
         return rc;
     }
 
-
    /**
     *  Returns the <code>ConverterInfo</code> object that supports
     *  the specified device/office mime type conversion.  If there
@@ -220,7 +213,7 @@ public final class ConverterInfoMgr {
     *  @param  deviceMime  The device mime.
     *  @param  officeMime  The office mime.
     *
-    *  @return  The first plug-in that supports the specified 
+    *  @return  The first plug-in that supports the specified
     *           conversion.
     */
     public static ConverterInfo findConverterInfo(String deviceMime, String officeMime) {
@@ -254,15 +247,14 @@ public final class ConverterInfoMgr {
         return null;
     }
 
-
    /**
     *  Returns an array of two <code>ConverterInfo</code> objects that
     *  can be chained to perform the specified mime type conversion.
     *  If there are multiple <code>ConverterInfo</code> objects that
     *  support this conversion, only the first is returned.
     *
-    *  @param  deviceMimeFrom  The device from mime.
-    *  @param  deviceMimeTo    The device to mime.
+    *  @param  deviceFromMime  The device from mime.
+    *  @param  deviceToMime    The device to mime.
     *
     *  @return  An array of two <code>ConverterInfo</code> objects
     *           that can be chained to perform the specified
@@ -323,11 +315,10 @@ public final class ConverterInfoMgr {
         return null;
     }
 
-
     /**
-     *  main to let the user specify what plug-ins to register from 
+     *  main to let the user specify what plug-ins to register from
      *  jarfiles and to display the currently registered plug-ins.
-     * 
+     *
      *  @param  args  Not used.
      */
     public static void main(String args[]) {
@@ -360,7 +351,7 @@ public final class ConverterInfoMgr {
             }
 
             System.out.println("");
- 
+
             // Quit
             //
             if (c == 'Q') {
@@ -392,7 +383,7 @@ public final class ConverterInfoMgr {
                     }
                 }
 
-            // Unload by Display Name or Jarfile 
+            // Unload by Display Name or Jarfile
             //
         } else if (c == 'T') {
         if (validate== true){
@@ -426,7 +417,7 @@ public final class ConverterInfoMgr {
                     } else {
                         System.out.println("Remove failed.");
                     }
-             
+
                 } catch (Exception e) {
                     System.out.println("Error removing value from registry");
                     System.out.println("Error msg: " + e.getMessage());
@@ -451,7 +442,7 @@ public final class ConverterInfoMgr {
                     System.out.println("Error adding data to registry");
                     System.out.println("Error msg: " + e.getMessage());
                 }
-            
+
                 if (c == 'F') {
                     System.out.println("Enter office mime: ");
                 } else {
@@ -506,7 +497,7 @@ public final class ConverterInfoMgr {
                     int feCnt = 1;
                     while (fromEnum.hasMoreElements())
                     {
-                        System.out.println("    DeviceMime   : (#" + feCnt + ") : " + 
+                        System.out.println("    DeviceMime   : (#" + feCnt + ") : " +
                             (String)fromEnum.nextElement());
                         feCnt++;
                     }
@@ -533,4 +524,3 @@ public final class ConverterInfoMgr {
         }
     }
 }
-
commit 75577473dbcde31cdbb7be3e2d2abbc78f076ccc
Author: Nigel Hawkins <n.hawkins at gmx.com>
Date:   Tue Nov 2 12:47:52 2010 +0000

    Fix javadoc comments in filters/xmerge/pexcel

diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BeginningOfFile.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BeginningOfFile.java
index a58635b..e5d74dc 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BeginningOfFile.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BeginningOfFile.java
@@ -49,8 +49,7 @@ public class BeginningOfFile implements BIFFRecord {
    /**
     *  Constructor that initializes the member values.
     *
-    *  @param  ver	Version Number
-    *				Substream type (workbook = 0x05, worksheet = 0x10)
+    *  @param  global	True for a workbook, false for a worksheet
     */
     public BeginningOfFile(boolean global) {
         setVersion((short) 271);
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BlankCell.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BlankCell.java
index 767a24c..e1970b4 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BlankCell.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BlankCell.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -26,7 +26,7 @@
  ************************************************************************/
 
 package org.openoffice.xmerge.converter.xml.sxc.pexcel.records;
- 
+
 import java.io.OutputStream;
 import java.io.InputStream;
 import java.io.IOException;
@@ -36,36 +36,35 @@ import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
 /**
- * Represents a BIFF Record that describes a blank cell 
+ * Represents a BIFF Record that describes a blank cell
  */
 public class BlankCell extends CellValue {
-    
+
     /**
-     * Constructs a BlankCell <code>InputStream</code> 
+     * Constructs a BlankCell <code>InputStream</code>
      *
      * @param	is InputStream containing a BlankCell.
      */
     public BlankCell(InputStream is) throws IOException {
-       read(is); 
+       read(is);
     }
-    
+
     /**
       * Constructs a <code>BlankCell</code> using specified attributes
-     * 
-     * @param row row number
-     * @param col column number 
-     * @param cellContents contents of the cell 
-     * @param ixfe font index
-      */	 
+     *
+     * @param row     row number
+     * @param column  column number
+     * @param ixfe    font index
+      */
     public BlankCell(int row, int column, int ixfe) throws IOException {
 
         setRow(row);
            setCol(column);
         setIxfe(ixfe);
     }
-    
+
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>BlankCell</code>
      */
@@ -81,35 +80,35 @@ public class BlankCell extends CellValue {
         output.write(ixfe);
 
         Debug.log(Debug.TRACE, "Writing BlankCell record");
-    
+
     }
-    
+
     /**
-     * Reads a BlankCell <code>InputStream</code> 
+     * Reads a BlankCell <code>InputStream</code>
      *
-     * @param	is InputStream containing a BlankCell.
+     * @param	input InputStream containing a BlankCell.
      */
     public int read(InputStream input) throws IOException {
 
         int numOfBytesRead 	= input.read(rw);
-        numOfBytesRead++; 
+        numOfBytesRead++;
         col					+= input.read();
         numOfBytesRead		+= input.read(ixfe);
 
-        Debug.log(Debug.TRACE, "\tRow : "+ EndianConverter.readShort(rw) + 
+        Debug.log(Debug.TRACE, "\tRow : "+ EndianConverter.readShort(rw) +
                             " Column : " + col +
-                            " ixfe : " + EndianConverter.readShort(ixfe));        
-        
+                            " ixfe : " + EndianConverter.readShort(ixfe));
+
         return numOfBytesRead;
     }
-    
+
     /**
-     * Gets the <code>String</code> representing the cells contents 
+     * Gets the <code>String</code> representing the cells contents
      *
      * @return the <code>String</code> representing the cells contents
      */
     public String getString() throws IOException {
 
         return (new String(""));
-    }  
+    }
 }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoolErrCell.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoolErrCell.java
index f07212f..1cb78a7 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoolErrCell.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoolErrCell.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -31,26 +31,25 @@ import java.io.OutputStream;
 import java.io.InputStream;
 import java.io.IOException;
 
-
 import org.openoffice.xmerge.util.Debug;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
 /**
- * Represents a BIFF Record that describes a Boolean or Error value 
+ * Represents a BIFF Record that describes a Boolean or Error value
  */
 public class BoolErrCell extends CellValue {
 
     private byte   bBoolErr;
     private byte   fError;
-    
+
      /**
      * Constructs a BoolErrCell from arguments
       *
-     * @param row row number
-     * @param col column number 
-     * @param ixfe font index
-     * @param bBoolErr Boolean value or error value 
-     * @param fError Boolean error flag 
+     * @param row      row number
+     * @param column   column number
+     * @param ixfe     font index
+     * @param bBoolErr Boolean value or error value
+     * @param fError   Boolean error flag
       */
     public BoolErrCell(int row, int column, int ixfe, int bBoolErr, int fError) throws IOException {
 
@@ -59,35 +58,35 @@ public class BoolErrCell extends CellValue {
         this.fError = (byte)fError;
         setRow(row);
         setCol(column);
-    }   
-    
+    }
+
     /**
-     * Constructs a BoolErrCell from the <code>InputStream</code> 
+     * Constructs a BoolErrCell from the <code>InputStream</code>
       *
-      * @param	is InputStream containing a BoolErrCell  
+      * @param	is InputStream containing a BoolErrCell
       */
     public BoolErrCell(InputStream is) throws IOException {
-        read(is);	
+        read(is);
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>BoolErrCEll</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.BOOLERR_CELL;
-    } 
+    }
 
     /**
      * Writes a <code>BoolErrCell</code> to the specified <code>Outputstream</code>
      *
-     * @param os the <code>OutputStream</code> to write to  
-     */    
+     * @param output the <code>OutputStream</code> to write to
+     */
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
-        
+
         super.write(output);
 
         output.write(bBoolErr);
@@ -95,11 +94,11 @@ public class BoolErrCell extends CellValue {
 
         Debug.log(Debug.TRACE,"Writing BoolErrCell record");
     }
-       
+
     /**
-     * Reads a BoolErrCell from the <code>InputStream</code> 
+     * Reads a BoolErrCell from the <code>InputStream</code>
       *
-      * @param	is InputStream containing a BoolErrCell 
+      * @param	input InputStream containing a BoolErrCell
       */
     public int read(InputStream input) throws IOException {
 
@@ -107,15 +106,15 @@ public class BoolErrCell extends CellValue {
 
         bBoolErr			= (byte) input.read();
         fError				= (byte) input.read();
-           numOfBytesRead += 2;     
+           numOfBytesRead += 2;
 
         Debug.log(Debug.TRACE, " bBoolErr : " + bBoolErr +
-                            " fError : " + fError);        
+                            " fError : " + fError);
         return numOfBytesRead;
     }
 
     /**
-     * Gets the <code>String</code> representing the cells contents 
+     * Gets the <code>String</code> representing the cells contents
      *
      * @return the <code>String</code> representing the cells contents
      */
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoundSheet.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoundSheet.java
index 4056808..ac081af 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoundSheet.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/BoundSheet.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -36,21 +36,21 @@ import org.openoffice.xmerge.util.Debug;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
 /**
- * Represents a BoundSheet Record which describes the name of a worksheet 
+ * Represents a BoundSheet Record which describes the name of a worksheet
  */
 public class BoundSheet implements BIFFRecord {
 
     private byte	reserved;
-    private byte	cch; 
+    private byte	cch;
     private byte[]	sheetName;
-    
+
     /**
      * Constructs a pocket Excel Document assigns it the document name passed in
       *
-     * @param	name name of the worksheet represented	
+     * @param	name name of the worksheet represented
      */
     public BoundSheet(String name) throws IOException {
-        setSheetName(name);	
+        setSheetName(name);
         reserved = 0;
     }
 
@@ -61,21 +61,21 @@ public class BoundSheet implements BIFFRecord {
       * @param	is InputStream containing a Pocket Excel Data file.
       */
     public BoundSheet(InputStream is) throws IOException {
-        read(is);	
+        read(is);
     }
 
     /**
       * Sets the worksheet name. The sheetname length must be doubled as the
-     * String is stored in unicode format. 
+     * String is stored in unicode format.
       *
-      * @param	sheetname	worksheet name 
+      * @param	sheetname	worksheet name
       */
     void setSheetName(String sheetName) throws IOException {
         this.cch		= (byte) sheetName.length();
         this.sheetName	= new byte[cch*2];
         this.sheetName	= sheetName.getBytes("UTF-16LE");
     }
-    
+
     public String getSheetName() {
         String name;
 
@@ -86,21 +86,21 @@ public class BoundSheet implements BIFFRecord {
         }
         return name;
     }
- 
+
      /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>BoundSheet</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.BOUND_SHEET;
-    }   
-    
+    }
+
      /**
      * Write this particular <code>BIFFRecord</code> to the <code>OutputStream</code>
      *
-     * @param ouput the <code>OutputStream</code>
-     */    
+     * @param output the <code>OutputStream</code>
+     */
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
@@ -110,27 +110,27 @@ public class BoundSheet implements BIFFRecord {
 
         Debug.log(Debug.TRACE,"Writing BoundSheet record");
     }
-    
+
     /**
       * Reads a BoundSheet from the <code>InputStream</code> The byte array
      * must be twice the size of the String as it uses unicode.
       *
-      * @param	is InputStream containing the record data 
+      * @param	input InputStream containing the record data
       */
     public int read(InputStream input) throws IOException {
-    
+
         reserved			= (byte) input.read();
         cch					= (byte) input.read();
         int numOfBytesRead = 2;
         int strLen = cch*2;
         sheetName	= new byte[strLen];
-        numOfBytesRead		+= input.read(sheetName, 0, strLen);        
-        
-           Debug.log(Debug.TRACE,"\tReserved : "+ reserved + 
+        numOfBytesRead		+= input.read(sheetName, 0, strLen);
+
+           Debug.log(Debug.TRACE,"\tReserved : "+ reserved +
                             " cch : " + cch +
-                            " sheetName : " + new String(sheetName,"UTF-16LE"));          
+                            " sheetName : " + new String(sheetName,"UTF-16LE"));
 
         return numOfBytesRead;
     }
-    
+
 }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/CellValue.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/CellValue.java
index cdfb47e..bc4d427 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/CellValue.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/CellValue.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -39,28 +39,28 @@ public abstract class CellValue implements BIFFRecord {
     protected byte[] rw	= new byte[2];
     protected byte   col;
     protected byte[] ixfe = new byte[2];
-    
-       /**
-     * Get the row number of this cell 
+
+     /**
+     * Get the row number of this cell
      *
-     * @return the row number of this cell 
+     * @return the row number of this cell
      */
     public int getRow() {
         return EndianConverter.readShort(rw) + 1;
     }
 
     /**
-     * Set the row number of this cell 
+     * Set the row number of this cell
      *
-     * @param row sets the row number for this cell 
+     * @param row sets the row number for this cell
      */
     public void setRow(int row) {
         this.rw = EndianConverter.writeShort((short) (row - 1));
     }
        /**
-     * Get the Index to the <code>ExtendedFormat</code> 
+     * Get the Index to the <code>ExtendedFormat</code>
      *
-     * @return the index number of this cell's <code>ExtendedFormat</code> 
+     * @return the index number of this cell's <code>ExtendedFormat</code>
      */
     public int getIxfe() {
         return EndianConverter.readShort(ixfe);
@@ -76,18 +76,18 @@ public abstract class CellValue implements BIFFRecord {
     }
 
     /**
-     * Get the column number of this cell 
+     * Get the column number of this cell
      *
-     * @return the column number of this cell 
+     * @return the column number of this cell
      */
     public int getCol() {
-        return col + 1;			// The cols start at 1 
+        return col + 1;			// The cols start at 1
     }
 
     /**
-     * Set the row number of this cell 
+     * Set the column number of this cell
      *
-     * @param col sets the row number for this cell 
+     * @param col sets the column number for this cell
      */
     public void setCol(int col) {
         this.col = (byte) (col - 1);		// The cols start at 1
@@ -96,7 +96,7 @@ public abstract class CellValue implements BIFFRecord {
     /**
      * Writes basic cell value attributes to the specified <code>Outputstream</code>
      *
-     * @param os the <code>OutputStream</code> to write to  
+     * @param output the <code>OutputStream</code> to write to
      */
     public void write(OutputStream output) throws IOException {
 
@@ -106,31 +106,29 @@ public abstract class CellValue implements BIFFRecord {
     }
 
     /**
-     * Writes a<code>LabelCell</code> to the specified <code>Outputstream</code>
+     * Reads a <code>LabelCell</code> from the specified <code>Inputstream</code>
      *
-     * @param os the <code>OutputStream</code> to write to  
+     * @param input the <code>InputStream</code> to read from
      */
     public int read(InputStream input) throws IOException {
 
         int numOfBytesRead	= input.read(rw);
-        col					+= input.read();    
+        col					+= input.read();
         numOfBytesRead++;
         numOfBytesRead		+= input.read(ixfe);
-    
-        Debug.log(Debug.TRACE, "\tRow : "+ EndianConverter.readShort(rw) + 
+
+        Debug.log(Debug.TRACE, "\tRow : "+ EndianConverter.readShort(rw) +
                             " Column : " + col +
-                            " ixfe : " + EndianConverter.readShort(ixfe));        
-        
-        return numOfBytesRead;    
+                            " ixfe : " + EndianConverter.readShort(ixfe));
+
+        return numOfBytesRead;
     }
-    
-    
+
     /**
      * Returns the contents of the cell as a String
      *
-     * @return the contents of the cell 
+     * @return the contents of the cell
      */
     abstract public String getString() throws IOException;
-    
-}
 
+}
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ColInfo.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ColInfo.java
index ea6b472..97cb5ad 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ColInfo.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ColInfo.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -36,7 +36,7 @@ import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
 /**
- * ColInfo describes the formatting for a column 
+ * ColInfo describes the formatting for a column
  *
  */
 public class ColInfo implements BIFFRecord {
@@ -45,17 +45,17 @@ public class ColInfo implements BIFFRecord {
     private byte[] colLast	= new byte[2];	// last column this formatting applies to
     private byte[] colDX	= new byte[2];	// column width
     private byte[] ixfe		= new byte[2];	// index for formatting
-    private byte   grbit;					// options flags 
+    private byte   grbit;					// options flags
     private float  scale = (float) 2.5;		// 1.798;
-    
+
     /**
       * Constructs a pocket Excel Document from the
       * <code>InputStream</code> and assigns it the document name passed in
       *
-      * @param	colFirst	the first column this formatting applies to	
-      * @param	colLast		last column this formatting applies to	
-      * @param	coldx		column width 	
-      * @param	grbit		options flags	
+      * @param	colFirst	the first column this formatting applies to
+      * @param	colLast		last column this formatting applies to
+      * @param	colDX		column width
+      * @param	ixfe		index for formatting
       */
     public ColInfo(int colFirst, int colLast, int colDX, int ixfe) {
         this.colFirst	= EndianConverter.writeShort((short)colFirst);
@@ -78,7 +78,7 @@ public class ColInfo implements BIFFRecord {
     /**
      * Reads ColInfo record from the InputStream
      *
-     * @param input the InputStream to read from 
+     * @param input the InputStream to read from
      * @return the number of bytes read
      */
     public int read(InputStream input) throws IOException {
@@ -91,46 +91,46 @@ public class ColInfo implements BIFFRecord {
         numOfBytesRead		+= input.read(ixfe);
         grbit				= (byte) input.read();
         numOfBytesRead		++;
-        
-        Debug.log(Debug.TRACE,"\tcolFirst : "+ EndianConverter.readShort(colFirst) + 
+
+        Debug.log(Debug.TRACE,"\tcolFirst : "+ EndianConverter.readShort(colFirst) +
                             " colLast : " + EndianConverter.readShort(colLast) +
                             " colDX : " + EndianConverter.readShort(colDX) +
                             " ixfe : " + EndianConverter.readShort(ixfe) +
                             " grbit : " + grbit);
-        
+
         return numOfBytesRead;
     }
-    
+
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
-     * @return the hex code for <code>ColInfo</code> 
+     * @return the hex code for <code>ColInfo</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.COLINFO;
     }
     /**
-     * Get the width of this column 
+     * Get the width of this column
      *
-     * @return the width of this column 
+     * @return the width of this column
      */
     public short getColWidth() {
         return EndianConverter.readShort(colDX);
-    }	
-    
+    }
+
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the first column that this formatting applies to.
      *
-     * @return the hex code for <code>ColInfo</code> 
+     * @return The first column.
      */
     public short getFirst() {
         return EndianConverter.readShort(colFirst);
     }
-    
+
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the last column that this formatting applies to.
      *
-     * @return the hex code for <code>ColInfo</code> 
+     * @return The last column.
      */
     public short getLast() {
         return EndianConverter.readShort(colLast);
@@ -139,10 +139,10 @@ public class ColInfo implements BIFFRecord {
     /**
      * Writes a ColInfo to the specified <code>Outputstream</code>
      *
-     * @param os the <code>OutputStream</code> to write to  
+     * @param output the <code>OutputStream</code> to write to
      */
     public void write(OutputStream output) throws IOException {
-    
+
         output.write(getBiffType());
         output.write(colFirst);
         output.write(colLast);
@@ -153,5 +153,5 @@ public class ColInfo implements BIFFRecord {
         Debug.log(Debug.TRACE,"Writing ColInfo record");
 
     }
-    
+
 }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefColWidth.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefColWidth.java
index 24acb37..bb2f2af 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefColWidth.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefColWidth.java
@@ -44,18 +44,21 @@ public class DefColWidth implements BIFFRecord {
     private byte[] coldx = new byte[2];
     private byte[] ixfe  = new byte[2];
     
-/**
- * Constructs a pocket Excel Document from the
- * <code>InputStream</code> and assigns it the document name passed in
- *
- * @param	is InputStream containing a Pocket Excel Data file.
- */
+    /**
+     * Default constructor. Sets width to 9.
+     */
     public DefColWidth() {
         grbit	= new byte[] {0x00, 0x00};
         coldx	= new byte[] {0x00, 0x09};
         ixfe	= new byte[] {0x00, 0x00};
     }
 
+    /**
+     * Constructs a pocket Excel Document from the
+     * <code>InputStream</code> and assigns it the document name passed in
+     *
+     * @param	is InputStream containing a Pocket Excel Data file.
+     */
     public DefColWidth(InputStream is) throws IOException {
         read(is);
     }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefRowHeight.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefRowHeight.java
index a4cfb84..43f980f 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefRowHeight.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefRowHeight.java
@@ -44,10 +44,7 @@ public class DefRowHeight implements BIFFRecord {
     private byte[] unknown2 = new byte[2];
     
     /**
-      * Constructs a pocket Excel Document from the
-      * <code>InputStream</code> and assigns it the document name passed in
-      *
-      * @param	is InputStream containing a Pocket Excel Data file.
+      * Default constructor. Sets something unknown to 255.
       */
     public DefRowHeight() {
         unknown1 = new byte[] {(byte)0x00, (byte)0x00};
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefinedName.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefinedName.java
index 8613771..f252564 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefinedName.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/DefinedName.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -50,23 +50,24 @@ public class DefinedName implements BIFFRecord {
     private byte[] ixals	= new byte[2];
     private byte[] rgch;
     private byte[] rgce;
-    private FormulaHelper fh = new FormulaHelper();    
+    private FormulaHelper fh = new FormulaHelper();
     private String definition = new String("");
     private Workbook wb;
 
     /**
-      * Constructs a Defined Name from the <code>InputStream</code> 
+      * Constructs a Defined Name from an existing <code>NameDefinition</code>
       *
-      * @param	is InputStream containing the record data 
+      * @param	nd The name definition.
+      * @param	wb The workbook we are defined in.
       */
     public DefinedName(NameDefinition nd, Workbook wb) throws IOException {
-        
+
         fh.setWorkbook(wb);
         this.wb = wb;
         String name = nd.getName();
-        
+
         // we have to insert an = to stop the formulaParser throwing an exception
-        definition = "=" + nd.getDefinition(); 
+        definition = "=" + nd.getDefinition();
 
         cch = (byte)name.length();
         rgch = new byte[cch*2];
@@ -74,10 +75,12 @@ public class DefinedName implements BIFFRecord {
         grbit = EndianConverter.writeShort((short)0);
         ixals[0] = (byte)0xFF;ixals[1] = (byte)0xFF;
     }
+
     /**
-      * Constructs a Defined Name from the <code>InputStream</code> 
+      * Constructs a Defined Name from the <code>InputStream</code>
       *
-      * @param	is InputStream containing the record data 
+      * @param	is InputStream containing the record data
+      * @param  wb The workbook we are defined in.
       */
     public DefinedName(InputStream is, Workbook wb) throws IOException {
 
@@ -87,51 +90,49 @@ public class DefinedName implements BIFFRecord {
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>DefinedName</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.DEFINED_NAME;
     }
-    
+
     /**
       * Reads a Defined Name from the <code>InputStream</code> The byte array
      * must be twice the size of the String as it uses unicode.
       *
-      * @param	is InputStream containing the record data 
+      * @param	input InputStream containing the record data
       */
     public int read(InputStream input) throws IOException {
-        
-        int numOfBytesRead	= input.read(grbit);    
+
+        int numOfBytesRead	= input.read(grbit);
         cch				 	= (byte) input.read();
         numOfBytesRead++;
         numOfBytesRead		+= input.read(cce);
-        numOfBytesRead		+= input.read(ixals);    
-        
+        numOfBytesRead		+= input.read(ixals);
+
         rgch = new byte[cch*2];
-        input.read(rgch, 0, cch*2);        
-        
-        rgce = new byte[EndianConverter.readShort(cce)];
-        input.read(rgce, 0, EndianConverter.readShort(cce));        
-        
+        input.read(rgch, 0, cch*2);
 
+        rgce = new byte[EndianConverter.readShort(cce)];
+        input.read(rgce, 0, EndianConverter.readShort(cce));
 
-        Debug.log(Debug.TRACE, "\tgrbit : "+ EndianConverter.readShort(grbit) + 
+        Debug.log(Debug.TRACE, "\tgrbit : "+ EndianConverter.readShort(grbit) +
                             " cch : " + cch +
                             " cce : " + EndianConverter.readShort(cce) +
                             " ixals : " + EndianConverter.readShort(ixals) +
-                            "\n\trgch : " + rgch +        
+                            "\n\trgch : " + rgch +
                             " rgce : " + rgce);
-                            
+
         return numOfBytesRead;
     }
-    
+
      /**
      * Write this particular <code>BIFFRecord</code> to the <code>OutputStream</code>
      *
-     * @param ouput the <code>OutputStream</code>
-     */        
+     * @param output the <code>OutputStream</code>
+     */
     public void write(OutputStream output) throws IOException {
 
         try {
@@ -142,7 +143,6 @@ public class DefinedName implements BIFFRecord {
             Debug.log(Debug.TRACE,"Error in Parsing Name Definition");
             cce = EndianConverter.writeShort((short) 0);
         }
-        
 
         output.write(getBiffType());
         output.write(grbit);
@@ -171,56 +171,56 @@ public class DefinedName implements BIFFRecord {
         } catch (UnsupportedEncodingException e){
             name = "unknown";
         }
-        return name;	
+        return name;
     }
 
     /**
      * Returns a definition table which can be used by the pocket excel
      * decoder to build a complete definitions table for writing to the sxc
-     * document 
+     * document
      */
      public NameDefinition getNameDefinition() {
-     
+
          String baseCellAddress;
         getDefinition();		// This must be called first so we know the type
 
         baseCellAddress = "$" + wb.getSheetName(0) + ".A1";
-            
+
         NameDefinition nd = new NameDefinition(getName(),definition, baseCellAddress, isRangeType(), isExpressionType());
         return nd;
      }
-     
+
     /**
-     * Returns the definition 
+     * Returns the definition
      *
-     * @return the <code>String</code> containing the definition 
+     * @return the <code>String</code> containing the definition
      */
     private String getDefinition() {
         // pexcel sometimes creates Name definition with no defintion, bug??
-        if(EndianConverter.readShort(cce)!=0) {				
-            definition = fh.convertPXLToCalc(rgce);	
+        if(EndianConverter.readShort(cce)!=0) {
+            definition = fh.convertPXLToCalc(rgce);
             definition = definition.substring(1);	// remove the '='
             definition = definition.replace(',', ';');
-        }	
-        return definition;			
+        }
+        return definition;
     }
-    
+
     /**
-     * Returns the defintion 
+     * Returns the defintion
      *
-     * @return the <code>String</code> containing the definition 
+     * @return the <code>String</code> containing the definition
      */
     private boolean isRangeType() {
-    
-        return fh.isRangeType();			
+
+        return fh.isRangeType();
     }
     /**
-     * Returns the defintion 
+     * Returns the defintion
      *
-     * @return the <code>String</code> containing the definition 
+     * @return the <code>String</code> containing the definition
      */
     private boolean isExpressionType() {
 
-        return fh.isExpressionType();			
+        return fh.isExpressionType();
     }
 }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ExtendedFormat.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ExtendedFormat.java
index 30fdfb0..67f3ae3 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ExtendedFormat.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/ExtendedFormat.java
@@ -280,8 +280,8 @@ org.openoffice.xmerge.converter.xml.OfficeConstants {
     /**
      * Compare two ExtendedFormat to see if the font index is the same 
      *
-     * @param the ExtendedFormat to be used in the comaprison
-     * @return boolean if the two are the same otherwise false
+     * @param  rhs The ExtendedFormat to be used in the comaprison
+     * @return True if the two are the same otherwise false
      */	
     public boolean compareTo(ExtendedFormat rhs) {
 
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FloatNumber.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FloatNumber.java
index 902feef..a088bc2 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FloatNumber.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FloatNumber.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -41,10 +41,9 @@ import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 public class FloatNumber extends CellValue {
 
     protected byte[] num  = new byte[8];
-    
+
     /**
-      * Constructs a pocket Excel Document from the
-      * <code>InputStream</code> and assigns it the document name passed in
+      * Constructs a FloatNumber by reading from an InputStream.
      *
       * @param	is InputStream containing a Pocket Excel Data file.
       */
@@ -53,12 +52,12 @@ public class FloatNumber extends CellValue {
     }
 
     /**
-      * Constructs a <code>FloatNumber</code> using specified attributes 
+      * Constructs a <code>FloatNumber</code> using specified attributes
      *
-     * @param row row number
-     * @param col column number 
-     * @param cellContents contents of the cell 
-     * @param ixfe font index
+     * @param row          row number
+     * @param column       column number
+     * @param cellContents contents of the cell
+     * @param ixfe         font index
       */
     public FloatNumber(int row, int column, String cellContents, int ixfe) throws IOException {
 
@@ -70,7 +69,7 @@ public class FloatNumber extends CellValue {
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>FloatNumber</code>
      */
@@ -81,40 +80,39 @@ public class FloatNumber extends CellValue {
     /**
      * Reads a<code>FloatNumber</code> from the specified <code>InputStream</code>
      *
-     * @param input the <code>InputStram</code> to read from  
-     */    
+     * @param input the <code>InputStram</code> to read from
+     */
     public int read(InputStream input) throws IOException {
-    
+
         int numOfBytesRead = super.read(input);
 
-        numOfBytesRead += input.read(num);    
-        
+        numOfBytesRead += input.read(num);
+
         Debug.log(Debug.TRACE," num : " + getString());
         return numOfBytesRead;
     }
-    
+
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
-        
+
         super.write(output);
 
         output.write(num);
 
         Debug.log(Debug.TRACE,"Writing FloatNumber record");
     }
-   
 
     /**
-     * Gets the numerical value the cell represents 
+     * Gets the numerical value the cell represents
      *
-     * @return the <code>String</code> representing a double value 
+     * @return the <code>String</code> representing a double value
      */
     public String getString() throws IOException {
 
         double value = EndianConverter.readDouble(num);
         Double myDo = new Double(value);
-        return myDo.toString();	
+        return myDo.toString();
     }
 
 }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FontDescription.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FontDescription.java
index d555c06..21440ae 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FontDescription.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/FontDescription.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -39,9 +39,8 @@ import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.util.ColourConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
-
 /**
- * Represents a BIFF Record descibing a font used 
+ * Represents a BIFF Record descibing a font used
  */
 public class FontDescription implements BIFFRecord {
 
@@ -61,22 +60,20 @@ public class FontDescription implements BIFFRecord {
     public static final int ITALIC		= 0x02;
 
     /**
-      * Constructs a FontDescription from the bold italic and undelrine attributes  
+      * Constructs a FontDescription from a Format.
       *
-      * @param	italic		Italic attribute 	
-      * @param	bold		Bold attribute 	
-      * @param	underline	Underline attribute 	
+      * @param	fmt The <code>Format</code> to use as source.
       */
     public FontDescription(Format fmt) throws IOException {
 
         Debug.log(Debug.TRACE,"Building FontDescriptor based on Format : " + fmt);
 
-        this.dwHeight	= EndianConverter.writeShort((short) (fmt.getFontSize()*20)); 
-            
+        this.dwHeight	= EndianConverter.writeShort((short) (fmt.getFontSize()*20));
+
         grbit = new byte[] {(byte)0x00, (byte)0x00};
         bls = EndianConverter.writeShort((short) 400);
         uls = 0;
-            
+
         if (fmt.getAttribute(Format.ITALIC))
             grbit[0] |= ITALIC;
 
@@ -86,13 +83,12 @@ public class FontDescription implements BIFFRecord {
         if (fmt.getAttribute(Format.UNDERLINE))
             uls	|= UNDERLINE;
 
-
         bFamily		= 0;
         bCharSet	= 0;
 
         String fontName = fmt.getFontName();
-        if(	!fontName.equals("Tahoma") && 
-            !fontName.equals("Courier New")) { 
+        if(	!fontName.equals("Tahoma") &&
+            !fontName.equals("Courier New")) {
             // We will set our default font to be Tahoma
             fontName = new String("Tahoma");
         }
@@ -119,12 +115,12 @@ public class FontDescription implements BIFFRecord {
      * @return true if italic otherwise false
      */
     public boolean isItalic() {
-    
+
         return (EndianConverter.readShort(grbit) == 2);
     }
 
     /**
-     * Tests if this font descriptor defines underline 
+     * Tests if this font descriptor defines underline
      *
      * @return true if underline otherwise false
      */
@@ -134,7 +130,7 @@ public class FontDescription implements BIFFRecord {
     }
 
     /**
-     * Tests if this font descriptor defines bold 
+     * Tests if this font descriptor defines bold
      *
      * @return true if bold otherwise false
      */
@@ -144,7 +140,7 @@ public class FontDescription implements BIFFRecord {
     }
 
     /**
-     * Get the background color this format uses 
+     * Get the background color this format uses
      *
      * @return the background color
      */
@@ -154,21 +150,21 @@ public class FontDescription implements BIFFRecord {
         if(rgb!=0xFF) {
             ColourConverter cc = new ColourConverter(PocketExcelConstants.cLookup);
             c = cc.convertToRGB(rgb);
-        } 
+        }
         return c;
     }
-    
+
     /**
      * Compares current font descriptor against one passed in
      *
-     * @return true if attrbitues are the same 
+     * @return true if attrbitues are the same
      */
     public boolean compareTo(FontDescription rhs) {
 
         if(EndianConverter.readShort(icvFore) !=
-        EndianConverter.readShort(rhs.icvFore)) 
+        EndianConverter.readShort(rhs.icvFore))
             return false;
-            
+
         if (EndianConverter.readShort(dwHeight) !=
         EndianConverter.readShort(dwHeight))
             return false;
@@ -187,26 +183,25 @@ public class FontDescription implements BIFFRecord {
 
         return true;
     }
-    
 
     /**
-      * Constructs a Font Description from the <code>InputStream</code> 
+      * Constructs a Font Description from the <code>InputStream</code>
       *
-      * @param	is InputStream containing a <code>FontDescription</code> 
+      * @param	is InputStream containing a <code>FontDescription</code>
       */
     public FontDescription(InputStream is) throws IOException {
         read(is);
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>FontDescription</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.FONT_DESCRIPTION;
     }
-   
+
        /**
      * Get the Font size
      *
@@ -220,7 +215,7 @@ public class FontDescription implements BIFFRecord {
      *
      */
     public String getFont() {
-    
+
         String name;
 
         try {
@@ -230,43 +225,43 @@ public class FontDescription implements BIFFRecord {
         }
         return name;
     }
-   
+
     /**
-      * Constructs a Font Description from the <code>InputStream</code> 
+      * Constructs a Font Description from the <code>InputStream</code>
       *
-      * @param	is InputStream containing a <code>FontDescription</code> 
+      * @param	input InputStream containing a <code>FontDescription</code>
       */
     public int read(InputStream input) throws IOException {
 
         int numOfBytesRead	= input.read(dwHeight);
-        numOfBytesRead		+= input.read(grbit);    
-        numOfBytesRead		+= input.read(icvFore);        
+        numOfBytesRead		+= input.read(grbit);
+        numOfBytesRead		+= input.read(icvFore);
         numOfBytesRead		+= input.read(bls);
         numOfBytesRead		+= input.read(Reserved2);
         uls 				= (byte) input.read();
         bFamily				= (byte) input.read();
-        bCharSet			= (byte) input.read();    
+        bCharSet			= (byte) input.read();
         Reserved3			= (byte) input.read();
         cch					= (byte) input.read();
         numOfBytesRead += 5;
-        
+
         rgch = new byte[cch*2];
-        input.read(rgch, 0, cch*2);        
-    
-        Debug.log(Debug.TRACE,"\tdwHeight : "+ EndianConverter.readShort(dwHeight) + 
+        input.read(rgch, 0, cch*2);
+
+        Debug.log(Debug.TRACE,"\tdwHeight : "+ EndianConverter.readShort(dwHeight) +
                             " grbit : " + EndianConverter.readShort(grbit) +
                             " bls : " + EndianConverter.readShort(bls) +
                             " uls : " + uls +
-                            "\n\tFamily : " + bFamily +        
+                            "\n\tFamily : " + bFamily +
                             " bCharSet : " + bCharSet +
                             " cch : " + cch +
                             " rgch : " + new String(rgch,"UTF-16LE"));
-                            
+
         return numOfBytesRead;
     }
-    
+
     public void write(OutputStream output) throws IOException {
-            
+
         output.write(getBiffType());
         output.write(dwHeight);
         output.write(grbit);
@@ -282,5 +277,5 @@ public class FontDescription implements BIFFRecord {
 
         Debug.log(Debug.TRACE,"Writing FontDescription record");
     }
-    
+
 }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Formula.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Formula.java
index e78645a..9f5a16c 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Formula.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Formula.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -39,9 +39,8 @@ import org.openoffice.xmerge.converter.xml.sxc.pexcel.records.formula.FormulaHel
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.records.Workbook;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
-
 /**
- * Represents a BIFF Record describing a formula  
+ * Represents a BIFF Record describing a formula
  */
 public class Formula extends CellValue implements OfficeConstants {
 
@@ -50,16 +49,17 @@ public class Formula extends CellValue implements OfficeConstants {
     private byte[] cce		= new byte[2];
     private byte[] rgce;
     private FormulaHelper fh = new FormulaHelper();
-   
+
     /**
-      * Constructs a <code>Formula</code> using specified attributes
-     * 
-     * @param row row number
-     * @param col column number 
-     * @param cellContents contents of the cell 
-     * @param ixfe font index
-     * @param value the value of the cell
-      */
+     * Constructs a <code>Formula</code> using specified attributes
+     *
+     * @param row          row number
+     * @param column       column number
+     * @param cellContents contents of the cell
+     * @param ixfe         font index
+     * @param fmt          cell format description.
+     * @param wb           the workbook we are defined in.
+     */
     public Formula(int row, int column, String cellContents, int ixfe, Format fmt, Workbook wb)
     throws Exception {
 
@@ -69,10 +69,10 @@ public class Formula extends CellValue implements OfficeConstants {
         setCol(column);
         setIxfe(ixfe);
         setFormula(cellContents);
-        
+
         String category = fmt.getCategory();
         String value = fmt.getValue();
-        
+
         if(category.equalsIgnoreCase(CELLTYPE_BOOLEAN)) {
             num[0]=(byte)0x01;
             num[1]=(byte)0x00;
@@ -111,24 +111,25 @@ public class Formula extends CellValue implements OfficeConstants {
             num = EndianConverter.writeDouble(cellLong);
         }
     }
-  
+
     /**
       * Translates a <code>String</code> written in infix which represents a
      * formula into a byte[] what can be written to pocket excel file.
       *
-      * @param	formula string
+      * @param	inFormula formula string
       */
     public void setFormula(String inFormula) throws Exception {
 
         rgce = fh.convertCalcToPXL(inFormula);
         cce = EndianConverter.writeShort((short) rgce.length);
     }
-  
+
     /**
       * Constructs a pocket Excel formula from the
-      * <code>InputStream</code> 
+      * <code>InputStream</code>
       *
       * @param	is InputStream containing a Pocket Excel Data file.
+      * @param	wb The workbook we are defined in.
       */
     public Formula(InputStream is, Workbook wb) throws IOException {
         read(is);
@@ -136,51 +137,52 @@ public class Formula extends CellValue implements OfficeConstants {
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>Formula</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.FORMULA_CELL;
     }
-    
+
     /**
      * Reads the formula data members from the stream. Byte arrays for Strings
      * are doubled as they are stored as  unicode
      *
-     * @return total number of bytes read 
+     * @param input  the input stream to read from.
+     * @return total number of bytes read
      */
     public int read(InputStream input) throws IOException {
 
         int numOfBytesRead = super.read(input);
 
         numOfBytesRead += input.read(num);
-        grbit				= (byte) input.read();        
+        grbit				= (byte) input.read();
         numOfBytesRead		++;
         numOfBytesRead		+= input.read(cce);
-       
+
         int strLen = EndianConverter.readShort(cce);
         rgce = new byte[strLen];
-        input.read(rgce, 0, strLen);        
-        
+        input.read(rgce, 0, strLen);
+
         Debug.log(Debug.TRACE, " num : " + num +
-                            "\n\tgrbit : " + grbit +        
+                            "\n\tgrbit : " + grbit +
                             " cce : " + EndianConverter.readShort(cce) +
-                            " rgce : " + new String(rgce,"UTF-16LE") + 
+                            " rgce : " + new String(rgce,"UTF-16LE") +
                             "\n" + numOfBytesRead + " Bytes Read");
-        
+
         return numOfBytesRead;
     }
 
      /**
-     * Writes the Formula record to the <code>OutputStream</code> 
+     * Writes the Formula record to the <code>OutputStream</code>
      *
-     * @param the <code>OutputStream</code> being written to 
-     */   
+     * @param output The <code>OutputStream</code> being written to
+     */
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
-        
+
         super.write(output);
 
         output.write(num);
@@ -204,7 +206,7 @@ public class Formula extends CellValue implements OfficeConstants {
     }
 
     /**
-     * Gets the <code>String</code> representing the cells contents 
+     * Gets the <code>String</code> representing the cells contents
      *
      * @return the <code>String</code> representing the cells contents
      */
@@ -212,13 +214,13 @@ public class Formula extends CellValue implements OfficeConstants {
 
         return fh.convertPXLToCalc(rgce);
     }
-    
+
     /**
      * Excel dates are the number of days since 1/1/1900. This method converts
      * to this date.
      *
-     * @param s String representing a date in the form YYYY-MM-DD 
-     * @return The excel serial date  
+     * @param s String representing a date in the form YYYY-MM-DD
+     * @return The excel serial date
      */
     public long toExcelSerialDate(String s) throws IOException {
 
@@ -229,17 +231,17 @@ public class Formula extends CellValue implements OfficeConstants {
         long serialDate =	(1461 * (year + 4800 + (month - 14) / 12)) / 4 +
                             (367 * (month - 2 - 12 * ((month - 14) / 12))) / 12 -
                             (3 * ((year + 4900 + (month - 14) / 12)) / 100) / 4 +
-                            day - 2415019 - 32075;	
+                            day - 2415019 - 32075;
 
         return serialDate;
     }
-    
+
     /**
      * Excel times are a fraction of a 24 hour day expressed in seconds. This method converts
      * to this time.
      *
-     * @param s String representing a time in the form ??HH?MM?SS? 
-     * @return The excel serial time  
+     * @param s String representing a time in the form ??HH?MM?SS?
+     * @return The excel serial time
      */
     public double toExcelSerialTime(String s) throws IOException {
 
@@ -249,7 +251,7 @@ public class Formula extends CellValue implements OfficeConstants {
 
         int timeSecs = (hours*3600) + (mins*60) + (secs);
 
-        double d = (double) timeSecs / (24 * 3600);	
+        double d = (double) timeSecs / (24 * 3600);
 
         return d;
     }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/LabelCell.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/LabelCell.java
index 879c187..9506ed1 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/LabelCell.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/LabelCell.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -35,7 +35,6 @@ import org.openoffice.xmerge.util.Debug;
 import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
-
 /**
  * Reperesent a BIFF Record descibing a cell containing a string
  */
@@ -43,19 +42,19 @@ public class LabelCell extends CellValue {
 
     private byte[] cch	= new byte[2];
     private byte[] rgch;
-    
+
     /**
       * Constructs a <code>LabelCell</code> using specified attributes
-     * 
-     * @param row row number
-     * @param col column number 
-     * @param cellContents contents of the cell 
-     * @param ixfe font index
+     *
+     * @param row          row number
+     * @param column       column number
+     * @param cellContents contents of the cell
+     * @param ixfe         font index
       */
     public LabelCell(int row, int column, String cellContents, int ixfe)
     throws IOException {
 
-           setLabel(cellContents); 
+           setLabel(cellContents);
            setRow(row);
            setCol(column);
         setIxfe(ixfe);
@@ -67,18 +66,18 @@ public class LabelCell extends CellValue {
      * @param is the <code>Inputstream</code> to read from
      */
     public LabelCell(InputStream is) throws IOException {
-       read(is); 
+       read(is);
     }
 
     /**
      * Writes a <code>LabelCell</code> to the specified <code>Outputstream</code>
      *
-     * @param os the <code>OutputStream</code> to write to  
+     * @param output The <code>OutputStream</code> to write to
      */
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
-        
+
         super.write(output);
 
         output.write(cch);
@@ -86,40 +85,39 @@ public class LabelCell extends CellValue {
 
         Debug.log(Debug.TRACE,"Writing Label record");
     }
-    
+
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>LabelCell</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.LABEL_CELL;
     }
-    
+
     /**
      * Reads a<code>LabelCell</code> from the specified <code>InputStream</code>
      *
-     * @param input the <code>InputStram</code> to read from  
+     * @param input the <code>InputStram</code> to read from
      */
     public int read(InputStream input) throws IOException {
 
         int numOfBytesRead = super.read(input);
 
         numOfBytesRead += input.read(cch);
-           
+
         int strLen = EndianConverter.readShort(cch)*2;
         rgch = new byte[strLen];
-        input.read(rgch, 0, strLen); 
-        
+        input.read(rgch, 0, strLen);
+
         Debug.log(Debug.TRACE, " cch : " + EndianConverter.readShort(cch) +
-                            " rgch : " + new String(rgch, "UTF-16LE"));        
-        
-        return numOfBytesRead;    
-    }
+                            " rgch : " + new String(rgch, "UTF-16LE"));
 
+        return numOfBytesRead;
+    }
 
     /**
-     * Gets the <code>String</code> representing the cells contents 
+     * Gets the <code>String</code> representing the cells contents
      *
      * @return the <code>String</code> representing the cells contents
      */
@@ -128,7 +126,7 @@ public class LabelCell extends CellValue {
     }
 
     /**
-     * Sets the <code>String</code> representing the cells contents 
+     * Sets the <code>String</code> representing the cells contents
      *
      * @return the <code>String</code> representing the cells contents
      */
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Pane.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Pane.java
index b57135a..bf5e796 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Pane.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Pane.java
@@ -124,9 +124,11 @@ public class Pane implements BIFFRecord {
     } 	
     
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Set the pane number
+     * 0 - bottom right, 1 - top right
+     * 2 - bottom left, 3 - top left
      *
-     * @return the hex code for <code>Pane</code>
+     * @param paneNumber the pane number
      */
     public void setPaneNumber(int paneNumber) {
         pnnAcct = (byte) paneNumber;
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Row.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Row.java
index e0e00f6..44bc16d 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Row.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Row.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -26,7 +26,7 @@
  ************************************************************************/
 
 package org.openoffice.xmerge.converter.xml.sxc.pexcel.records;
- 
+
 import java.io.OutputStream;
 import java.io.InputStream;
 import java.io.IOException;
@@ -35,7 +35,6 @@ import org.openoffice.xmerge.util.Debug;
 import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
-
 /**
  * Represents s BIFF Record that describes the format of a column
  */
@@ -46,12 +45,12 @@ public class Row implements BIFFRecord {
     private byte[] grbit	= new byte[2];
     private byte[] ixfe		= new byte[2];
     private float  scale = (float) 1;
-    
+
     /**
      * Constructs a pocket Excel Document from the
      * <code>InputStream</code> and assigns it the document name passed in
      *
-     * @param	rw Zero based row number 
+     * @param	rw Zero based row number
      * @param	miyRw row height
      */
     public Row(int rw, int miyRw, boolean userDefined) {
@@ -67,62 +66,62 @@ public class Row implements BIFFRecord {
     }
 
     /**
-     * Constructs a Row fro man <code>InputStream</code> 
+     * Constructs a Row fro man <code>InputStream</code>
      *
-     * @param	is InputStream containing a Pane Record 
+     * @param	is InputStream containing a Pane Record
      */
     public Row(InputStream is) throws IOException {
-        read(is);	
+        read(is);
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>Row</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.ROW_DESCRIPTION;
     }
-    
+
     /**
-     * Get the height of this row 
+     * Get the height of this row
      *
-     * @return the height of this row 
+     * @return the height of this row
      */
     public short getRowHeight() {
         return EndianConverter.readShort(miyRw);
     }
-    
+
     /**
-     * Get the rown number for this style 
+     * Get the row number
      *
-     * @return the row this style applies to 
+     * @return the row this style applies to
      */
     public short getRowNumber() {
         return EndianConverter.readShort(rw);
-    }	
+    }
 
     /**
-     * Reads a Row from an <code>InputStream</code> 
+     * Reads a Row from an <code>InputStream</code>
      *
-     * @param	is InputStream containing a Pane Record 
+     * @param	input InputStream containing a Row Record
      */
     public int read(InputStream input) throws IOException {
-        
+
         int numOfBytesRead	= input.read(rw);
         numOfBytesRead		+= input.read(miyRw);
         short scaledHeight = (short) (EndianConverter.readShort(miyRw) / scale);
-        miyRw = EndianConverter.writeShort(scaledHeight); 
+        miyRw = EndianConverter.writeShort(scaledHeight);
         numOfBytesRead		+= input.read(grbit);
         numOfBytesRead		+= input.read(ixfe);
-            
-        Debug.log(Debug.TRACE,"\trw : "+ EndianConverter.readShort(rw) + 
+
+        Debug.log(Debug.TRACE,"\trw : "+ EndianConverter.readShort(rw) +
                             " miyRw : " + EndianConverter.readShort(miyRw) +
                             " grbit : " + EndianConverter.readShort(grbit) +
-                            " ixfe : " + EndianConverter.readShort(ixfe));           
+                            " ixfe : " + EndianConverter.readShort(ixfe));
         return numOfBytesRead;
     }
-    
+
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
@@ -134,5 +133,5 @@ public class Row implements BIFFRecord {
         Debug.log(Debug.TRACE,"Writing Row record");
 
     }
-    
+
 }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Selection.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Selection.java
index effa109..191a0e9 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Selection.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Selection.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -36,7 +36,6 @@ import org.openoffice.xmerge.util.Debug;
 import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
-
 /**
  * Represents a BIFF Record that describes the selected area of a worksheet
  */
@@ -48,66 +47,66 @@ public class Selection implements BIFFRecord {
     private byte   colRight;
     private byte[] rwActive		= new byte[2];
     private byte   colActive;
-    
+
     /**
      * Default Constructor
      */
     public Selection() {
         this.rwTop			= EndianConverter.writeShort((short) 0);
-        this.colLeft		= 0; 
+        this.colLeft		= 0;
         this.rwBottom		= EndianConverter.writeShort((short) 0);
-        this.colRight		= 0; 
+        this.colRight		= 0;
         this.rwActive		= EndianConverter.writeShort((short) 0);
-        this.colActive		= 0; 
-        
+        this.colActive		= 0;
+
     }
 
     /**
-     * Constructs a Selection Record from the <code>InputStream</code> 
+     * Constructs a Selection Record from the <code>InputStream</code>
      *
      * @param	is InputStream containing a Pocket Excel Data file.
      */
     public Selection(InputStream is) throws IOException {
-        read(is);	
+        read(is);
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>Selection</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.CURRENT_SELECTION;
     }
-    
+
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the active cell for this particular <code>BIFFRecord</code>
      *
-     * @return the hex code for <code>Selection</code>
+     * @return the cell position
      */
     public Point getActiveCell() {
         Point p = new Point(colActive, EndianConverter.readShort(rwActive));
         return p;
-    } 
-    
+    }
+
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Set the active cell position for this particular <code>BIFFRecord</code>
      *
-     * @return the hex code for <code>Selection</code>
+     * @param p The active cell position
      */
     public void setActiveCell(Point p) {
 
         colActive = (byte) p.getX();
         rwActive = EndianConverter.writeShort((short) p.getY());
     }
-    
+
     /**
-     * Reads a Selection Record from the <code>InputStream</code> 
+     * Reads a Selection Record from the <code>InputStream</code>
      *
-     * @param	is InputStream containing a Pocket Excel Data file.
+     * @param	input InputStream containing a Pocket Excel Data file.
      */
     public int read(InputStream input) throws IOException {
-        
+
         int numOfBytesRead	= input.read(rwTop);
         colLeft				+= (byte) input.read();
         numOfBytesRead		+= input.read(rwBottom);
@@ -115,17 +114,17 @@ public class Selection implements BIFFRecord {
         numOfBytesRead		+= input.read(rwActive);
         colActive			+= (byte) input.read();
         numOfBytesRead += 3;
-                
-        Debug.log(Debug.TRACE,"\trwTop : "+ EndianConverter.readShort(rwTop) + 
+
+        Debug.log(Debug.TRACE,"\trwTop : "+ EndianConverter.readShort(rwTop) +
                             " colLeft : " + colLeft +
                             " rwBottom : " + EndianConverter.readShort(rwBottom) +
-                            " colRight : "+ colRight + 
+                            " colRight : "+ colRight +
                             " rwActive : " + EndianConverter.readShort(rwActive) +
                             " colActive : " + colActive);
-        
-        return numOfBytesRead;                
+
+        return numOfBytesRead;
     }
-    
+
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
@@ -138,5 +137,5 @@ public class Selection implements BIFFRecord {
 
         Debug.log(Debug.TRACE,"Writing Selection record");
     }
-    
+
 }
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/StringValue.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/StringValue.java
index f2fa163..1a98c67 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/StringValue.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/StringValue.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -36,7 +36,6 @@ import org.openoffice.xmerge.util.Debug;
 import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
-
 /**
  * Represents a BIFF Record that describes the value of a formula that
  * evaluates to a string
@@ -47,55 +46,55 @@ public class StringValue implements BIFFRecord {
     private byte[]  rgch;
 
     /**
-      * Constructs a StringValue Record from an <code>InputStream</code> 
+      * Constructs a StringValue Record from a string
       *
-      * @param	is InputStream containing a StringValue Record 
+      * @param	str String containing value
       */
     public StringValue(String str) throws IOException {
         cch = EndianConverter.writeShort((short) str.length());
         rgch = new byte[str.length()];
         rgch = str.getBytes("UTF-16LE");
     }
-    
+
     /**
-      * Constructs a StringValue Record from an <code>InputStream</code> 
+      * Constructs a StringValue Record from an <code>InputStream</code>
       *
-      * @param	is InputStream containing a StringValue Record 
+      * @param	is InputStream containing a StringValue Record
       */
     public StringValue(InputStream is) throws IOException {
         read(is);
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>StringValue</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.FORMULA_STRING;
     }
-   
+
        /**
-      * Reads a StringVlaue Record from an <code>InputStream</code> 
+      * Reads a StringVlaue Record from an <code>InputStream</code>
       *
-      * @param	is InputStream containing a StringValue Record 
+      * @param	input InputStream containing a StringValue Record
       */
     public int read(InputStream input) throws IOException {
-        
+
         cch[0] = (byte) input.read();
         cch[1] = (byte) input.read();
         int numOfBytesRead = 1;
-        
+
         int strlen = EndianConverter.readShort(cch)*2;
         rgch = new byte[strlen];
-        numOfBytesRead	+= input.read(rgch, 0, strlen);        
-    
-        Debug.log(Debug.TRACE,"\tcch : "+ cch + 
+        numOfBytesRead	+= input.read(rgch, 0, strlen);
+
+        Debug.log(Debug.TRACE,"\tcch : "+ cch +
                             " rgch : " + rgch);
-        
+
         return numOfBytesRead;
     }
-    
+
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
@@ -106,7 +105,7 @@ public class StringValue implements BIFFRecord {
     }
 
     /**
-     * Gets the <code>String</code> representing the cells contents 
+     * Gets the <code>String</code> representing the cells contents
      *
      * @return the <code>String</code> representing the cells contents
      */
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window1.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window1.java
index 428de2a..576ed9e 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window1.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window1.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -35,15 +35,14 @@ import org.openoffice.xmerge.util.Debug;
 import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 
-
 /**
- * Represents a BIFF REcord that describes workbook window attributes 
+ * Represents a BIFF REcord that describes workbook window attributes
  */
 public class Window1 implements BIFFRecord {
 
     private byte[] grbit 	= new byte[2];
     private byte[] itabCur	= new byte[2];		// index of selected worksheet
-    
+
     /**
       * Constructor
       */
@@ -53,57 +52,57 @@ public class Window1 implements BIFFRecord {
     }
 
        /**
-      * Constructs a Window1 Record from an <code>InputStream</code> 
+      * Constructs a Window1 Record from an <code>InputStream</code>
       *
-      * @param	is InputStream containing a Window1 Record 
+      * @param	is InputStream containing a Window1 Record
       */
     public Window1(InputStream is) throws IOException{
-        read(is);	
+        read(is);
     }
-    
+
     /**
      * Set the number of the active sheet
      *
-     * @param activeSheet number of the active sheet 
+     * @param activeSheet number of the active sheet
      */
     public void setActiveSheet(int activeSheet) {
         itabCur = EndianConverter.writeShort((short) activeSheet);
     }
-    
+
     /**
-     * Get the number of the active sheet 
+     * Get the number of the active sheet
      *
      * @return 	 number of the active sheet
      */
     public int getActiveSheet() {
         return EndianConverter.readShort(itabCur);
-    }	
+    }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>Window1</code>
      */
     public short getBiffType() {
         return PocketExcelConstants.WINDOW_INFO;
     }
-    
+
        /**
-      * Reads a Window1 Record from an <code>InputStream</code> 
+      * Reads a Window1 Record from an <code>InputStream</code>
       *
-      * @param	is InputStream containing a Window1 Record 
+      * @param	input InputStream containing a Window1 Record
       */
     public int read(InputStream input) throws IOException {
-        
+
         int numOfBytesRead	= input.read(grbit);
         numOfBytesRead		+= input.read(itabCur);
-                
-        Debug.log(Debug.TRACE,"\tgrbit : "+ EndianConverter.readShort(grbit) + 
+
+        Debug.log(Debug.TRACE,"\tgrbit : "+ EndianConverter.readShort(grbit) +
                             " itabCur : " + EndianConverter.readShort(itabCur));
-                            
-        return numOfBytesRead;        
+
+        return numOfBytesRead;
     }
-    
+
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window2.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window2.java
index 8557b9c..7069cd3 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window2.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Window2.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -37,7 +37,6 @@ import org.openoffice.xmerge.util.EndianConverter;
 import org.openoffice.xmerge.converter.xml.sxc.pexcel.PocketExcelConstants;
 import org.openoffice.xmerge.converter.xml.sxc.SheetSettings;
 
-
 /**
  * Represents a BIFF Record that describes worksheet window attributes
  */
@@ -49,27 +48,27 @@ public class Window2 implements BIFFRecord {
     private byte[] rwTop	= new byte[2];
     private byte   colLeft;
     private byte[] grbit	= new byte[2];
-    
+
     /**
-     * Constructor  
+     * Constructor
      */
     public Window2() {
         this.rwTop		= EndianConverter.writeShort((short) 0);
-        this.colLeft	= 0; 
+        this.colLeft	= 0;
         this.grbit		= EndianConverter.writeShort((short) 0);
     }
-    
+
        /**
-      * Constructs a Window2 Record from an <code>InputStream</code> 
+      * Constructs a Window2 Record from an <code>InputStream</code>
       *
-      * @param	is InputStream containing a Window2 Record 
+      * @param	is InputStream containing a Window2 Record
       */
     public Window2(InputStream is) throws IOException {
-        read(is);	
+        read(is);
     }
 
     /**
-     * Get the hex code for this particular <code>BIFFRecord</code> 
+     * Get the hex code for this particular <code>BIFFRecord</code>
      *
      * @return the hex code for <code>Window2</code>
      */
@@ -94,8 +93,8 @@ public class Window2 implements BIFFRecord {
         }
     }
 
-    /** 
-     * This method tests if this object describes a freeze 
+    /**
+     * This method tests if this object describes a freeze
      *
      * @return true if freeze otherwise false
      */
@@ -109,7 +108,7 @@ public class Window2 implements BIFFRecord {
         return true;
     }
 
-    /** 
+    /**
      * This method tests if this object describes a split
      *
      * @return true if split otherwise false
@@ -123,25 +122,25 @@ public class Window2 implements BIFFRecord {
 
         return true;
     }
-    
+
        /**
-      * Reads a Window2 Record from an <code>InputStream</code> 
+      * Reads a Window2 Record from an <code>InputStream</code>
       *
-      * @param	is InputStream containing a Window2 Record 
+      * @param	input InputStream containing a Window2 Record
       */
     public int read(InputStream input) throws IOException {
-        
+
         int numOfBytesRead	= input.read(rwTop);
         colLeft				= (byte) input.read();
         numOfBytesRead++;
         numOfBytesRead		+= input.read(grbit);
-        
-        Debug.log(Debug.TRACE,"\trwTop : "+ EndianConverter.readShort(rwTop) + 
+
+        Debug.log(Debug.TRACE,"\trwTop : "+ EndianConverter.readShort(rwTop) +
                             " colLeft : " + colLeft +
                             " grbit : " + EndianConverter.readShort(grbit));
-        return numOfBytesRead;        
+        return numOfBytesRead;
     }
-    
+
     public void write(OutputStream output) throws IOException {
 
         output.write(getBiffType());
diff --git a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Workbook.java b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Workbook.java
index b6c7e78..4360db7 100644
--- a/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Workbook.java
+++ b/xmerge/source/pexcel/java/org/openoffice/xmerge/converter/xml/sxc/pexcel/records/Workbook.java
@@ -1,7 +1,7 @@
 /*************************************************************************
  *
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
- * 
+ *
  * Copyright 2000, 2010 Oracle and/or its affiliates.
  *
  * OpenOffice.org - a multi-platform office productivity suite
@@ -44,9 +44,9 @@ import org.openoffice.xmerge.converter.xml.sxc.ColumnRowInfo;
 
 /**
  *  This class is used by <code> PxlDocument</code> to maintain pexcel
- *  workbooks. 
+ *  workbooks.
  *
- *  @author  Martin Maher 
+ *  @author  Martin Maher
  */
 public class Workbook implements org.openoffice.xmerge.Document,
 OfficeConstants {
@@ -69,7 +69,6 @@ OfficeConstants {
         eof				= new Eof();
     }
 
-
     /**
      * Constructs a pocket Excel Workbook with the name of the file passed in
      * as an argument. Also fills out a basic header block containing the
@@ -91,7 +90,9 @@ OfficeConstants {
      * Constructs a pocket Excel Workbook from the
      * <code>InputStream</code> and assigns it the document name passed in
      *
-     * @param	is InputStream containing a Pocket Excel Data file.
+     * @param	name Name of the Pocket Excel Data file. (including the file
+     * 				 extension)
+     * @param	is   InputStream containing a Pocket Excel Data file.
      */
     public Workbook(String name, InputStream is) throws IOException {
         read(is);
@@ -99,9 +100,9 @@ OfficeConstants {
     }
 
     /**
-      *	Writes the current workbook to the <code>Outputstream</code> 
-      * 
-      * @param	os The destination outputstream	
+      *	Writes the current workbook to the <code>Outputstream</code>
+      *
+      * @param	os The destination outputstream
       */
     public void write(OutputStream os) throws IOException {
         bof.write(os);
@@ -133,7 +134,7 @@ OfficeConstants {
 
     /**
       *	Reads a workbook from the <code>InputStream</code> and contructs a
-     *	workbook object from it 
+     *	workbook object from it
      *
       * @param	is InputStream containing a Pocket Excel Data file.
       */
@@ -150,7 +151,7 @@ OfficeConstants {
                 Debug.log(Debug.TRACE,"End of file reached");
                 break;
             }
-                
+
             switch (b)
             {
                 case PocketExcelConstants.DEFINED_NAME:
@@ -158,12 +159,12 @@ OfficeConstants {
                     DefinedName dn = new DefinedName(is, this);
                     definedNames.add(dn);
                     break;
-                    
+
                 case PocketExcelConstants.BOF_RECORD:
                     Debug.log(Debug.TRACE,"BOF Record");
                     bof.read(is);
                     break;
-                    
+
                 case PocketExcelConstants.EOF_MARKER:
                     Debug.log(Debug.TRACE,"EOF Marker");
                     eof.read(is);
@@ -173,7 +174,7 @@ OfficeConstants {
                         ws = new Worksheet(this);
                     }
                     break;
-                    
+
                 case PocketExcelConstants.FONT_DESCRIPTION:
                     Debug.log(Debug.TRACE,"FONT: Font Description (31h)");
                     FontDescription fd = new FontDescription(is);
@@ -184,7 +185,7 @@ OfficeConstants {
                     Debug.log(Debug.TRACE,"WINDOW1: Window Information (3Dh) [PXL 2.0]");
                     win1.read(is);
                     break;
-                    
+
                 case PocketExcelConstants.CODEPAGE:
                     Debug.log(Debug.TRACE,"CODEPAGE : Codepage and unknown fields (42h)");
                     cp.read(is);
@@ -195,32 +196,32 @@ OfficeConstants {
                     BoundSheet bs = new BoundSheet(is);
                     boundsheets.add(bs);
                     break;
-                        
+
                 case PocketExcelConstants.EXTENDED_FORMAT:
                     Debug.log(Debug.TRACE,"XF: Extended Format (E0h) [PXL 2.0]");
                     ExtendedFormat xf = new ExtendedFormat(is);
                     extendedFormats.add(xf);
                     break;
-                                            
+
                 default:
                     b = is.read();
                     break;
             }
-                
+
         }
         is.close();
     }
-    
+
     /**
-      *	Adds a font recrod to the workbook 
+      *	Adds a font recrod to the workbook
      *
-      * @param	f the font record to add	
-      */	
+      * @param	f the font record to add
+      */
     public int addFont(FontDescription f) {
 
         boolean alreadyExists = false;
         int i = 0;
-        
+
         for(Enumeration e = fonts.elements();e.hasMoreElements();) {
             FontDescription fd = (FontDescription) e.nextElement();
             if(fd.compareTo(f)) {
@@ -230,18 +231,18 @@ OfficeConstants {
                 i++;
             }
         }
-        
-        if(!alreadyExists) 
+
+        if(!alreadyExists)
                 fonts.add(f);
 
-        return i; 
+        return i;
     }
 
     /**
-      *	Adds a ExtendedFormat record to the workbook 
+      *	Adds a ExtendedFormat record to the workbook
      *
-      * @param	f the font recrod to add	
-      */	
+      * @param	fmt the font record to add
+      */
     public int addExtendedFormat(Format fmt) throws IOException {
 
         FontDescription fd = new FontDescription(fmt);
@@ -250,7 +251,7 @@ OfficeConstants {
 
         boolean alreadyExists = false;
         int i = 0;
-        
+
         for(Enumeration e = extendedFormats.elements();e.hasMoreElements();) {
             ExtendedFormat currentXF = (ExtendedFormat) e.nextElement();
             if(xf.compareTo(currentXF)) {
@@ -263,30 +264,30 @@ OfficeConstants {
 
         if(!alreadyExists)
             extendedFormats.add(xf);
-        
-        return i; 
+
+        return i;
     }
-    
+
     /**
-      *	Gets a worksheet at a particular index from mthe current workbook. 
+      *	Gets a worksheet at a particular index from mthe current workbook.
      *
-      * @param	index the index of the worksheet to retrieve	
+      * @param	index the index of the worksheet to retrieve
       */
     public Worksheet getWorksheet(int index) {
 
         return ((Worksheet) worksheets.elementAt(index));
     }
-    
+
     /**
      * Returns a FontDescription indictated by the
      * index parameter passed in to the method
      *
      * @param ixfnt index to the FontDescriptions, this is a 0 based index
-     * @return FontDescription indexed by ixfe 
+     * @return FontDescription indexed by ixfe
      */
     public FontDescription getFontDescription(int ixfnt) {
 
-        return (FontDescription) fonts.elementAt(ixfnt);	
+        return (FontDescription) fonts.elementAt(ixfnt);
     }
 
     /**
@@ -294,15 +295,15 @@ OfficeConstants {
      * index parameter passed in to the method
      *
      * @param ixfe index to the FontDescriptions, this is a 0 based index
-     * @return FontDescription indexed by ixfe 
+     * @return FontDescription indexed by ixfe
      */
     public ExtendedFormat getExtendedFormat(int ixfe) {
 

... etc. - the rest is truncated


More information about the Libreoffice-commits mailing list