<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/107533/">http://git.reviewboard.kde.org/r/107533/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit 10b7f75003ba77fac9c01c7b1aaaf03a03430e7d by Vishesh Handa to branch master.</pre>

 <br />

<p>- Commit</p>

<br />

<p>On November 30th, 2012, 1:54 p.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 and Sebastian Trueg.</div>

<div>By Vishesh Handa.</div>

<p style="color: grey;"><i>Updated Nov. 30, 2012, 1:54 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;">    NepomukMainModel: Never reset the localSocketModel

    We now never create a new LocalSocketModel on reconnecting. Instead we

    continue using the old one. This way we do not leak memory, as we could

    not delete the old model (open iterators).

    This is only possibly cause of the recent changes in Soprano which never

    delete the inner ClientConnection which is shared between the model and

    the LocalSocketClient. Before that commit the ClientConnection was

    deleted which would result in the old model becoming invalid.

This depends on https://git.reviewboard.kde.org/r/107531/</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>libnepomukcore/resource/nepomukmainmodel.cpp <span style="color: grey">(76bd6a3)</span></li>

</ul>

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

  </td>

 </tr>

</table>

  </div>

 </body>

</html>