https://bz.apache.org/bugzilla/show_bug.cgi?id=61858

--- Comment #1 from PJ Fanning <fannin...@yahoo.com> ---
Thanks Phaneeswar for the proposed patch.
It does modify a public API (the cell method) and this is not something that we
don't do without deprecating the existing method. The replacement method can be
added and the deprecated version can call the replacement version.

It's not generally required but I personally much prefer to review patches
added via github - see https://github.com/apache/poi

My view on the patch, as is, is that it may not be necessary. If you look at
the XLSX2CSV example in Poi, you will see that you can provide your own
DataFormatter and with a custom DataFormatter, you could omit the number
formatting and just represent the numbers in plain format.

https://github.com/apache/poi/blob/trunk/src/examples/src/org/apache/poi/xssf/eventusermodel/XLSX2CSV.java

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Reply via email to