D22270: Remove composite reset timer

Vlad Zagorodniy noreply at phabricator.kde.org
Thu Jul 4 17:03:14 BST 2019


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190704/31d8a73f/attachment.html>


More information about the kwin mailing list