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

Caolán McNamara caolanm at redhat.com
Fri Feb 7 12:57:54 CET 2014


On Fri, 2014-01-31 at 06:49 -0800, julien2412 wrote:
> Hello,
> 
> Cppcheck reported this:
> sdext/source/presenter/PresenterPaneBorderManager.cxx
> 171	duplicateExpression	style	Same expression on both sides of '|'

> 
> I thought about putting this:
>         case mnOutside:
>         case mnOutside | mnLeft:
> 
> Is it ok or do I miss something?

I think you should go ahead with that. Seems the sane thing to do.

C.



More information about the LibreOffice mailing list