D22084: Span wayland objects to lifespan of the QApplication

[Anthony Fieroni]

anthonyfieroni added inline comments.

INLINE COMMENTS

> waylandintegration.cpp:95-99
>          connect(m_waylandBlurManager, &KWayland::Client::BlurManager::removed, this,
>              [this] () {
>                  m_waylandBlurManager->deleteLater();
> -                m_waylandBlurManager = nullptr;
>              }
>          );

Since qApp is the parent are they needed? When destroyed qApp will call delete on m_waylandBlurManager, no?

REPOSITORY
  R130 Frameworks integration plugin using KWayland

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

To: davidedmundson, #plasma
Cc: anthonyfieroni, zzag, plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190624/58d308b8/attachment-0001.html>