diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2014-01-14 21:11:31 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2014-01-14 21:28:08 +0100 |
| commit | 7d0db7f0ee7c478cc58669121f517dfdf54eeaee (patch) | |
| tree | bd4c82a20403cae1184faa50b42216439b4def14 | |
| parent | 5ea58f534b9226ef8f5c9589066ce20e3c64e0d8 (diff) | |
This shall probably check nId != 0 && ..., not do a bitwise &
Change-Id: I2417f7bff3fc5f2bef6c804af12d1873802b0392
| -rw-r--r-- | framework/source/uielement/newmenucontroller.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/framework/source/uielement/newmenucontroller.cxx b/framework/source/uielement/newmenucontroller.cxx index 0472218e9359..9857350998e3 100644 --- a/framework/source/uielement/newmenucontroller.cxx +++ b/framework/source/uielement/newmenucontroller.cxx @@ -226,7 +226,7 @@ void NewMenuController::setAccelerators( PopupMenu* pPopupMenu ) for ( sal_uInt32 i = 0; i < nItemCount; i++ ) { sal_uInt16 nId( pPopupMenu->GetItemId( sal_uInt16( i ))); - if ( nId & ( pPopupMenu->GetItemType( nId ) != MENUITEM_SEPARATOR )) + if ( nId && ( pPopupMenu->GetItemType( nId ) != MENUITEM_SEPARATOR )) { aIds.push_back( nId ); aMenuShortCuts.push_back( aEmptyKeyCode ); |