[Libreoffice] [PATCH] Replace (Byte)String with O(U)String in sw
caolanm at redhat.com
Mon Oct 3 02:20:49 PDT 2011
On Mon, 2011-10-03 at 10:07 +0100, Michael Meeks wrote:
> Hi Christina,
> On Sun, 2011-10-02 at 20:52 +0200, Chr. Rossmanith wrote:
> > could someone please review this small patch. I'll extend the
> > replacement (and push on my own) but want to be sure that there are no
> > mistakes. At least it seems to build :-)
> Looks great to me :-)
yup, fine, no errors there.
FWIW, personally, I've tended towards...
But its just an OSL_FAIL, so whatever titchy (if any) difference using a
OStringBuffer and RTL_CONSTASCII_STRINGPARAM makes is irrelevant there,
and the same goes for the output encoding really, its going to be ascii,
so ASCII, UTF-8, system encoding for the error message will all be
equivalent in the real world.
More information about the LibreOffice