[Libreoffice] [PUSHED] Re: [PATCH 02/12] RTL_CONSTASCII_USTRINGPARAM in components cui options

Caolán McNamara caolanm at redhat.com
Wed Nov 17 08:18:35 PST 2010


On Tue, 2010-11-16 at 22:39 +0100, Pierre-André Jacquod wrote:
> On 11/16/2010 10:37 PM, Pierre-André Jacquod wrote:
> > Hello,
> > being off for some days, here the collection of patches I produced in
> > between.

Mostly good, but careful here, see...

-        aAutoStr += ::rtl::OUString::createFromAscii( " (" );
+        aAutoStr += ::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("(") );

you changed the string by accident from a bracket with a preceding space
to one with no preceding space, clearly what's between "" has to remain
the same :-). Fixed that typo and the rest looks good, pushed.

C.




More information about the LibreOffice mailing list