[PUSHED-3-6] fix for fdo#55565 regression

Michael Meeks michael.meeks at suse.com
Thu Oct 4 02:20:48 PDT 2012


Hi Noel,

On Wed, 2012-10-03 at 10:15 +0100, Noel Power wrote:
> please consider this patch
> 
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=fdb42c397f1d15d05a74492a233df72d832cab1c

	Pushed; IIRC using:

- if ( aProps[ i ].Name.equals( "TextPreRotateAngle" ) ..
+ if ( aProps[ i ].Name == "TextPreRotateAngle" ...

	Is more efficient (by the power of Lubos' cunning), avoiding a dynamic
allocation, and a couple of atomic operations each comparison. Though
perhaps I remember wrong.

	If that's right, would it make sense to have an easy-hack to go over
all re-based modules hunting and replacing .equals with == ?

$ git grep 'equals(' | grep \.cxx | less

	Or am I loosing it ;-)

	ATB,

		Michael.

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



More information about the LibreOffice mailing list