AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Check if there is an old item before removing it

Browse Source 
If something had changed with the menu item before it had actually been
originally added to the native menu then this would crash when trying
to remove a null item from the native menu.  This prevents that from
happening.

Change-Id: I8d78b5504759225364d5fd051a23c8d6dbd7d1eb
Reviewed-by: James Turner <james.turner@kdab.com>
Reviewed-by: Morten Johan Sørvig <morten.sorvig@digia.com>
This commit is contained in:
Andy Shaw 2012-10-10 11:23:31 +02:00 committed by The Qt Project
parent f144d538fd
commit dc0927f2b8
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/plugins/platforms/cocoa/qcocoamenu.mm
View File

@ -219,9 +219,8 @@ void QCocoaMenu::syncMenuItem(QPlatformMenuItem *menuItem)
if (cocoaItem->sync() != oldItem) {
// native item was changed for some reason
if (!wasMerged) {
if (!wasMerged && oldItem)
[m_nativeMenu removeItem:oldItem];
}
QCocoaMenuItem* beforeItem = itemOrNull(m_menuItems.indexOf(cocoaItem) + 1);
insertNative(cocoaItem, beforeItem);