Re: cppcheck: duplicate branch for ParaPropertyPanel (svx) + interpr7.cxx (sc)

2014-04-16 Thread Caolán McNamara
On Fri, 2013-09-27 at 14:52 -0700, julien2412 wrote: > Hello, > > Cppcheck reported this: > sc/source/core/tool/interpr7.cxx > 94 else if(pNodeSet->nodeTab[0]->type == > XML_ELEMENT_NODE) > 95 { > 96 xmlNode

Re: cppcheck: duplicate branch for ParaPropertyPanel (svx)

2013-09-28 Thread Ivan Timofeev
On 28.09.2013 01:52, julien2412 wrote: > svx/source/sidebar/paragraph/ParaPropertyPanel.cxx > 309 duplicateBranch style Found duplicate branches for 'if' and 'else'. > 328 duplicateBranch style Found duplicate branches for 'if' and 'else'. > Indeed: > 309 if( Application::GetSetting

Re: cppcheck: duplicate branch for ParaPropertyPanel (svx)

2013-09-28 Thread Ivan Timofeev
On 28.09.2013 10:57, Ivan Timofeev wrote: > The code itself does not make sense too - > the images are swapped, but the tooltips are not... :) please ignore this brainfart, oh... ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists

cppcheck: duplicate branch for ParaPropertyPanel (svx) + interpr7.cxx (sc)

2013-09-27 Thread julien2412
grok.libreoffice.org/xref/core/svx/source/sidebar/paragraph/ParaPropertyPanel.cxx#309 I suppose it should be fixed but I don't know how. Any idea? Julien -- View this message in context: http://nabble.documentfoundation.org/cppcheck-duplicate-branch-for-Para