[REVIEW 3-5] Custom Shapes compile speedup ...

Radek Doulik rodo at novell.com
Fri Mar 2 10:51:19 PST 2012


Hi Michael,

thanks. The patch looks good to me. Pushed.

It doesn't change the properties data, so it shouldn't break anything. I
guess I will cook another sample document containing all presets for
easier future testing.

Cheers
Radek

On Fri, 2012-03-02 at 17:44 +0000, Michael Meeks wrote:
> Hi guys,
> 
> 	I'd like to propose this for cherry-picking:
> 
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=f291fb57d087de41385a22dfacd8b694d8b9abc7
> 
> 	the rather large and tedious patch it generates is this:
> 
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=d4157cefc5416ebea75a460e7dbff91293bff321
> 
> 	Which also needs picking. Wrt. compile times here is what it does for
> me:
> 
> 	        Linux       Windows VM
> 	Before: 1m13.067s   23m10.179
> 	After:  0m16.993s    0m5.808
> 
> 	So - it really did turn out to be faster to fix than to compile ;-) Of
> course, prolly I -totally- broke it, but it still loads the nice
> customshapes-all document (at least for me) - I'd like Radek to review
> it before we go anywhere.
> 
> 	I'm just poking to see if I can squeeze a bit more time & space out of
> this, but I'm fairly confident that this is the major win.
> 
> 	HTH,
> 
> 		Michael.
> 




More information about the LibreOffice mailing list