D23514: [platforms/x11] Never block on retrace, always present after paint

Vlad Zagorodniy noreply at phabricator.kde.org
Thu Sep 5 09:56:36 BST 2019


zzag added a comment.


  Given that the feature freeze is closer and closer, are you okay if your changes go to 5.18? That way we'll have more time to test these patches.

INLINE COMMENTS

> workspace.cpp:1594-1595
>              support.append(QStringLiteral("OpenGL 2 Shaders are used\n"));
>              support.append(QStringLiteral("Painting blocks for vertical retrace: "));
> -            if (m_compositor->scene()->blocksForRetrace())
> -                support.append(QStringLiteral(" yes\n"));
> -            else
> -                support.append(QStringLiteral(" no\n"));
> +            support.append(QStringLiteral(" removed\n"));
>              break;

No, we don't need to print this line anymore because it will always say "removed."

REPOSITORY
  R108 KWin

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

To: romangg, #kwin
Cc: zzag, alexeymin, kwin, LeGast00n, The-Feren-OS-Dev, sbergeron, jraleigh, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, Pitel, iodelay, crozbo, bwowk, ZrenBot, ngraham, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20190905/2cc5a543/attachment.html>


More information about the kwin mailing list