graesslin added a comment.

  @hein, @davidedmundson if the main concern is too often update of title I 
suggest we implement limiting of updates in KWin. So the batch would happen in 
KWin, taken away stress from KWin and indirectly also from Plasma.
  
  Against updating icons it won't help, though as KWin doesn't set this.

INLINE COMMENTS

> xwindowsystemeventbatcher.cpp:48
> +            if (!m_timerId) {
> +                m_timerId = startTimer(BATCH_TIME);
> +            }

if we can depend on Qt 5.9 you could simplify the BATCH_TIME to either:

  startTimer(std::chrono::milliseconds(10));

or together with C++14 to:

  using namespace std::chrono_literals;
  startTimer(10ms);

REPOSITORY
  R120 Plasma Workspace

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

To: davidedmundson, #plasma
Cc: broulik, hein, graesslin, plasma-devel, ZrenBot, progwolff, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

Reply via email to