From 1235801aba23d033808a44516784dfe454e67b8a Mon Sep 17 00:00:00 2001
From: Glen Stampoultzis <glens@apache.org>
Date: Thu, 29 Apr 2004 07:16:36 +0000
Subject: [PATCH] Bugfix:
 http://issues.apache.org/bugzilla/show_bug.cgi?id=24925

git-svn-id: https://svn.apache.org/repos/asf/jakarta/poi/trunk@353553 13f79535-47bb-0310-9956-ffa450edef68
---
 .../apache/poi/hssf/model/FormulaParser.java    | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/src/java/org/apache/poi/hssf/model/FormulaParser.java b/src/java/org/apache/poi/hssf/model/FormulaParser.java
index e95bba1ff..977b87843 100644
--- a/src/java/org/apache/poi/hssf/model/FormulaParser.java
+++ b/src/java/org/apache/poi/hssf/model/FormulaParser.java
@@ -738,17 +738,18 @@ end;
     }
     
     private void setParameterRVA(Node n, int formulaType) {
-        Ptg p = (Ptg) n.getValue();
+        Ptg p = n.getValue();
+        int numOperands = n.getNumChildren();
         if (p instanceof AbstractFunctionPtg) {
-            int numOperands = n.getNumChildren();
-            for (int i =0;i<n.getNumChildren();i++) {
+            for (int i =0;i<numOperands;i++) {
                 setParameterRVA(n.getChild(i),((AbstractFunctionPtg)p).getParameterClass(i),formulaType);
-                if (n.getChild(i).getValue() instanceof AbstractFunctionPtg) {
-                    setParameterRVA(n.getChild(i),formulaType);
-                }
-            }  
+//                if (n.getChild(i).getValue() instanceof AbstractFunctionPtg) {
+//                    setParameterRVA(n.getChild(i),formulaType);
+//                }
+                setParameterRVA(n.getChild(i),formulaType);
+            }
         } else {
-            for (int i =0;i<n.getNumChildren();i++) {
+            for (int i =0;i<numOperands;i++) {
                 setParameterRVA(n.getChild(i),formulaType);
             }
         }