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

[George Goldberg]

-----------------------------------------------------------
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.


Changes
-------

Argh sorry, the last diff was based off the wrong commit so had a load of irrelevant stuff in it.


Summary
-------

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 (updated)
-----

  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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110731/f3cc91b5/attachment.htm