<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/100429/">http://git.reviewboard.kde.org/r/100429/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 25th, 2011, 7:06 p.m., <b>George Kiagiadakis</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;">It's ok in overall, I won't really go deep into reviewing every detail as I expect the code to change anyway...</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;">I have fixed the issues and merged the result to master. You can close this request. Thanks for your work.</pre>
<br />
<p>- George</p>
<br />
<p>On January 25th, 2011, 4:37 p.m., Florian Reinhard wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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 Telepathy, George Kiagiadakis and David Edmundson.</div>
<div>By Florian Reinhard.</div>
<p style="color: grey;"><i>Updated Jan. 25, 2011, 4:37 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;">port to telepathy-qt4-0.5
port to recent qt-gstreamer
it's not meant to be finished, it compiles, runs but doesn't work for me
use one of these qt-gstreamer sources:
http://gstreamer.freedesktop.org/src/qt-gstreamer/
git://anongit.freedesktop.org/gstreamer/qt-gstreamer
</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;">none, doesn't work for me atm :/</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>CMakeLists.txt <span style="color: grey">(285ebb75a51fa931df40654169eada8cef886e19)</span></li>
<li>libtelepathy-kde-call/callchannelhandler.cpp <span style="color: grey">(8eb00bebb5dc90ef687894d32b85e63fba8c9a76)</span></li>
<li>libtelepathy-kde-call/configuration_test.cpp <span style="color: grey">(69d2e7f31ad96b6e1a35fc947fa8d67237f39dfe)</span></li>
<li>libtelepathy-kde-call/deviceelementfactory.cpp <span style="color: grey">(dbcd7a1a543cf1800ba1763e9e5770f9a120f3f8)</span></li>
<li>libtelepathy-kde-call/phononintegration.cpp <span style="color: grey">(884904afa15688787304ddf8f2c8f4b90375b5bf)</span></li>
<li>src/callhandler.h <span style="color: grey">(8d551ebb15488cce7755243f544b8c53afccd4be)</span></li>
<li>src/callhandler.cpp <span style="color: grey">(475856e1fbe07430c3eebd5bfa91b24f27458571)</span></li>
<li>src/dtmfhandler.cpp <span style="color: grey">(24739b63270946654897d896c6ee43a6b030601f)</span></li>
<li>src/main.cpp <span style="color: grey">(a3e590c30d53468d2281c8494b50c7bf3b9d539f)</span></li>
<li>test-contactlist/libkcallprivate/abstractclientapprover.h <span style="color: grey">(a89ec766232a509ef6a72c0c49ae47fd2003f367)</span></li>
<li>test-contactlist/libkcallprivate/abstractclientapprover.cpp <span style="color: grey">(3fa8ce5cad0b77f78dba5de69d417294cccb63f9)</span></li>
<li>test-contactlist/libkcallprivate/accountitem.h <span style="color: grey">(cd40a935f74b62a71bee97927cc83810efa6f794)</span></li>
<li>test-contactlist/libkcallprivate/accountitem.cpp <span style="color: grey">(ce1d4aa2c33a29edd6970d0209ec560097bd4dd8)</span></li>
<li>test-contactlist/libkcallprivate/accountmanager.h <span style="color: grey">(40cd91936d1e4bd73de201488cb1c24c3389486f)</span></li>
<li>test-contactlist/libkcallprivate/accountmanager.cpp <span style="color: grey">(222f6bcc327343a001529035f081bec4d287c1d7)</span></li>
<li>test-contactlist/libkcallprivate/contactitem.cpp <span style="color: grey">(3dbc1f2866c4af2d29065bda4b9ceab3da008aa4)</span></li>
<li>test-contactlist/libkcallprivate/pendingcontacts.cpp <span style="color: grey">(4cabbad40c3528174683ee26a60afe5299750d7e)</span></li>
<li>test-contactlist/systrayicon.cpp <span style="color: grey">(e41e7cf0ad6b196dc455db4cd5b1d0de8f76be44)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100429/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>