RTL_CONSTASCII_(U)STRINGPARAM officially obsolete

Jan Holesovsky kendy at suse.cz
Thu Apr 5 23:44:23 PDT 2012


Hi Olivier,

On 2012-04-03 at 18:17 +0200, Lubos Lunak wrote:

> functionFoo( rtl::OUString(RTL_CONSTASCII_USTRINGPARAM("XXXXX")))
> 
> can be written as
> 
> functionFoo( "XXXXX" );

Thank you for cleaning up the RTL_CONSTASCII_USTRINGPARAM!  I just
noticed though that your recent commit

http://cgit.freedesktop.org/libreoffice/core/commit/?id=fe51974e6515f48a6e6881e5837027d08c8d66e9

changes

- xProps->setPropertyValue( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "DisableDataTableDialog" ) ),
+ xProps->setPropertyValue( ::rtl::OUString( "DisableDataTableDialog" ),

That should be:

- xProps->setPropertyValue( ::rtl::OUString( RTL_CONSTASCII_USTRINGPARAM( "DisableDataTableDialog" ) ),
+ xProps->setPropertyValue( "DisableDataTableDialog",

Can you please revisit the changes you have made, and remove the (now
unnecessary) explicit OUString construction too?

Thank you,
Kendy



More information about the LibreOffice mailing list