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

[julien2412]

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.
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice