Review Request: New ContactsListModel

Daniele Elmo Domenichelli daniele.domenichelli at gmail.com
Thu Jan 3 11:59:52 UTC 2013


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

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


More information about the KDE-Telepathy mailing list