a few more comments ...

Michael Meeks michael.meeks at collabora.com
Tue Jun 6 12:17:31 UTC 2017


Hi Mohammed,

On 06/06/17 13:07, Mohammed Abdul Azeem wrote:
>             Now I read it again, I'm rather skeptical that:
> 
>     +        if( !maUsedBuffers.empty() )
...
>             is helpful. I'm not sure that we can re-use these buffers
>     efficiently
>     here - perhaps its better just to allocate new ones in the stream read
>     method; can you profile with and without that (would love to get rid of
>     the extra complexity if possible).
> 
> Here you go.
> 
> with UsedBuffers:
> https://demo.collaboracloudsuite.com/tdf/index.php/s/Hbsp5rI8FE7CHmF
> without
> UsedBuffers: https://demo.collaboracloudsuite.com/tdf/index.php/s/PIcWD261zVcwpWj

	Interesting =) well it seems to save a small amount of time - which is
surprising; the uno_type_sequence_reference2One seems to take rather a
long time - I imagine we could be more careful there.

	Anyhow - thanks for poking at it,

	ATB,

		Michael.

-- 
michael.meeks at collabora.com <><, Pseudo Engineer, itinerant idiot


More information about the LibreOffice mailing list