Jan Holesovsky kendy at suse.cz
Fri Apr 6 04:50:35 PDT 2012

Hi Olivier,

On 2012-04-06 at 09:15 -0300, Olivier Hallot wrote:

> Thanks for pointing this.. I have reviewed one of the patch and I hope
> that is ok now. If not, please let me know.

Thank you, looks great! :-)

> Also, please confirm if I get it right:
> - rtl::OUString aOUstring = OUString("foo");
> + rtl::OUString aOUString = "foo";

+rtl::OUString aOUString( "foo" );

is the preferred form, I'd say.

> and
> rtl::OUString aOUString[10];
> aOUString[4] = "foo";


> if I get it right, there will be an enormous refactoring under way...

:-)  The good thing is that all this is going to look much nicer now.

All the best,

More information about the LibreOffice mailing list