[PATCH] fdo#46581 Avoid temporary rtl::OUString (LGPLv3/MPL)

Michael Meeks michael.meeks at suse.com
Wed Apr 18 04:35:47 PDT 2012


Hi there,

	It's a nice patch :-) however - I believe that Lubos just did a chunk
of work to make this even easier to read, such that we can do:

-            if ( m_nxtSym == OUString(RTL_CONSTASCII_USTRINGPARAM( "\"" )) )
+            if ( m_nxtSym == "\"" )

	Which will turn into something like:

+            if ( m_nxtSym.equalsAsciiL(RTL_CONSTASCII_USTRINGPARAM( "\"" )) )

	But be a lot shorter :-) any chance you can adapt your patch to that
style ? [ it'd be good to fix other instances of this specific wasteful
temporary around the place too ].

	It'd be great to have a blanket license mail on the list we can link
into:

	http://wiki.documentfoundation.org/Development/Developers

	If that's possible :-)

	Thanks !

		Michael.

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



More information about the LibreOffice mailing list