<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/114106/">http://git.reviewboard.kde.org/r/114106/</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;">Thanks. 

As for your other comment:

 >There is may be small performance issue we query both contact and self capabilities, while if any of two ret false, another query is needless.

It's very important to analyse where slowdowns are before trying to optimise things. Otherwise it's not going to be the best use of your time</pre>

 <br />

<p>- David Edmundson</p>

<br />

<p>On November 25th, 2013, 1:32 p.m. UTC, Alexandr Akulich 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 Alexandr Akulich.</div>

<p style="color: grey;"><i>Updated Nov. 25, 2013, 1:32 p.m.</i></p>

<div style="margin-top: 1.5em;">

 <b style="color: #575012; font-size: 10pt;">Repository: </b>

ktp-common-internals

</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;">KTp/Contact: Get rid few condition double-checks.

There is may be small performance issue: we query both contact and self capabilities, while if any of two ret false, another query is needless.

(I talk about code like such one:)

    bool contactCanHandleFiles = capabilities().fileTransfers();

    bool selfCanHandleFiles = manager()->connection()->selfContact()->capabilities().fileTransfers();

    return contactCanHandleFiles && selfCanHandleFiles;

I didn't change it and hope for smart compiler.</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;">Tested in different states. Capabilities of contacts are same as before this change. There is no crashes or so.</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>KTp/contact.cpp <span style="color: grey">(64dd49b)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>