D24667: Activate when both associatedWidget and menu are null
Konrad Materka
noreply at phabricator.kde.org
Tue Oct 15 16:11:28 BST 2019
kmaterka added inline comments.
INLINE COMMENTS
> kstatusnotifieritem.cpp:619
>
> - if (d->associatedWidget == d->menu) {
> + if (d->associatedWidget && d->associatedWidget == d->menu) {
> d->statusNotifierItemDBus->ContextMenu(pos.x(), pos.y());
This should fix a situation when both are NULL
> kstatusnotifieritemdbus_p.cpp:293
> {
> - if (m_statusNotifierItem->d->associatedWidget == m_statusNotifierItem->d->menu) {
> - ContextMenu(x, y);
This check is duplicated, it should be safe to remove it here.
REPOSITORY
R289 KNotifications
REVISION DETAIL
https://phabricator.kde.org/D24667
To: kmaterka, davidedmundson, broulik, nicolasfella
Cc: 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/20191015/4c0998e7/attachment.html>
More information about the Kde-frameworks-devel
mailing list