Review Request 108846: when offline, hide add contact SNI

[David Edmundson]

> On Feb. 8, 2013, 12:29 p.m., Dan Vrátil wrote:
> > contact-request-handler.cpp, line 211
> > <http://git.reviewboard.kde.org/r/108846/diff/1/?file=112992#file112992line211>
> >
> >     You should always use Tp::ContactPtr.

Ref-counting for the duration of a method doesn't actually add anything, it can't be de-ref'd during.
However for consistency of having a rule I'll push that now.


- David


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


On Feb. 8, 2013, 9:50 a.m., Stefan Eggers wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108846/
> -----------------------------------------------------------
> 
> (Updated Feb. 8, 2013, 9:50 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> when offline, hide add contact SNI
> 
> 
> This addresses bug 294478.
>     http://bugs.kde.org/show_bug.cgi?id=294478
> 
> 
> Diffs
> -----
> 
>   contact-request-handler.h 43456ee8f0a354ed075335d80a125da51713a0e9 
>   contact-request-handler.cpp f40f397e86393716cebf128630b2c4c43691930e 
>   telepathy-module.cpp 7ab13b26f8d52648c2c33c0fde0ff1648181c8c4 
> 
> Diff: http://git.reviewboard.kde.org/r/108846/diff/
> 
> 
> Testing
> -------
> 
> - add two accounts A and B in a local ejabber server
> - account A adds account B to contact list
> - approval request shows up in account B when B comes online
> - stop ejabber server so that B is offline
> - with this patch the approval request does not show anymore while B stays offline; it comes back when B is online again
> 
> 
> Thanks,
> 
> Stefan Eggers
> 
>

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