[Libreoffice] [PUSHED] Re: [PATCH] +question RTL_CONSTASCII_USTRINGPARAM for calc_core_tools

Caolán McNamara caolanm at redhat.com
Thu Nov 11 02:10:25 PST 2010


On Thu, 2010-11-11 at 09:29 +0100, Julien Nabet wrote:
> Hello,
> 
> Here's a patch for calc/sc/source/core/tools
> 
> I compiled it and it's ok. Hope it's ok !

It looks good to me, pushed, thanks for this.

> How can we replace createFromAscii in this kind of line :
> OUString::createFromAscii(aPropNames[i])

Can't really, at least not with a trivial change. Could always unroll
the loop and use

OUString::createFromAscii("first one");
OUString::createFromAscii("second one");

etc, but probably not worth it. I suggest skipping those and have a look
at e.g. svtools and svl for loads of easier hanging fruit in some shared
libs if calc is running out of easy ones to fix.

C.



More information about the LibreOffice mailing list