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



 <p>Ship it!</p>



 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Very good.</pre>
 <br />







<p>- Sebastian</p>


<br />
<p>On June 26th, 2011, 9:27 p.m., Artem Serebriyskiy 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 Nepomuk.</div>
<div>By Artem Serebriyskiy.</div>


<p style="color: grey;"><i>Updated June 26, 2011, 9:27 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;">Add methods userName() and description() to Nepomuk Services - in object /servicecontrol
These methods are necessary for Nepomuk Plasma dataengine, usefull for Nepomuk System Tray, and, generally speaking, usefull for every application that don&#39;t want to mess up with .desktop files. Because before this patch there is no way to query name and description of running Nepomuk Service except to find corresponding .desktop file and parse it.</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;">Compling. Launching. 
The only error discovered - qdbusviewer refused to call these methods. But calling using python-dbus works as expected, so I think it is qdbusviewer problem.</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>nepomuk/interfaces/org.kde.nepomuk.ServiceControl.xml <span style="color: grey">(dd91c8f)</span></li>

 <li>nepomuk/servicestub/servicecontrol.h <span style="color: grey">(bd77324)</span></li>

 <li>nepomuk/servicestub/servicecontrol.cpp <span style="color: grey">(2ad4f9f)</span></li>

</ul>

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




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








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