You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ni...@apache.org on 2008/09/15 22:58:30 UTC

svn commit: r695621 - in /poi/trunk/src/java/org/apache/poi/hssf/usermodel: HSSFRow.java HSSFWorkbook.java

Author: nick
Date: Mon Sep 15 13:58:30 2008
New Revision: 695621

URL: http://svn.apache.org/viewvc?rev=695621&view=rev
Log:
Make MissingCellPolicy javadocs even more explicit about iterators, as people seem not to quite get it

Modified:
    poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFRow.java
    poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java

Modified: poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFRow.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFRow.java?rev=695621&r1=695620&r2=695621&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFRow.java (original)
+++ poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFRow.java Mon Sep 15 13:58:30 2008
@@ -545,6 +545,8 @@
      * Note that the 4th element might well not be cell 4, as the iterator
      *  will not return un-defined (null) cells.
      * Call getCellNum() on the returned cells to know which cell they are.
+     * As this only ever works on physically defined cells, 
+     *  the {@link MissingCellPolicy} has no effect.
      */
     public Iterator cellIterator()
     {

Modified: poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java?rev=695621&r1=695620&r2=695621&view=diff
==============================================================================
--- poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java (original)
+++ poi/trunk/src/java/org/apache/poi/hssf/usermodel/HSSFWorkbook.java Mon Sep 15 13:58:30 2008
@@ -382,7 +382,10 @@
      *  getting missing or blank cells from a row.
      * This will then apply to all calls to
      *  {@link HSSFRow.getCell()}. See
-     *  {@link MissingCellPolicy}
+     *  {@link MissingCellPolicy}.
+     * Note that this has no effect on any
+     *  iterators, only on when fetching Cells
+     *  by their column index.
      */
     public void setMissingCellPolicy(MissingCellPolicy missingCellPolicy) {
         this.missingCellPolicy = missingCellPolicy;



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org