[Libreoffice] Double Duplicate branches for if and else in sw/source/core/doc/ftnidx.cxx

Caolán McNamara caolanm at redhat.com
Wed Feb 1 06:50:16 PST 2012


On Tue, 2012-01-31 at 13:29 -0800, julien2412 wrote:
> Hello,
> 
> Cppchecks reports this :
> core/sw/source/core/doc/ftnidx.cxx
> 195	duplicateBranch	style	Found duplicate branches for if and else.
> 284	duplicateBranch	style	Found duplicate branches for if and else.
...

> Could we just remove the if in both cases or must something be added ?

I can't guess if the intention might have been to do something
different. I suggest you go ahead and just remove the duplication and
simplify it.

C.



More information about the LibreOffice mailing list