<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/106467/">http://git.reviewboard.kde.org/r/106467/</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;">I suppose 100 is fine in most situations except for long literals which are rarely queried anyway.

This looks good. Did tests show that the overall performance improves on the getData call?</pre>

 <br />

<p>- Sebastian</p>

<br />

<p>On September 17th, 2012, 9:37 a.m., Vishesh Handa wrote:</p>

<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/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 Nepomuk, Soprano and Sebastian Trueg.</div>

<div>By Vishesh Handa.</div>

<p style="color: grey;"><i>Updated Sept. 17, 2012, 9:37 a.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;">    Only use 1 SQLFetchData command in most of the cases.

    Callgrind stats show that 67.5% of the time in this function is spent in

    the first SQLFetchData, and an additional 27% in the second SQLGetData.

    We can avoid some of this extra cost, by only calling the function

    once.

I can change the size of the default buffer if required.</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>backends/virtuoso/odbcqueryresult.cpp <span style="color: grey">(a4f2387)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>