<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="https://git.reviewboard.kde.org/r/115618/">https://git.reviewboard.kde.org/r/115618/</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;">This review has been submitted with commit 645d85f52d69ffc22529eb0853ebf061e5f4e4fd by Alexandr Akulich to branch master.</pre>
<br />
<p>- Commit Hook</p>
<br />
<p>On February 10th, 2014, 10:19 a.m. UTC, Alexandr Akulich wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 Telepathy.</div>
<div>By Alexandr Akulich.</div>
<p style="color: grey;"><i>Updated Feb. 10, 2014, 10:19 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
ktp-kded-module
</div>
<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;">Manually constructed factories and managers replaced by KTp ones.
Such change let us to cleanup code and let freely use KTp methods without dbus traffic duplication (which is occurs in last commit).
As according things:
Reworked includes.
Fixed a lot of coding style issues.
ContactNotify: Removed "using namespace KTp" from header.</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;">Compiled successfully. Seems to work, but not tested a lot.</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>error-handler.cpp <span style="color: grey">(2b84784)</span></li>
<li>screensaveraway.cpp <span style="color: grey">(6c47cfb)</span></li>
<li>telepathy-kded-module-plugin.cpp <span style="color: grey">(43cb383)</span></li>
<li>telepathy-module.h <span style="color: grey">(ec08c6e)</span></li>
<li>telepathy-module.cpp <span style="color: grey">(bf154c6)</span></li>
<li>telepathy-mpris.cpp <span style="color: grey">(1c7b98c)</span></li>
<li>autoaway.h <span style="color: grey">(ac42718)</span></li>
<li>autoaway.cpp <span style="color: grey">(79f62f0)</span></li>
<li>autoconnect.h <span style="color: grey">(aeb4b54)</span></li>
<li>autoconnect.cpp <span style="color: grey">(b779426)</span></li>
<li>contact-request-handler.h <span style="color: grey">(69d2a1e)</span></li>
<li>contact-request-handler.cpp <span style="color: grey">(1b13194)</span></li>
<li>contactnotify.h <span style="color: grey">(4dc71e4)</span></li>
<li>contactnotify.cpp <span style="color: grey">(9f001f4)</span></li>
<li>error-handler.h <span style="color: grey">(dd62822)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/115618/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>