https://issues.apache.org/bugzilla/show_bug.cgi?id=44675


Josh Micich <[EMAIL PROTECTED]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED




--- Comment #3 from Josh Micich <[EMAIL PROTECTED]>  2008-05-02 21:13:25 PST ---
Fixed in svn r652994.

Prior to r641185 SUM would get parameter operand class Ptg.CLASS_REF.  When I
refactored the function metadata (to recognise var arg functions) I left the
parameter operand classes anattached, because there were no junits to confirm
this requirement. I added such a junit with this most recent fix.

The 'operand class' stuff is not well tested by POI.  Furthermore Excel often 
tolerates incorrect token operand classes, so when POI gets it wrong the errors
are not always immediately apparent.

Another enhancement I added here was to validate function arg counts when
parsing formulas.  None of the existing junits were disturbed by this (they all
supply correct function arg counts in every case).


-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to