<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/111707/">http://git.reviewboard.kde.org/r/111707/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On July 26th, 2013, 5:28 p.m. UTC, <b>Albert Astals Cid</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">int lidAction() const; looks a bit weird as slot signature, but i guess the fact that is a slot is to trick qt4_add_dbus_adaptor?</pre>
 </blockquote>







</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">To make Qt export it on the bus.</pre>
<br />










<p>- Àlex</p>


<br />
<p>On July 26th, 2013, 2:58 a.m. UTC, Àlex Fiestas 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 Release Team, Solid, Kai Uwe Broulik, Dario Freddi, and Oliver Henshaw.</div>
<div>By Àlex Fiestas.</div>


<p style="color: grey;"><i>Updated July 26, 2013, 2:58 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;">We need to know this value in kscreen so we can act in consequence. We discussed this in the sprint already and everybody was ok.

Release team, can we push this to 4.11? it only exports a value on the bus using a established and standard way of doing it (we use this in 4 more places, exact code).</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>powerdevil/daemon/CMakeLists.txt <span style="color: grey">(d0274eb)</span></li>

 <li>powerdevil/daemon/actions/bundled/handlebuttonevents.h <span style="color: grey">(a16feba)</span></li>

 <li>powerdevil/daemon/actions/bundled/handlebuttonevents.cpp <span style="color: grey">(78e1d27)</span></li>

 <li>powerdevil/daemon/actions/bundled/powerdevilhandlebuttoneventsaction.desktop <span style="color: grey">(82f3e1b)</span></li>

</ul>

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







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








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