broulik added inline comments.

INLINE COMMENTS

> foldermodel.cpp:1854
>      menu->popup(m_menuPosition);
> -    connect(menu, &QMenu::aboutToHide, [menu]() { menu->deleteLater(); });
> +    connect(menu, &QMenu::aboutToHide, [this, menu]() {
> +        menu->deleteLater();

Use four argument connect since now you access `this` but given you delete the 
menu anyway,  this change probably isn't neccessary to begin with

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D16368

To: hein, ngraham, broulik, davidedmundson
Cc: davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart

Reply via email to