----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/107629/#review24548 -----------------------------------------------------------
KTp/contact.h <http://git.reviewboard.kde.org/r/107629/#comment18837> What about a class KTp::ContactCapabilities : public Tp:ContactCapabilities {...}; and a method KTp::ContactCapabilities capabilities() const; instead? This would make it more coherent with the Tp::Contact class and with the presence method returning a KTp::Presence instead of a Tp::Presence KTp/contact.cpp <http://git.reviewboard.kde.org/r/107629/#comment18836> You could simplify and avoid a few method calls by doing Tp::ConnectionPtr connection = manager()->connection(); if (!connection) { return false; } [...] (same for the other capabilities) - Daniele Elmo Domenichelli On Dec. 19, 2012, 4:19 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/107629/ > ----------------------------------------------------------- > > (Updated Dec. 19, 2012, 4:19 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > - Add a new list model of Contacts > - Add a ContactFactory and Contact class allowing the subclassing of > Tp::Contact > - Move capabilities hack outside of models so it can be used by future > KTp::Contact > > > Diffs > ----- > > KTp/CMakeLists.txt 64a12b22b5125d1ba44a3e82af20c8d3d7dccd73 > KTp/Models/CMakeLists.txt 404eee32735489a80c6e93e5f90b1db549536386 > KTp/Models/accounts-filter-model.cpp > dd911ac294dd8aeaa7e2e1d6f8bc11665121d1ec > KTp/Models/capabilities-hack-private.h > aa3478faadceb55450cc9e6725c407095f8e3f94 > KTp/Models/capabilities-hack-private.cpp > 46ba34e38bc4f4743beb032bf01992be952aaeef > KTp/Models/contacts-list-model.h PRE-CREATION > KTp/Models/contacts-list-model.cpp PRE-CREATION > KTp/capabilities-hack-private.h PRE-CREATION > KTp/capabilities-hack-private.cpp PRE-CREATION > KTp/contact-factory.h PRE-CREATION > KTp/contact-factory.cpp PRE-CREATION > KTp/contact.h PRE-CREATION > KTp/contact.cpp PRE-CREATION > > Diff: http://git.reviewboard.kde.org/r/107629/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
