--0b5e1cca9f084d158f80ee1020da6eda
Content-Transfer-Encoding: 7bit
Content-Type: text/plain; charset="ascii"
Mime-Version: 1.0

graesslin added a comment.


  In https://phabricator.kde.org/D8040#150577, @davidedmundson wrote
  
  > If you do want to set this based on the initial scene backend, skip using this method and call the static method on QQuickWindow instead.
  
  
  After reading through the docs I think the combination of both is fine: We can use the QtQuickSettings to have a base. But if we init a QPainter compositor we just enforce software through the QQuickWindow API. That should (TM) work as well. Dynamic switching is then not required. AFAIK the only "dangerous" combination would be QPainter compositor and OpenGL compositing. The other 3 combinations should just work fine.

REPOSITORY
  R108 KWin

BRANCH
  master

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

To: davidedmundson, #plasma, graesslin
Cc: graesslin, plasma-devel, kwin, #kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart

--0b5e1cca9f084d158f80ee1020da6eda
Content-Transfer-Encoding: quoted-printable
Content-Type: text/html; charset="ascii"
Mime-Version: 1.0

<table><tr><td style=3D"">graesslin added a comment.
</td><a style=3D"text-decoration: none; padding: 4px 8px; margin: 0 8px 8px=
; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; back=
ground-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1=
f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href=
=3D"https://phabricator.kde.org/D8040" rel=3D"noreferrer">View Revision</a>=
</tr></table><br /><div><div><blockquote style=3D"border-left: 3px solid #8=
C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style=3D"font-style: normal;
          padding-bottom: 4px;">In <a href=3D"https://phabricator.kde.org/D=
8040#150577" style=3D"background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel=3D"noreferrer">D8040#150=
577</a>, <a href=3D"https://phabricator.kde.org/p/davidedmundson/" style=3D"
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel=3D"noreferrer">@davidedmundson</a> wro=
te</div>
<div style=3D"margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>If you do want to set this based o=
n the initial scene backend, skip using this method and call the static met=
hod on QQuickWindow instead.</p></div>
</blockquote>

<p>After reading through the docs I think the combination of both is fine: =
We can use the QtQuickSettings to have a base. But if we init a QPainter co=
mpositor we just enforce software through the QQuickWindow API. That should=
 (TM) work as well. Dynamic switching is then not required. AFAIK the only =
&quot;dangerous&quot; combination would be QPainter compositor and OpenGL c=
ompositing. The other 3 combinations should just work fine.</p></div></div>=
<br /><div><strong>REPOSITORY</strong><div><div>R108 KWin</div></div></div>=
<br /><div><strong>BRANCH</strong><div><div>master</div></div></div><br /><=
div><strong>REVISION DETAIL</strong><div><a href=3D"https://phabricator.kde=
.org/D8040" rel=3D"noreferrer">https://phabricator.kde.org/D8040</a></div><=
/div><br /><div><strong>To: </strong>davidedmundson, Plasma, graesslin<br /=
><strong>Cc: </strong>graesslin, plasma-devel, kwin, KWin, bwowk, ZrenBot, =
progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, seba=
s, apol, mart<br /></div>
--0b5e1cca9f084d158f80ee1020da6eda--