'Review Request: Abort color correction initialization and disable it in case of errors'

[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kwin
Subject:    Review Request: Abort color correction initialization and disable it in case of errors
From:       "Casian Andrei" <skeletk13 () gmail ! com>
Date:       2012-12-16 8:50:12
Message-ID: 20121216085012.5183.68823 () vidsolbach ! de
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107754/
-----------------------------------------------------------

Review request for kwin, Thomas L=C3=BCbking and Martin Gr=C3=A4=C3=9Flin.

Description
-------

Abort color correction initialization and disable it in case of errors
    =

    Checks are now performed for GL errors and in case of errors everything
    is aborted. The error handling mechanism introduced for this purpose
    somewhat improves the color correction code.

Diffs
-----

  kwin/libkwineffects/kwinglcolorcorrection.h ec080c0 =

  kwin/libkwineffects/kwinglcolorcorrection.cpp b5127e6 =

  kwin/libkwineffects/kwinglcolorcorrection_p.h f54aa49 =

  kwin/scene_opengl.h e6142c0 =

  kwin/scene_opengl.cpp 03a31cf =

Diff: http://git.reviewboard.kde.org/r/107754/diff/

Testing
-------

Tested when a GL invalid operation error occurs - color correction is disab=
led and will not be enabled again because of the error. However, the check =
box in KCM remains checked - but this is ok I think.

Tested when no GL invalid operation is detected when doing color correction=
 stuff. Everything works fine from my point of view.

The curious thing is that the invalid operation occurs sometime before the =
color correction stuff, and it's probably not related, but I did not invest=
igate thus far.

Thanks,

Casian Andrei

[Attachment #5 (text/html)]

<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 \
solid;">  <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/107754/">http://git.reviewboard.kde.org/r/107754/</a>
  </td>
    </tr>
   </table>
   <br />

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" \
style="background-image: \
url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); \
background-position: left top; background-repeat: repeat-x; border: 1px black \
solid;">  <tr>
  <td>

<div>Review request for kwin, Thomas Lübking and Martin Gräßlin.</div>
<div>By Casian Andrei.</div>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
 <table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" \
style="border: 1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">Abort color correction initialization and disable it in case of errors  
    Checks are now performed for GL errors and in case of errors everything
    is aborted. The error handling mechanism introduced for this purpose
    somewhat improves the color correction code.</pre>
  </td>
 </tr>
</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: \
1px solid #b8b5a0">  <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: \
-moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: \
break-word;">Tested when a GL invalid operation error occurs - color correction is \
disabled and will not be enabled again because of the error. However, the check box \
in KCM remains checked - but this is ok I think.

Tested when no GL invalid operation is detected when doing color correction stuff. \
Everything works fine from my point of view.

The curious thing is that the invalid operation occurs sometime before the color \
correction stuff, and it&#39;s probably not related, but I did not investigate thus \
far.</pre>  </td>
 </tr>
</table>

<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>kwin/libkwineffects/kwinglcolorcorrection.h <span style="color: \
grey">(ec080c0)</span></li>

 <li>kwin/libkwineffects/kwinglcolorcorrection.cpp <span style="color: \
grey">(b5127e6)</span></li>

 <li>kwin/libkwineffects/kwinglcolorcorrection_p.h <span style="color: \
grey">(f54aa49)</span></li>

 <li>kwin/scene_opengl.h <span style="color: grey">(e6142c0)</span></li>

 <li>kwin/scene_opengl.cpp <span style="color: grey">(03a31cf)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/107754/diff/" style="margin-left: \
3em;">View Diff</a></p>

  </td>
 </tr>
</table>

  </div>
 </body>
</html>

_______________________________________________
kwin mailing list
kwin@kde.org
https://mail.kde.org/mailman/listinfo/kwin

[prev in list] [next in list] [prev in thread] [next in thread] 