Hello,
Cppcheck reported this:
[sc/source/core/opencl/op_statistical.cxx:5103] ->
[sc/source/core/opencl/op_statistical.cxx:5099]: (style) Found duplicate
branches for 'if' and 'else'
Indeed, we have:
5095 if (!pCurDVR->IsStartFixed() && pCurDVR->IsEndFixed())
5096 {
5097 ss << "gid0; endFlag = "<< nCurWindowSize
<<"-gid0;\n";
5098 }
5099 else if (pCurDVR->IsStartFixed() &&
!pCurDVR->IsEndFixed())
5100 {
5101 ss << "gid0; endFlag = gid0+"<< nCurWindowSize
<<";\n";
5102 }
5103 else
5104 {
5105 ss << "gid0; endFlag = gid0+"<< nCurWindowSize
<<";\n";
5106 }
See
http://opengrok.libreoffice.org/xref/core/sc/source/core/opencl/op_statistical.cxx#5095
Any idea what should it be?
Julien
--
View this message in context:
http://nabble.documentfoundation.org/cppcheck-Duplicate-branch-if-else-in-op-statistical-cxx-sc-opencl-tp4088219.html
Sent from the Dev mailing list archive at Nabble.com.
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice
