avik 2003/12/31 23:51:34
Modified: src/java/org/apache/poi/hssf/model FormulaParser.java
src/java/org/apache/poi/hssf/record/formula
AbstractFunctionPtg.java
Log:
bug 25457, formula parser fixes by Peter M Murray
sync to head
Revision Changes Path
1.15 +69 -32
jakarta-poi/src/java/org/apache/poi/hssf/model/FormulaParser.java
Index: FormulaParser.java
===================================================================
RCS file:
/home/cvs/jakarta-poi/src/java/org/apache/poi/hssf/model/FormulaParser.java,v
retrieving revision 1.14
retrieving revision 1.15
diff -u -r1.14 -r1.15
--- FormulaParser.java 31 Aug 2003 06:16:56 -0000 1.14
+++ FormulaParser.java 1 Jan 2004 07:51:34 -0000 1.15
@@ -80,6 +80,7 @@
* @author Andrew C. oliver (acoliver at apache dot org)
* @author Eric Ladner (eladner at goldinc dot com)
* @author Cameron Riley (criley at ekmail.com)
+ * @author Peter M. Murray (pete at quantrix dot com)
*/
public class FormulaParser {
@@ -228,13 +229,31 @@
/** Get an Identifier */
private String GetName() {
StringBuffer Token = new StringBuffer();
- if (!IsAlpha(look)) {
+ if (!IsAlpha(look) && look != '\'') {
Expected("Name");
}
- while (IsAlNum(look)) {
- Token = Token.append(Character.toUpperCase(look));
- GetChar();
- }
+ if(look == '\'')
+ {
+ Match('\'');
+ boolean done = look == '\'';
+ while(!done)
+ {
+ Token.append(Character.toUpperCase(look));
+ GetChar();
+ if(look == '\'')
+ {
+ Match('\'');
+ done = look != '\'';
+ }
+ }
+ }
+ else
+ {
+ while (IsAlNum(look)) {
+ Token.append(Character.toUpperCase(look));
+ GetChar();
+ }
+ }
SkipWhite();
return Token.toString();
}
@@ -478,12 +497,18 @@
/** Parse and Translate a Math Factor */
private void Factor() {
- if (look == '(' ) {
+ if (look == '-')
+ {
+ Match('-');
+ Factor();
+ tokens.add(new UnaryMinusPtg());
+ }
+ else if (look == '(' ) {
Match('(');
Expression();
Match(')');
tokens.add(new ParenthesisPtg());
- } else if (IsAlpha(look)){
+ } else if (IsAlpha(look) || look == '\''){
Ident();
} else if(look == '"') {
StringLiteral();
@@ -501,26 +526,42 @@
}
}
- private void StringLiteral() {
- Match('"');
- StringBuffer Token = new StringBuffer();
- for(;;) {
- if(look == '"') {
- GetChar();
- SkipWhite(); //potential white space here since it doesnt
matter up to the operator
- if(look == '"')
- Token.append("\"");
- else
- break;
- } else if(look == 0) {
- break;
- } else {
- Token.append(look);
- GetChar();
- }
+ private void StringLiteral()
+ {
+ // Can't use match here 'cuz it consumes whitespace
+ // which we need to preserve inside the string.
+ // - pete
+ // Match('"');
+ if (look != '"')
+ Expected("\"");
+ else
+ {
+ GetChar();
+ StringBuffer Token = new StringBuffer();
+ for (;;)
+ {
+ if (look == '"')
+ {
+ GetChar();
+ SkipWhite(); //potential white space here
since it doesnt matter up to the operator
+ if (look == '"')
+ Token.append("\"");
+ else
+ break;
+ }
+ else if (look == 0)
+ {
+ break;
+ }
+ else
+ {
+ Token.append(look);
+ GetChar();
+ }
+ }
+ tokens.add(new StringPtg(Token.toString()));
}
- tokens.add(new StringPtg(Token.toString()));
- }
+ }
/** Recognize and Translate a Multiply */
private void Multiply(){
@@ -590,11 +631,7 @@
/** Parse and Translate an Expression */
private void Expression() {
- if (IsAddop(look)) {
- EmitLn("CLR D0"); //unaryAdd ptg???
- } else {
- Term();
- }
+ Term();
while (IsAddop(look)) {
if (look == '+' ) Add();
else if (look == '-') Subtract();
@@ -725,7 +762,7 @@
private void setRootLevelRVA(Node n, int formulaType) {
//Pg 16, excelfileformat.pdf @ openoffice.org
Ptg p = (Ptg) n.getValue();
- if (formulaType == this.FORMULA_TYPE_NAMEDRANGE) {
+ if (formulaType == FormulaParser.FORMULA_TYPE_NAMEDRANGE) {
if (p.getDefaultOperandClass() == Ptg.CLASS_REF) {
setClass(n,Ptg.CLASS_REF);
} else {
1.12 +2 -2
jakarta-poi/src/java/org/apache/poi/hssf/record/formula/AbstractFunctionPtg.java
Index: AbstractFunctionPtg.java
===================================================================
RCS file:
/home/cvs/jakarta-poi/src/java/org/apache/poi/hssf/record/formula/AbstractFunctionPtg.java,v
retrieving revision 1.11
retrieving revision 1.12
diff -u -r1.11 -r1.12
--- AbstractFunctionPtg.java 31 Aug 2003 06:16:56 -0000 1.11
+++ AbstractFunctionPtg.java 1 Jan 2004 07:51:34 -0000 1.12
@@ -536,7 +536,7 @@
functionData[25][0]=new Byte(Ptg.CLASS_VALUE);functionData[25][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[25][2]=new Integer(1);
functionData[26][0]=new Byte(Ptg.CLASS_VALUE);functionData[26][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[26][2]=new Integer(1);
functionData[27][0]=new Byte(Ptg.CLASS_VALUE);functionData[27][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[27][2]=new Integer(2);
- functionData[28][0]=new Byte(Ptg.CLASS_VALUE);functionData[28][1]=new
byte[] {Ptg.CLASS_REF};functionData[28][2]=new Integer(-1);
+ functionData[28][0]=new Byte(Ptg.CLASS_VALUE);functionData[28][1]=new
byte[] {Ptg.CLASS_VALUE, Ptg.CLASS_REF};functionData[28][2]=new Integer(-1);
functionData[29][0]=new Byte(Ptg.CLASS_VALUE);functionData[29][1]=new
byte[] {Ptg.CLASS_REF};functionData[29][2]=new Integer(-1);
functionData[30][0]=new Byte(Ptg.CLASS_VALUE);functionData[30][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[30][2]=new Integer(2);
functionData[31][0]=new Byte(Ptg.CLASS_VALUE);functionData[31][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[31][2]=new Integer(3);
@@ -571,7 +571,7 @@
functionData[61][0]=new Byte(Ptg.CLASS_VALUE);functionData[61][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[61][2]=new Integer(3);
functionData[62][0]=new Byte(Ptg.CLASS_VALUE);functionData[62][1]=new
byte[] {Ptg.CLASS_REF};functionData[62][2]=new Integer(-1);
functionData[63][0]=new Byte(Ptg.CLASS_VALUE);functionData[63][1]=new
byte[] {Ptg.CLASS_REF};functionData[63][2]=new Integer(1);
- functionData[64][0]=new Byte(Ptg.CLASS_VALUE);functionData[64][1]=new
byte[] {Ptg.CLASS_REF};functionData[64][2]=new Integer(-1);
+ functionData[64][0]=new Byte(Ptg.CLASS_VALUE);functionData[64][1]=new
byte[] {Ptg.CLASS_VALUE, Ptg.CLASS_REF};functionData[64][2]=new Integer(-1);
functionData[65][0]=new Byte(Ptg.CLASS_VALUE);functionData[65][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[65][2]=new Integer(3);
functionData[66][0]=new Byte(Ptg.CLASS_VALUE);functionData[66][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[66][2]=new Integer(3);
functionData[67][0]=new Byte(Ptg.CLASS_VALUE);functionData[67][1]=new
byte[] {Ptg.CLASS_VALUE};functionData[67][2]=new Integer(1);
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
