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
