D22295: Remove scene check on repaints

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

INLINE COMMENTS

> romangg wrote in composite.cpp:583-584
> So what we do more in this case is a single region addtion to repaints_region, right?

Yes, we'll be adding regions for no reason. Even though we don't hit this if statement in 99%, 1% is still there.

I'm all for mergin setCompositeTimer and scheduleRepaint methods, but I think we have to keep this check until X11 support is dropped.

REPOSITORY
  R108 KWin

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

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
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.