replacing OUString::valueOf(static_cast<sal_Int32>) ??
Michael Meeks
michael.meeks at suse.com
Wed Jan 9 09:58:26 PST 2013
On Wed, 2013-01-09 at 18:26 +0100, Lubos Lunak wrote:
> > but you'll at least implicitly cast the same way on all platforms (since
> > the sal types don't map to arbitrary types, but to types of a particular
> > size).
>
> The explicit cast will also always be the same way on all platforms.
Sure - but the explicit cast is not required :-) people can omit it,
compile perfectly cleanly on 64bit, commit - and then all the 32bit
systems break :-)
Unless I'm missing something, there is no way beyond: "do manual work
regularly to fix it" to deal with that sort of fall-out - and that's the
primary reason why we might want to prefer valueOfInt32() and
deprecating valueOf() - even though it's longer.
At least in my mind :-) but we're starting to bike-shed here... I
didn't see anyone volunteering to do the actual batch cleanup there ;-)
Regards,
Michael.
--
michael.meeks at suse.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice
mailing list