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

--- Comment #2 from Dmitriy Likhten <[email protected]> ---
1) The logic in SXSSFSheet#autoSizeColumn for width calculation needs to be
moved over into some generic code to convert the width to that 256 inflated
number (sorry I'm still not 100% familiar with why that happens)

2) SheetUtil.getColumnWidth has a ton of logic, but that logic is intertwined
with looping through every single row in the sheet for a given column. It would
be really useful to have a SheetUtil.getCellWidth which would allow me to do my
count-tracking manually *but* it would still re-use the logic that you guys
built for figuring out the width of a column. That way I don't actually have
the ability to get things wrong.

3) Unit testing: Since all the interesting logic is going to be left in *your*
maintained code, my testing will be simple. I can stub all the width
calculations and just focus on my (trivial) summation logic. This way my code
cannot introduce new width calculation bugs.

-- 
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