<font size="4"><font face="courier new,monospace">Thanks, someone please also carefully review <a href="https://git.reviewboard.kde.org/r/110162/">https://git.reviewboard.kde.org/r/110162/</a><br><br></font></font><br><div class="gmail_quote">

On 24 April 2013 23:26, Smit Patel <span dir="ltr"><<a href="mailto:smitpatel24@gmail.com" target="_blank">smitpatel24@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





 <div>
  <div style="font-family:Verdana,Arial,Helvetica,Sans-Serif"><div class="im">
   <table style="border:1px #c9c399 solid" bgcolor="#f9f3c9" cellpadding="8" width="100%">
    <tbody><tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="http://git.reviewboard.kde.org/r/110163/" target="_blank">http://git.reviewboard.kde.org/r/110163/</a>
     </td>
    </tr>
   </tbody></table>
   <br>



 </div><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>- Smit</p><div class="im">


<br>
<p>On April 24th, 2013, 9:18 p.m. UTC, Jarosław Staniek wrote:</p>








</div><table style="background-image:url('');background-repeat:repeat-x;border:1px black solid" bgcolor="#fefadf" cellpadding="8" cellspacing="0" width="100%">
 <tbody><tr>
  <td><div class="im">

<div>Review request for Calligra, Oleg Kukharchuk, Adam Pigg, Smit Patel, Dimitrios Tanis, Tomasz Olszak, and Radosław Wicik.</div>
<div>By Jarosław Staniek.</div>


</div><p style="color:grey"><i>Updated April 24, 2013, 9:18 p.m.</i></p><div class="im">






<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Description </h1>
 <table style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
 <tbody><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">Use KexiDB's TableViewData in the biblio table model. As a result the model is properly editable. 

Moreover, unlike the original solution, thanks to buffering in TableViewData all operations are no longer expensive:
- row count is O(1),
- data altering is O(1),
- data retrieving (individual cells) is O(1)

Filtering works too. The code is also relatively simple.

This patch depends on <a href="https://git.reviewboard.kde.org/r/110162" target="_blank">https://git.reviewboard.kde.org/r/110162</a> (KexiTableViewColumn and KexiTableViewData from calligradb).</pre>
  </td>
 </tr>
</tbody></table>


<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Testing </h1>
<table style="border:1px solid #b8b5a0" bgcolor="#ffffff" cellpadding="10" cellspacing="0" width="100%">
 <tbody><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">Features of the Bibliography Database window work properly.

Not sure about memory leaks, e.g. two cases of Cursor lifetime: m_conn->deleteCursor()</pre>
  </td>
 </tr>
</tbody></table>




<h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Diffs </h1>
<ul style="margin-left:3em;padding-left:0">

 <li>libs/kotext/BibliographyTableModel.h <span style="color:grey">(PRE-CREATION)</span></li>

 <li>libs/kotext/BibliographyTableModel.cpp <span style="color:grey">(PRE-CREATION)</span></li>

</ul>

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







  </div></td>
 </tr>
</tbody></table>








  </div>
 </div>


</blockquote></div><br><br clear="all"><br>-- <br><div dir="ltr">regards / pozdrawiam, Jaroslaw Staniek<br> Kexi & Calligra & KDE | <a href="http://calligra.org/kexi" target="_blank">http://calligra.org/kexi</a> | <a href="http://kde.org" target="_blank">http://kde.org</a><br>

 Qt Certified Specialist | <a href="http://qt-project.org" target="_blank">http://qt-project.org</a><br> <a href="http://www.linkedin.com/in/jstaniek" target="_blank">http://www.linkedin.com/in/jstaniek</a></div>