<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://git.reviewboard.kde.org/r/122160/">https://git.reviewboard.kde.org/r/122160/</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;">Ship It!</pre>
 <br />









<p>- Olivier Jean de Gaalon</p>


<br />
<p>On April 21st, 2015, 6:59 p.m. UTC, Milian Wolff wrote:</p>








<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
 <tr>
  <td>

<div>Review request for KDevelop.</div>
<div>By Milian Wolff.</div>


<p style="color: grey;"><i>Updated April 21, 2015, 6:59 p.m.</i></p>









<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt;">Repository: </b>
kdev-clang
</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;">In such cases, it's worthwile to use the source buddy as best-guess as
it allows us to get rid of the parse run for the "header TU" which
gets thrown away later on anyways.

Performance wise, this patch improves the index run on a 48 file
project using STL: It goes from ~17s down to ~14s, just b/c we safe
a few needless clang runs on "header TUs".

REVIEW: 122160</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">(a28f0c1925bf20bfbde22e4dc0fd702960f4e3cd)</span></li>

 <li>clangparsejob.cpp <span style="color: grey">(d4c24d314598a17e043c4427d1781f6d7551b6a5)</span></li>

 <li>clangsupport.cpp <span style="color: grey">(f2c1a1ac7a0a366e698ae836954b397a95015ecc)</span></li>

 <li>documentfinderhelpers.h <span style="color: grey">()</span></li>

 <li>documentfinderhelpers.cpp <span style="color: grey">()</span></li>

 <li>duchain/CMakeLists.txt <span style="color: grey">(8ed282b652361c45c668fa948917e075fed4f277)</span></li>

 <li>duchain/clangindex.cpp <span style="color: grey">(25c7855e7bd67aa2a3be28f1ea803dd025e6a1ba)</span></li>

</ul>

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






  </td>
 </tr>
</table>







  </div>
 </body>
</html>