D24240: [effects/presentwindows] Port present windows to EffectQuickView

Vlad Zahorodnii <[email protected]>
Newsgroups gmane.comp.kde.devel.kwin
Message-ID <[email protected]>
zzag added inline comments.

INLINE COMMENTS

> presentwindows.cpp:238
>          effects->addRepaintFull();
> -    else if (!m_activated && m_motionManager.managingWindows() && !(m_closeWindow && m_closeView->isVisible())) {
> +    else if (!m_activated && m_motionManager.managingWindows() && !(m_closeView->isVisible())) {
>          // We have finished moving them back, stop processing

> m_closeView->isVisible()

This can potentially segfault if the user unchecked "Provide buttons to close the windows" in pw kcm.

REPOSITORY
  R108 KWin

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

To: davidedmundson, #kwin
Cc: zzag, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.