PR 13292, size of FuncPtg
git-svn-id: https://svn.apache.org/repos/asf/jakarta/poi/trunk@353087 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
parent
e35ddf54f2
commit
a1d25d01ea
@ -797,7 +797,9 @@ end;
|
||||
}
|
||||
return (String) stack.pop(); //TODO: catch stack underflow and throw parse exception.
|
||||
}
|
||||
|
||||
/** Create a tree representation of the RPN token array
|
||||
*used to run the class(RVA) change algo
|
||||
*/
|
||||
private Node createTree() {
|
||||
java.util.Stack stack = new java.util.Stack();
|
||||
int numPtgs = tokens.size();
|
||||
@ -838,6 +840,7 @@ end;
|
||||
}
|
||||
|
||||
}
|
||||
/** Private helper class, used to create a tree representation of the formula*/
|
||||
class Node {
|
||||
private Ptg value=null;
|
||||
private Node[] children=new Node[0];
|
||||
|
@ -69,7 +69,7 @@ public abstract class AbstractFunctionPtg extends OperationPtg {
|
||||
//constant used allow a ptgAttr to be mapped properly for its functionPtg
|
||||
public static final String ATTR_NAME = "specialflag";
|
||||
|
||||
private final static int SIZE = 4;
|
||||
|
||||
|
||||
private static BinaryTree map = produceHash();
|
||||
protected static Object[][] functionData = produceFunctionData();
|
||||
@ -129,14 +129,13 @@ public abstract class AbstractFunctionPtg extends OperationPtg {
|
||||
}
|
||||
|
||||
public abstract void writeBytes(byte[] array, int offset);
|
||||
public abstract int getSize();
|
||||
|
||||
|
||||
|
||||
public int getSize() {
|
||||
return SIZE;
|
||||
}
|
||||
|
||||
private String lookupName(short index) {
|
||||
|
||||
protected String lookupName(short index) {
|
||||
return ((String)map.get(new Integer(index)));
|
||||
}
|
||||
|
||||
|
@ -2,13 +2,14 @@ package org.apache.poi.hssf.record.formula;
|
||||
import org.apache.poi.util.LittleEndian;
|
||||
|
||||
/**
|
||||
*
|
||||
* @author aviks
|
||||
* @author Jason Height (jheight at chariot dot net dot au)
|
||||
* @author Danny Mui (dmui at apache dot org) (Leftover handling)
|
||||
*/
|
||||
public class FuncPtg extends AbstractFunctionPtg{
|
||||
|
||||
public final static byte sid = 0x21;
|
||||
public final static int SIZE = 3;
|
||||
private int numParams=0;
|
||||
|
||||
/**
|
||||
@ -17,7 +18,7 @@ public class FuncPtg extends AbstractFunctionPtg{
|
||||
* <p>
|
||||
* If the leftovers are removed, a prompt "Warning: Data may have been lost occurs in Excel"
|
||||
*/
|
||||
protected byte[] leftOvers = null;
|
||||
//protected byte[] leftOvers = null;
|
||||
|
||||
private FuncPtg() {
|
||||
//Required for clone methods
|
||||
@ -31,12 +32,12 @@ public class FuncPtg extends AbstractFunctionPtg{
|
||||
//field_1_num_args = data[ offset + 0 ];
|
||||
field_2_fnc_index = LittleEndian.getShort(data,offset + 0 );
|
||||
|
||||
|
||||
/*
|
||||
if (data.length - offset > 2) { //save left overs if there are any
|
||||
leftOvers = new byte[2];
|
||||
System.arraycopy(data, offset+1, leftOvers, 0, leftOvers.length);
|
||||
}
|
||||
|
||||
*/
|
||||
try {
|
||||
numParams = ( (Integer)functionData[field_2_fnc_index][2]).intValue();
|
||||
} catch (NullPointerException npe) {
|
||||
@ -49,9 +50,9 @@ public class FuncPtg extends AbstractFunctionPtg{
|
||||
array[offset+0]= (byte) (sid + ptgClass);
|
||||
//array[offset+1]=field_1_num_args;
|
||||
LittleEndian.putShort(array,offset+1,field_2_fnc_index);
|
||||
if (leftOvers != null) {
|
||||
/**if (leftOvers != null) {
|
||||
System.arraycopy(leftOvers, 0, array, offset+2, leftOvers.length);
|
||||
}
|
||||
}**/
|
||||
}
|
||||
|
||||
public int getNumberOfOperands() {
|
||||
@ -60,8 +61,23 @@ public class FuncPtg extends AbstractFunctionPtg{
|
||||
|
||||
public Object clone() {
|
||||
FuncPtg ptg = new FuncPtg();
|
||||
ptg.field_1_num_args = field_1_num_args;
|
||||
//ptg.field_1_num_args = field_1_num_args;
|
||||
ptg.field_2_fnc_index = field_2_fnc_index;
|
||||
return ptg;
|
||||
}
|
||||
|
||||
public int getSize() {
|
||||
return SIZE;
|
||||
}
|
||||
|
||||
public String toString() {
|
||||
StringBuffer buffer = new StringBuffer();
|
||||
buffer
|
||||
.append("<FunctionPtg>").append("\n")
|
||||
.append(" numArgs(internal)=").append(this.numParams).append("\n")
|
||||
.append(" name =").append(lookupName(field_2_fnc_index)).append("\n")
|
||||
.append(" field_2_fnc_index=").append(field_2_fnc_index).append("\n")
|
||||
.append("</FunctionPtg>");
|
||||
return buffer.toString();
|
||||
}
|
||||
}
|
@ -8,6 +8,7 @@ import org.apache.poi.util.LittleEndian;
|
||||
public class FuncVarPtg extends AbstractFunctionPtg{
|
||||
|
||||
public final static byte sid = 0x22;
|
||||
private final static int SIZE = 4;
|
||||
|
||||
private FuncVarPtg() {
|
||||
//Required for clone methods
|
||||
@ -54,5 +55,20 @@ public class FuncVarPtg extends AbstractFunctionPtg{
|
||||
return ptg;
|
||||
}
|
||||
|
||||
public int getSize() {
|
||||
return SIZE;
|
||||
}
|
||||
|
||||
public String toString() {
|
||||
StringBuffer buffer = new StringBuffer();
|
||||
buffer
|
||||
.append("<FunctionVarPtg>").append("\n")
|
||||
.append(" field_1_num_args=").append(field_1_num_args).append("\n")
|
||||
.append(" name =").append(lookupName(field_2_fnc_index)).append("\n")
|
||||
.append(" field_2_fnc_index=").append(field_2_fnc_index).append("\n")
|
||||
.append("</FunctionPtg>");
|
||||
return buffer.toString();
|
||||
}
|
||||
|
||||
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user