Review Request: Code tidy of contact-request-handler

[Commit Hook]

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


This review has been submitted with commit 3b0fb7c2a8e7ae57819915aafa47aa2321cd732c by David Edmundson to branch master.

- Commit Hook


On Sept. 1, 2012, 6:20 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106298/
> -----------------------------------------------------------
> 
> (Updated Sept. 1, 2012, 6:20 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Remove status notifier item when all contacts have been processed
> 
> Tidied some of the surrounding code.
> Removed completely pointless member variable.
> Added guards round all usagse of QWeakPointer::data()
> 
> BUG: 302471
> 
> Pass Contact property to onFinalizeSubscriptionFinished from onAuthorizePresencePublicationFinished
> 
> BUG: 305706
> 
> 
> Update strings to be more user friendly
> 
> 
> Diffs
> -----
> 
>   contact-request-handler.h 45bbd8d29803476d8dee9a753faada362c02e869 
>   contact-request-handler.cpp 542e65b10bcc33f5abc2643986d2093c46ea87af 
> 
> Diff: http://git.reviewboard.kde.org/r/106298/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

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