<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/108808/">http://git.reviewboard.kde.org/r/108808/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">If the property does not exist or is of wrong type or contains junk, the reasonable return value would be 1.0, not 0.0 - agreed?

I mean, if the client withdraws the property and it would get read agin, the client would expect to be opaque as if it didn't support the property, yesno?

If the call gets us a BadWindow (likely the case except for below) it seems pretty much ok to return the cached value, but i don't think that's the case for other errors.

reg. colibri:

i don't see why Qt should withdraw the property before deleting the window.

Any chance that colibri attempts to set the opacity out of bounds (-1) so that Qt would fall back to withdrawing it or similar?</pre>

 <br />

<p>- Thomas</p>

<br />

<p>On February 6th, 2013, 2:38 p.m. UTC, Martin Gräßlin wrote:</p>

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

 <tr>

  <td>

<div>Review request for kdelibs and kwin.</div>

<div>By Martin Gräßlin.</div>

<p style="color: grey;"><i>Updated Feb. 6, 2013, 2:38 p.m.</i></p>

<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;">Do not reset opacity if call for GetWindowProperty fails

The opacity got unconditionally reset to the default value of fully

opaque. In case the property has a value of 0 and the call to get the

window property fails, the window would be shown again. This issue was

spotted with colibri which faded out it's window and got a flash to

fully opaque again.

I'm sorry that it's difficult to read this patch properly due to the

existing mixing of tabs and whitespaces. The patch changes two things:

* only updates the value if all checks succeed

* initialise the data_ret variable to not cause a crash on free

BUG: 314427

FIXED-IN: 4.10.1</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>kdeui/windowmanagement/netwm.cpp <span style="color: grey">(cf28339f90dff1d17ed17842c7c11d8a9718a459)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>