<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/101453/">http://git.reviewboard.kde.org/r/101453/</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;">Sry, but the patch doesn&#39;t make sense to me; I don&#39;t like the timeout approach and secondly, the check for optical drive is there on purpose, they are handled separately elsewhere. This special case is because of bug https://bugs.kde.org/show_bug.cgi?id=267398</pre>
 <br />







<p>- Lukáš</p>


<br />
<p>On May 27th, 2011, 7:50 p.m., Jacopo De Simoi 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 Solid, Lukáš Tinkl and Ozan Çağlayan.</div>
<div>By Jacopo De Simoi.</div>


<p style="color: grey;"><i>Updated May 27, 2011, 7:50 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;">Call DeviceEject only if the drive actually requires to be ejected, 
moreover call the special eject routine if the drive IS an optical Disc, not if it is NOT an optical disc…</pre>
  </td>
 </tr>
</table>




<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="http://bugs.kde.org/show_bug.cgi?id=270490">270490</a>


</div>


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

 <li>solid/solid/backends/udisks/udisksstorageaccess.h <span style="color: grey">(6bf729cce4e5c5ec79fa82dd56fcd825d9f79515)</span></li>

 <li>solid/solid/backends/udisks/udisksstorageaccess.cpp <span style="color: grey">(4cb0f7ceeee2f7a342346fd7507df6788a7bf82c)</span></li>

</ul>

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




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








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