[PATCH] Reduced duplicate code (simian) / l10ntools

Ivan Timofeev timofeev.i.s at gmail.com
Sun Feb 5 10:52:52 PST 2012


Hi Christina,

05.02.2012 20:26, Chr. Rossmanith пишет:
> there were three blocks of nearly identical code.

Awful! Thanks for working on this task.

> I've moved them to a method ResData2Output(). Please review.

Looks nice. All of the differences seem to be handled...

 > bWriteNoSlash seems to be always false

Yes. And ByteString sTmp is unused.

> Maybe the spelling of Semikolon should be changed while at it...

Yes, I think.

> and ByteString->rtl::OString...

I think it should be separated... :)

Hmm, interesting, where we can test the "new" code, just in case?

Ivan


More information about the LibreOffice mailing list