replacing OUString::valueOf(static_cast<sal_Int32>) ??

Noel Grandin noel at peralex.com
Wed Jan 16 22:40:00 PST 2013


On 2013-01-16 19:48, Lubos Lunak wrote:
> On Monday 14 of January 2013, Noel Grandin wrote:
>> OK, so I tried modifying my patch so that we have
>>     number(sal_Int64)
>>     number(float)
>>     number(double)
>>
>> At which point my unit tests fail when passing a 32-bit value in because
>> the compiler does not
>> know which overload to use - this is on 64-bit Ubuntu using gcc 4.7.2
>>
>> I can't add a sal_Int32 variant because I suspect that will make the
>> original problem come back, which was where
>> we started, with all the "static_cast<sal_Int32>" stuff.
>>
>> Any ideas?
>   You need to add overloads, for anything except signed/unsigned char/short:
>
>

Thanks, implemented that and pushed new version to gerrit:
https://gerrit.libreoffice.org/#/c/1625/



Disclaimer: http://www.peralex.com/disclaimer.html




More information about the LibreOffice mailing list