[Libreoffice] [PUSHED] Re: [PATCH] EasyHack RTL_CONSTASCII_USTRINGPARAM in starmath
michael.meeks at novell.com
Mon Nov 1 04:52:55 PDT 2010
On Sat, 2010-10-30 at 22:24 +0100, Caolán McNamara wrote:
> > This changes the only occurrence in writer/starmath.
> Thanks, Hmm, an empty string. I wonder why it using ::createFromAscii
> for the empty string case. I suggest that...
Interesting question that needs testing. Since OUString() just gives
you a new reference counted pointer to a singleton 'empty string'
Perhaps the hash is relying on a different pointer value there ? [ too
lazy to read it just now but ... ;-]
> Digging for UNOOUSTRING I see that indeed it is a define for
> rtl::OUString, i.e.
> so yes they can be replaced as well. I think I'd prefer to see them
> replaced with rtl::OUString(RTL_CONSTASCII_USTRINGPARAM(... instead of
I hope eventually, when we have in-line strings fixed up - we will have
a rather shorter macro to instantiate strings ;-)
michael.meeks at novell.com <><, Pseudo Engineer, itinerant idiot
More information about the LibreOffice