luebking added inline comments.

INLINE COMMENTS

> Kanedias wrote in scene_opengl.cpp:713
> How come! There's a condition on !m_cursorTexture for that, no? Or is 
> Cursor::cursorChanged itself called on every paint?

Sorry, I missed that the "if (!m_cursorTexture) {" condition still covers these 
calls - to my excuse: inline comments in phabricator are rather messy :-P

So the above worries are void - the only remainders are
a) possible glBlend restorage
b) null'ing m_cursorTexture on empty images to skip rendering (no idea how 
relevant that is) ie. not using the pointer vaildity as indicator for a 
required reset.

REPOSITORY
  R108 KWin

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

To: Kanedias, graesslin, davidedmundson
Cc: luebking, plasma-devel, kwin, #kwin, ZrenBot, spstarr, progwolff, 
lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, eliasp, sebas, 
apol, mart, hein, lukas

Reply via email to