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.

_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to