You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by dm...@apache.org on 2004/02/18 17:48:17 UTC
cvs commit: jakarta-poi/src/testcases/org/apache/poi/hssf/usermodel TestCloneSheet.java
dmui 2004/02/18 08:48:17
Modified: src/java/org/apache/poi/hssf/record Tag: REL_2_BRANCH
MergeCellsRecord.java
src/testcases/org/apache/poi/hssf/usermodel Tag:
REL_2_BRANCH TestCloneSheet.java
Log:
Removed variable that stores the number of merged regions in
its record since the array should be the basis for this number. Less
manual index tracking, the better.
Thanks for problem report!
PR: 27005
Reported by: skalchihin@mgsm.ru
Revision Changes Path
No revision
No revision
1.6.2.3 +16 -17 jakarta-poi/src/java/org/apache/poi/hssf/record/MergeCellsRecord.java
Index: MergeCellsRecord.java
===================================================================
RCS file: /home/cvs/jakarta-poi/src/java/org/apache/poi/hssf/record/MergeCellsRecord.java,v
retrieving revision 1.6.2.2
retrieving revision 1.6.2.3
diff -u -r1.6.2.2 -r1.6.2.3
--- MergeCellsRecord.java 17 Nov 2003 21:32:11 -0000 1.6.2.2
+++ MergeCellsRecord.java 18 Feb 2004 16:48:16 -0000 1.6.2.3
@@ -61,19 +61,18 @@
import org.apache.poi.util.LittleEndian;
/**
- * Title: Merged Cells Record<P>
+ * Title: Merged Cells Record
+ * <br>
* Description: Optional record defining a square area of cells to "merged" into
- * one cell. <P>
- * REFERENCE: NONE (UNDOCUMENTED PRESENTLY) <P>
+ * one cell. <br>
+ * REFERENCE: NONE (UNDOCUMENTED PRESENTLY) <br>
* @author Andrew C. Oliver (acoliver at apache dot org)
* @version 2.0-pre
*/
-
public class MergeCellsRecord
extends Record
{
public final static short sid = 0xe5;
- private short field_1_num_areas;
private ArrayList field_2_regions;
public MergeCellsRecord()
@@ -109,11 +108,11 @@
protected void fillFields(byte [] data, short size, int offset)
{
- field_1_num_areas = LittleEndian.getShort(data, 0 + offset);
- field_2_regions = new ArrayList(field_1_num_areas + 10);
+ short numAreas = LittleEndian.getShort(data, 0 + offset);
+ field_2_regions = new ArrayList(numAreas + 10);
int pos = 2;
- for (int k = 0; k < field_1_num_areas; k++)
+ for (int k = 0; k < numAreas; k++)
{
MergedRegion region =
new MergedRegion(LittleEndian
@@ -135,7 +134,9 @@
public short getNumAreas()
{
- return field_1_num_areas;
+ //if the array size is larger than a short (65536), the record can't hold that many merges anyway
+ if (field_2_regions == null) return 0;
+ return (short)field_2_regions.size();
}
/**
@@ -143,13 +144,14 @@
* it will be incremented automatically or decremented when an area is removed. If
* you are setting this to 0 then you are a terrible person. Just remove the record.
* (just kidding about you being a terrible person..hehe)
- *
+ * @deprecated We now link the size to the actual array of merged regions
+ * @see #getNumAreas()
* @param numareas number of areas
*/
public void setNumAreas(short numareas)
{
- field_1_num_areas = numareas;
+
}
/**
@@ -175,7 +177,6 @@
colto);
field_2_regions.add(region);
- field_1_num_areas++;
return field_2_regions.size() - 1;
}
@@ -187,7 +188,6 @@
public void removeAreaAt(int area)
{
field_2_regions.remove(area);
- field_1_num_areas--;
}
/**
@@ -246,9 +246,9 @@
retval.append("[MERGEDCELLS]").append("\n");
retval.append(" .sid =").append(sid).append("\n");
- retval.append(" .numregions =").append(field_1_num_areas)
+ retval.append(" .numregions =").append(getNumAreas())
.append("\n");
- for (int k = 0; k < field_1_num_areas; k++)
+ for (int k = 0; k < getNumAreas(); k++)
{
MergedRegion region = ( MergedRegion ) field_2_regions.get(k);
@@ -325,8 +325,7 @@
}
public Object clone() {
- MergeCellsRecord rec = new MergeCellsRecord();
- rec.field_1_num_areas = field_1_num_areas;
+ MergeCellsRecord rec = new MergeCellsRecord();
rec.field_2_regions = new ArrayList();
Iterator iterator = field_2_regions.iterator();
while (iterator.hasNext()) {
No revision
No revision
1.2.2.3 +11 -4 jakarta-poi/src/testcases/org/apache/poi/hssf/usermodel/TestCloneSheet.java
Index: TestCloneSheet.java
===================================================================
RCS file: /home/cvs/jakarta-poi/src/testcases/org/apache/poi/hssf/usermodel/TestCloneSheet.java,v
retrieving revision 1.2.2.2
retrieving revision 1.2.2.3
diff -u -r1.2.2.2 -r1.2.2.3
--- TestCloneSheet.java 9 Feb 2004 22:41:26 -0000 1.2.2.2
+++ TestCloneSheet.java 18 Feb 2004 16:48:17 -0000 1.2.2.3
@@ -2,7 +2,7 @@
/* ====================================================================
* The Apache Software License, Version 1.1
*
- * Copyright (c) 2003 The Apache Software Foundation. All rights
+ * Copyright (c) 2004 The Apache Software Foundation. All rights
* reserved.
*
* Redistribution and use in source and binary forms, with or without
@@ -55,10 +55,14 @@
package org.apache.poi.hssf.usermodel;
+import java.io.File;
+import java.io.FileInputStream;
+import java.io.FileOutputStream;
+
import junit.framework.TestCase;
-import org.apache.poi.hssf.usermodel.HSSFSheet;
-import org.apache.poi.hssf.usermodel.HSSFWorkbook;
+
import org.apache.poi.hssf.util.Region;
+import org.apache.poi.poifs.filesystem.POIFSFileSystem;
/**
* Test the ability to clone a sheet.
@@ -77,7 +81,10 @@
HSSFWorkbook b = new HSSFWorkbook();
HSSFSheet s = b.createSheet("Test");
s.addMergedRegion(new Region((short)0,(short)0,(short)1,(short)1));
- b.cloneSheet(0);
+ HSSFSheet clonedSheet = b.cloneSheet(0);
+
+ assertEquals("One merged area", 1, clonedSheet.getNumMergedRegions());
+
}
catch(Exception e){e.printStackTrace();fail(e.getMessage());}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: poi-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: poi-dev-help@jakarta.apache.org