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