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

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to