On Thu, 2015-11-26 at 19:03 +0100, Markus Mohrhard wrote:
> please add a performance test that ensures that future refactorings
> won't regress here. In general any patch claiming performance
> improvements should be accompanied by a performance test to make sure
Great idea =) and easy to do too; Kendy ?
> Related to the code I wanted to mention that the getenv calls should
> be cached in a static variable as they might become expensive when
I was concerned by that too =) I believe the plan is to bin those; but
yes - good to check the perf tests for sure; and to have some nice
before/after step here.
ATB,
Michael.
--
michael.me...@collabora.com <><, Pseudo Engineer, itinerant idiot
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
