<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/103593/">http://git.reviewboard.kde.org/r/103593/</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;">While it definitely makes sense to make the KDE logout work, isn't the root of the problem that KNotification doesn't call the slot if no sound is available? Other applications would be affected by this very issue, if they are triggering sounds the same way. In my opinion it'd make more sense to fix KNotification to issue a ignore() slot call when no sound is available.

It wouldn't hurt, of course, to fix it in both places. ;)</pre>

 <br />

<p>- Dirk</p>

<br />

<p>On December 31st, 2011, 3:16 a.m., Peter OGorman wrote:</p>

<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 KDE Runtime.</div>

<div>By Peter OGorman.</div>

<p style="color: grey;"><i>Updated Dec. 31, 2011, 3:16 a.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;">https://bugs.kde.org/show_bug.cgi?id=228005

if sound is not working for some reason (e.g. no phonon

backends are installed) the closed() signal never happens

and logoutSoundFinished() never gets called. This hack

makes sure that logout still occurs. Of course, on systems

with working sound there's a race.

</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>ksmserver/shutdown.cpp <span style="color: grey">(7fd1e11)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>