D22295: Remove scene check on repaints

Vlad Zagorodniy noreply at phabricator.kde.org
Mon Jul 8 08:40:24 BST 2019


zzag added a comment.


  > If not we later abort anyway because the state is not 'On'.
  
  Can you provide link to this code?

INLINE COMMENTS

> composite.cpp:583-584
>  {
> -    if (!hasScene())
> -        return;
>      repaints_region += QRegion(x, y, w, h);

Is it correct though? We'll keep adding repaint regions even if compositing is disabled.

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


More information about the kwin mailing list