https://issues.apache.org/bugzilla/show_bug.cgi?id=48284
Josh Micich <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #24614|0 |1 is obsolete| | --- Comment #2 from Josh Micich <[email protected]> 2009-11-25 17:21:12 UTC --- Created an attachment (id=24621) --> (https://issues.apache.org/bugzilla/attachment.cgi?id=24621) Revised patch making FormulaParseException a public top-level type I originally added this exception quite some time back in the patch of bug 44504. The purpose of the exception is to tell the difference between internal POI errors and parsing errors caused by a bad user-supplied formula. I deliberately restricted visibility for two reasons - firstly because I didn't want to augment POI's public API and secondly because I wasn't confident that the formula parser did a good job of recognising formula errors yet. Having said that, I think the formula parser has improved greatly since then (almost thirty bugs fixed and more than a dozen additional enhancements) so it is probably more reliable in throwing FormulaParseException correctly now. As far as the API is concerned, I'd like to make this class a top level class perhaps: org.apache.poi.ss.formula.FormulaParseException It should be OK to make this change since no-one has been able to specifically catch this exception so far. I see in your patch you left the the exception unchecked (which I agree with). I'm also considering declaring this exception on Cell.setCellFormula() for clarity. It's slightly unusual to have declared unchecked exceptions but that seems like the best approach here. The attached patch file includes the changes described here and some related clean-up. Let me know if you think it needs further changes. Otherwise I'll apply it over the weekend. -- 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]
