----------------------------------------------------------- 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
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
