Review Request: Refactor GroupsModel (from the Contact List) on top of PersonSetModel

David Edmundson kde at davidedmundson.co.uk
Fri May 4 12:00:03 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102156/#review13415
-----------------------------------------------------------


What's the status of these. 
Can I just discard these review requests?

- David Edmundson


On July 31, 2011, 6:31 p.m., George Goldberg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102156/
> -----------------------------------------------------------
> 
> (Updated July 31, 2011, 6:31 p.m.)
> 
> 
> Review request for Telepathy and Martin Klapetek.
> 
> 
> Description
> -------
> 
> 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 :)
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt b0cf53b6bcf6f2b6395ea24f79f50d6d5344d6fd 
>   src/KTelepathy/GroupedPersonSetModel PRE-CREATION 
>   src/ui/grouped-person-set-model.h PRE-CREATION 
>   src/ui/grouped-person-set-model.cpp PRE-CREATION 
>   src/ui/grouped-person-set-model_p.h PRE-CREATION 
>   src/ui/person-set-model.h 38860ae456a83d698ed19868fbabf4fed6555902 
>   src/ui/person-set-model.cpp 6d01c29f7e7173e2b8af99a709fc3a2137d3115d 
>   src/ui/person-set-model_p.h PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/102156/diff/
> 
> 
> Testing
> -------
> 
> PersonSetModel still passes modeltest after the refactoring, and the contactlist behaves as expected when adjusted to use this model instead of the GroupsModel directly.
> 
> 
> Thanks,
> 
> George Goldberg
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20120504/26f58d48/attachment.html>


More information about the KDE-Telepathy mailing list