<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/108791/">http://git.reviewboard.kde.org/r/108791/</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;">The patch looks good, but it causes the same issues of the KWallet stuff (see Bug 300062)
We should probably discuss the problem upstream on the telepathy mailing list.
Anyway it's a ship it from me, just add a comment to that bug and open a new one so that we don't remember to fix both these problems.</pre>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/108791/diff/1/?file=112179#file112179line71" style="color: black; font-weight: bold; text-decoration: underline;">main.cpp</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">int main(int argc, char *argv[])</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#f0f0f0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2">71</font></th>
<td bgcolor="#ffffff" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="kt">int</span> <span class="n">handlers</span> <span class="o">=</span> <span class="mi">1</span><span class="p">;</span></pre></td>
<th bgcolor="#f0f0f0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">71</font></th>
<td bgcolor="#ffffff" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="kt">int</span> <span class="n">handlers</span> <span class="o">=</span> <span class="mi">1</span><span class="p">;</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">You should change this to 2.
Yes it's probably a stupid method to do it, feel free change it.</pre>
</div>
<br />
<p>- Daniele E.</p>
<br />
<p>On February 4th, 2013, 11:17 p.m. UTC, Dan Vrátil 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 Telepathy.</div>
<div>By Dan Vrátil.</div>
<p style="color: grey;"><i>Updated Feb. 4, 2013, 11:17 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;">Uses QCA2 to verify the certificate chain. CA certs are read from KSslCertificateManager. Exception rules are stored in KSslCertificateManager too. This means that we now share certificates and exceptions with the rest of KDE \o/. However in order to use KSslCertificateManager and KSslInfoDialog I had to import several private headers from kdelibs.
The dialogs are copied from KIO::TcpSlaveBase, so they look exactly like kio_http.
If you set a "remember forever" rule during testing and you want to remove it, remove it from ~/.kde/share/config/ksslcertificatemanager and restart KDED.
I had to remove -DQT_NO_CAST_FROM_ASCII and -DQT_NO_KEYWORDS because QCA2 and kdelibs headers wouldn't compile otherwise.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=265315">265315</a>
</div>
<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">(417554a)</span></li>
<li>kssl/kssl.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kssl/ksslcertificatemanager.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kssl/ksslinfodialog.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>kssl/ksslsettings.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>main.cpp <span style="color: grey">(3350658)</span></li>
<li>tls-cert-verifier-op.h <span style="color: grey">(121cf41)</span></li>
<li>tls-cert-verifier-op.cpp <span style="color: grey">(c94999e)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/108791/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>