[PATCH] Reduced duplicate code (simian) / l10ntools
Ivan Timofeev
timofeev.i.s at gmail.com
Sun Feb 5 11:52:20 PST 2012
05.02.2012 23:37, Chr. Rossmanith пишет:
>>> I've moved them to a method ResData2Output(). Please review.
>>
>> Looks nice. All of the differences seem to be handled...
Oh, oh, oh! No, we must pass all the different values as paramerers to
the function.
Because it was
if ( pEntry && pResData->bText ) {
...
}
if ( pEntry && pResData->bQuickHelpText ) {
...
}
if ( pEntry && pResData->bTitle ) {
...
}
, but not "if - else if - else if"; and thus nType and sOutput will be
always the same in these 3 calls to ResData2Output.
I'm sorry...
Ivan
More information about the LibreOffice
mailing list