[Differential] [Changed Subscribers] D3099: Fix viewport restore in GLRenderTarget::popRenderTarget

[broulik (Kai Uwe Broulik)]

broulik added inline comments.

INLINE COMMENTS

> kwinglutils.cpp:1181
>      } else {
> -        glViewport (0, 0, s_virtualScreenSize.width(), s_virtualScreenSize.height());
> +        glViewport (-s_virtualScreenGeometry.x(), s_virtualScreenGeometry.height() - s_virtualScreenSize.height() - s_virtualScreenGeometry.y(),
> +                           s_virtualScreenSize.width(), s_virtualScreenSize.height());

I don't understand the maths, why is the "x" the negative virtualScreenGeometry and "y" some formula?

REPOSITORY
  rKWIN KWin

BRANCH
  render-target-viewport

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, #kwin, #plasma_on_wayland, mart
Cc: broulik, plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161019/70328821/attachment.html>