[Libreoffice] [PUSHED] Re: [PATCH 01/12] RTL_CONSTASCII_USTRINGPARAM in components/cui/...option

Caolán McNamara caolanm at redhat.com
Wed Nov 17 08:13:42 PST 2010


On Tue, 2010-11-16 at 22:37 +0100, Pierre-André Jacquod wrote:
> Hello,
> being off for some days, here the collection of patches I produced in
> between. 

Looks good, thanks for this, pushed, though with a few tiny mods, i.e. I
prefer, when we end up with e.g.

static ::rtl::OUString s_sNodeName
(::rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("FOO") ) );

to simplify the syntax of constructors to just

static ::rtl::OUString s_sNodeName (RTL_CONSTASCII_USTRINGPARAM("FOO"));

C.



More information about the LibreOffice mailing list