[PATCH reworked] Reduced duplicate code (simian) / l10ntools

Ivan Timofeev timofeev.i.s at gmail.com
Mon Feb 6 02:50:51 PST 2012


On 06.02.2012 13:34, Chr. Rossmanith wrote:
> I have replaced parameter ResData *pResData (which became ununsed) with
> parameter rtl::OString sTextType. Or should it be rtl::OString&
> sTextType instead?

A reference is better, however it is not really performance-critical I 
think. But if so, reference variables are 'r'-prefixed.

And I'd remove the return statement at the end of the function, it is 
redundant there.

Ivan


More information about the LibreOffice mailing list