[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
(similarly for equalsIgnoreAsciiCase and equalsIgnoreAsciiCaseAsciiL)
and aStr.getLen() in conditions by !aStr.isEmpty().


More information about the LibreOffice mailing list