Hi Tamas, On Fri, Aug 10, 2012 at 10:57:12AM +0200, [email protected] wrote: > + ::rtl::OUString aString;
Just for the future: you can use simply OUString here, in internal code the ::rtl:: prefix is no longer necessary. (I corrected this before pushing.) > + if ( aString.getLength() != 0 ) Anothor minor nit, there is isEmpty() for this purpose (forgot to correct it before pushing, just keep in mind for your next patch). Other than that, nice work, thanks! :) Miklos _______________________________________________ LibreOffice mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libreoffice
