Hi Mark,

On Sat, 2016-01-16 at 21:36 +0800, Mark Hung wrote:
> I'd look into performance issue. 

        Thanks =) should be an easy tweak there.

> Is there any benchmark or unit test that I can use to check
> performance enhancement ?

        Looking at a profile quickly; it seems that I get about 300k calls to
this method while saving - 140m pcycles - which to be fair is 0.1% of
save time =) so in fact not a big issue now I look more carefully [ at
least for XLSX. DOCX etc. may use it more, but there is far less XML
content there ].

> And what do you mean by regression test for the characters?

        Ah - that is simple; just create a document, add a character string
that hits this code-path, save as XLSX (using fast-parser) and re-load
it, and check that the string is the same.

        sc/qa/unit/subsequent_export-test.cxx

        Is a great piece of code to read for that =) but of course calc
focused; more ideally we'd test just the FastSerializer =)

        Thanks !

                Michael.

-- 
 michael.me...@collabora.com  <><, Pseudo Engineer, itinerant idiot

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to