https://bz.apache.org/bugzilla/show_bug.cgi?id=60397
Alex Herve <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #36284|0 |1 is obsolete| | CC| |[email protected] --- Comment #12 from Alex Herve <[email protected]> --- Created attachment 37601 --> https://bz.apache.org/bugzilla/attachment.cgi?id=37601&action=edit Patch to fix performance when adding merged regions Here is a patch that uses the getCount() method on the ctMergeCells instead of the expensive sizeOfMergeCellArray(). It passes all tests and I added a new test just to check if the counts are valid both after adding the first region and then a second one (following the structure of the test from bug 63371). For my use case in my tool, using the SNAPSHOT with the fix decreases runtime by over 50% for creating a large excel file with many merged cells. -- 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]
