yes.  you can discard anything that I've posted and is no longer relevant (last time I was around Martin was maintaining this repo, so he can say if the review requests are now irrelevant).<br><br>George<br><br><div class="gmail_quote">
On 4 May 2012 13:00, David Edmundson <span dir="ltr"><<a href="mailto:kde@davidedmundson.co.uk" target="_blank">kde@davidedmundson.co.uk</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/102156/" target="_blank">http://git.reviewboard.kde.org/r/102156/</a>
     </td>
    </tr>
   </tbody></table>
   <br>





 </div><pre style="white-space:pre-wrap;white-space:-moz-pre-wrap;white-space:-pre-wrap;white-space:-o-pre-wrap;word-wrap:break-word">What's the status of these. 
Can I just discard these review requests?</pre>
 <br>







<p>- David</p><div class="im">


<br>
<p>On July 31st, 2011, 6:31 p.m., George Goldberg 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 Telepathy and Martin Klapetek.</div>
<div>By George Goldberg.</div>


<p style="color:grey"><i>Updated July 31, 2011, 6:31 p.m.</i></p>






</div><h1 style="color:#575012;font-size:10pt;margin-top:1.5em">Description </h1><div class="im">
 <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">This patch to telepathy-kde takes the GroupsModel from the contactlist and refactors it into a subclass of PersonSetModel.

Martin: I'm not entirely convinced I understood your algorithm in GroupsModel for when the groups of a Person change, so I might have ended up making it considerably more inefficient in this review request. Please shout if you see anything that should be changed.

While writing this summary, I've just realised that I forgot to implement the [Online]UsersCount in this, so assume that I'm aware of that and will add it in a later version of the patch :)</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">PersonSetModel still passes modeltest after the refactoring, and the contactlist behaves as expected when adjusted to use this model instead of the GroupsModel directly.</pre>

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




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

 <li>src/CMakeLists.txt <span style="color:grey">(b0cf53b6bcf6f2b6395ea24f79f50d6d5344d6fd)</span></li>

 <li>src/KTelepathy/GroupedPersonSetModel <span style="color:grey">(PRE-CREATION)</span></li>

 <li>src/ui/grouped-person-set-model.h <span style="color:grey">(PRE-CREATION)</span></li>

 <li>src/ui/grouped-person-set-model.cpp <span style="color:grey">(PRE-CREATION)</span></li>

 <li>src/ui/grouped-person-set-model_p.h <span style="color:grey">(PRE-CREATION)</span></li>

 <li>src/ui/person-set-model.h <span style="color:grey">(38860ae456a83d698ed19868fbabf4fed6555902)</span></li>

 <li>src/ui/person-set-model.cpp <span style="color:grey">(6d01c29f7e7173e2b8af99a709fc3a2137d3115d)</span></li>

 <li>src/ui/person-set-model_p.h <span style="color:grey">(PRE-CREATION)</span></li>

</ul>

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




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








  </div>
 </div>


</blockquote></div><br>