D24755: [KStatusNotifierItem] Optionaly, do not take menu ownership

Anthony Fieroni noreply at phabricator.kde.org
Fri Oct 18 14:11:08 BST 2019


anthonyfieroni added inline comments.

INLINE COMMENTS

> kstatusnotifieritemprivate_p.h:158
>  
>      QMenu *menu;
>      QHash<QString, QAction *> actionCollection;

It should be QPointer<QMenu>, no other changes are needed.

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/20191018/c9e8d0fb/attachment.html>


More information about the Kde-frameworks-devel mailing list