[Libreoffice] [PUSHED] Re: [PATCH 02/12] RTL_CONSTASCII_USTRINGPARAM in components cui options
pjacquod at alumni.ethz.ch
Wed Nov 17 12:51:59 PST 2010
Sharp eyes.. Just to keep you trainded..:-(
No really sorry, Despite reviewing diff, I did not catch this one.
Will take more care
On 11/17/2010 05:18 PM, Caolán McNamara wrote:
> On Tue, 2010-11-16 at 22:39 +0100, Pierre-André Jacquod wrote:
>> On 11/16/2010 10:37 PM, Pierre-André Jacquod wrote:
>>> being off for some days, here the collection of patches I produced in
> 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.
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
More information about the LibreOffice