<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/100461/">http://git.reviewboard.kde.org/r/100461/</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;">All looks good to me.</pre>
<br />
<p>- David</p>
<br />
<p>On January 30th, 2011, 10:07 p.m., George Kiagiadakis 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 and David Edmundson.</div>
<div>By George Kiagiadakis.</div>
<p style="color: grey;"><i>Updated Jan. 30, 2011, 10:07 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;">shortlog:
George Kiagiadakis (5):
Fix the contactPresenceChanged signal to actually notify us about presence changes.
Let mainwindow know the correct chat window icon from the beggining of the session.
Fix the presence change messages.
Catch contact alias changes.
Do not show notifications about messages that have already been shown by the approver.
http://quickgit.kde.org/?p=clones/telepathy-chat-handler/gkiagia/telepathy-chat-handler.git&a=shortlog&h=refs/heads/fixes</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;">All changes have been tested, except the remote contact's alias change.</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>app/mainwindow.cpp <span style="color: grey">(98647c9cbf3bd3456c9fc79ab8acb048dad393e0)</span></li>
<li>lib/channelcontactlist.h <span style="color: grey">(f3fae069081febf3c5471bd8b74708f1ec72ec93)</span></li>
<li>lib/channelcontactlist.cpp <span style="color: grey">(499ae802d10990179a3f3c97d0f09639024df0b2)</span></li>
<li>lib/chatwindow.h <span style="color: grey">(434ece2946daeace4b93fafb14a8a67aa718e4c0)</span></li>
<li>lib/chatwindow.cpp <span style="color: grey">(92e7a5329f6a1ed00634aa28ce6dfe3590ace6a5)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100461/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>