D24755: [KStatusNotifierItem] Optionaly, do not take menu ownership
Anthony Fieroni
noreply at phabricator.kde.org
Sat Oct 19 15:30:50 BST 2019
anthonyfieroni added inline comments.
INLINE COMMENTS
> kstatusnotifieritem.cpp:454
> Qt::WindowFlags oldFlags = d->menu->windowFlags();
> d->menu->setParent(nullptr);
> d->menu->setWindowFlags(oldFlags);
Do not take ownership of the menu and delete it when it does not have a parent. takeOwnership is wrong approach, you can remove it.
REPOSITORY
R289 KNotifications
REVISION DETAIL
https://phabricator.kde.org/D24755
To: kmaterka, #frameworks, davidedmundson, broulik, nicolasfella
Cc: anthonyfieroni, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20191019/26a08863/attachment.html>
More information about the Kde-frameworks-devel
mailing list