https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
Dominik Stadler changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
--- Comment #6 from PJ Fanning ---
I accidentally committed the change to increase the limit to 255 operands in
https://svn.apache.org/viewvc?view=revision=1800949 - I will add some
test cases tomorrow.
--
You are
https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
--- Comment #5 from Javen O'Neal ---
Related: bug 58975
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe,
https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
--- Comment #4 from PJ Fanning ---
Greg - that seems like a pragmatic solution - we should just change to this:
private static final int DEFAULT_MAX_NUM_OPERANDS =
SpreadsheetVersion.EXCEL2007.getMaxFunctionArgs();
--
https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
--- Comment #3 from Greg Woolsey ---
Is it really so bad to just use the larger version constant in formula
calculation? Are there cases where we really _need_ to throw an error or stop
processing arguments after 30 for
https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
--- Comment #2 from PJ Fanning ---
Passing a SpreadsheetVersion around in all these function APIs looks like it
will be a big change.
I think we might need to use a ThreadLocal, similar to LocaleUtil.
--
You are
https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
--- Comment #1 from PJ Fanning ---
Floriano - would you be in a position to submit a patch?
org.apache.poi.ss.SpreadsheetVersion has the operand count as a param - 30 for
Excel97 and 255 for Excel2007.
So maybe the fix is
https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
Floriano changed:
What|Removed |Added
CC|florianogiambelli@tagetik.c |
https://bz.apache.org/bugzilla/show_bug.cgi?id=61252
Floriano changed:
What|Removed |Added
OS||All