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

Alex Herve <a.he...@rac.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #36284|0                           |1
        is obsolete|                            |
                 CC|                            |a.he...@rac.de

--- Comment #12 from Alex Herve <a.he...@rac.de> ---
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: dev-unsubscr...@poi.apache.org
For additional commands, e-mail: dev-h...@poi.apache.org

Reply via email to