D22270: Remove composite reset timer

Vlad Zagorodniy <[email protected]> Thu, 4 Jul 2019 16:03:14 +0000

Newsgroups	gmane.comp.kde.devel.kwin
Message-ID	<[email protected]>

zzag added inline comments.

INLINE COMMENTS

> zzag wrote in composite.cpp:115
>   connect(&compositeResetTimer, &QTimer::timeout, this, &Compositor::reinitialize);
>   compositeResetTimer.setSingleShot(true);
> 
> Only initializes the timer. I suppose we can drop this QTimer::singleShot thing, otherwise Compositor will be reinitialized (with no absolute good reason) several times during startup.

... unless I'm missing something. It's not obvious why QTimer::singleShot is here.

REPOSITORY
  R108 KWin

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

To: romangg, #kwin
Cc: zzag, kwin, LeGast00n, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart