Bug 303093 - Don't use OpenGL matrix stack in OpenGL 2 backend
Summary: Don't use OpenGL matrix stack in OpenGL 2 backend
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: scene-opengl (show other bugs)
Version: git master
Platform: unspecified Linux
: NOR task
Target Milestone: 4.10
Assignee: KWin default assignee
URL: https://git.reviewboard.kde.org/r/105...
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-06 07:59 UTC by Martin Flöser
Modified: 2012-07-22 08:15 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 4.10
Sentry Crash Report:
mgraesslin: ReviewRequest+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Flöser 2012-07-06 07:59:41 UTC
Currently the GL Matrix Stack is also used with OpenGL 2. That is pushMatrix, multMatrix and popMatrix are executed although this does not influence the rendering at all. The OpenGL 1 matrices are not passed to the shaders.

The OpenGL 2 stack can be improved by checking whether the shaders are used and not executing the legacy OpenGL call.
Comment 1 Martin Flöser 2012-07-22 08:15:57 UTC
Git commit 42678c60bd683bcaa54a9f192d92ea28df5ea3ba by Martin Gräßlin.
Committed on 06/07/2012 at 09:55.
Pushed by graesslin into branch 'master'.

Don't use GL Matrix Stack on OpenGL 2 backend

Currently the GL Matrix Stack is also used with OpenGL 2.
That is pushMatrix, multMatrix and popMatrix are executed
although this does not influence the rendering at all. The
OpenGL 1 matrices are not passed to the shaders.

With this change the calls to the matrix stack are no longer
executed if the Shader based backend is used. This means we
have a few less matrix multiplications in the rendering.

Mostly affects a few effects which have not yet completely be
ported over to OpenGL 2.
FIXED-IN: 4.10
REVIEW: 105455

M  +12   -0    kwin/libkwineffects/kwinglutils.cpp

http://commits.kde.org/kde-workspace/42678c60bd683bcaa54a9f192d92ea28df5ea3ba