Cppcheck Same expression on both sides of '|' in PresenterPaneBorderManager.cxx (sdext)

julien2412 serval2412 at yahoo.fr
Fri Jan 31 06:49:49 PST 2014


Hello,

Cppcheck reported this:
sdext/source/presenter/PresenterPaneBorderManager.cxx
171	duplicateExpression	style	Same expression on both sides of '|'.
    switch (nCode)
    {
        case mnOutside | mnOutside:<--- Same expression on both sides of
'|'.
        case mnOutside | mnLeft:

See
http://dev-builds.libreoffice.org/cppcheck_reports/master/733.html#line-171

I thought about putting this:
        case mnOutside:
        case mnOutside | mnLeft:

Is it ok or do I miss something?

Julien



--
View this message in context: http://nabble.documentfoundation.org/Cppcheck-Same-expression-on-both-sides-of-in-PresenterPaneBorderManager-cxx-sdext-tp4094873.html
Sent from the Dev mailing list archive at Nabble.com.


More information about the LibreOffice mailing list