[Libreoffice] [PATCH] [PUSHED] remove String with OUString
David Tardon
dtardon at redhat.com
Tue Mar 22 00:11:16 PDT 2011
On Tue, Mar 15, 2011 at 01:10:58PM +0100, Xisco Faulí wrote:
> Hello,
>
>
> It's the first time I do it so I searched for previous commits in order to
> get a referent. i couldn't find any example for String.AssignAscii so I
> replace it with OUStringBuffer.appendAscii.
Yeah, that is good. I only added RTL_CONSTASCII_STRINGPARAM around
string literals. I also replaced occurrences of
aStr.equals(rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("foo")))
by
aStr.equalsAsciiL(RTL_CONSTASCII_STRINGPARAM("foo"))
(similarly for equalsIgnoreAsciiCase and equalsIgnoreAsciiCaseAsciiL)
and aStr.getLen() in conditions by !aStr.isEmpty().
D.
More information about the LibreOffice
mailing list