Review Request 110227: [ktp-contact-list] Now playing shouldn't be disabled on pause

[Ian Monroe]

> On April 28, 2013, 9:31 p.m., David Edmundson wrote:
> > global-presence-chooser.cpp, line 371
> > <http://git.reviewboard.kde.org/r/110227/diff/1/?file=141287#file141287line371>
> >
> >     Are you sure this >= is the right way round?
> >     
> >     I would have thought this should be "if we select a valid presence online/away/etc" which is if (index <= lastPresenceIndex) {
> >     
> >     }
> >     
> >     or am I getting confused?
> >     
> >     
> >     nitpick:
> >     
> >     coding style
> >     if () {
> >     
> >     }
> >

Good point. :)


- Ian


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


On April 28, 2013, 12:55 a.m., Ian Monroe wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110227/
> -----------------------------------------------------------
> 
> (Updated April 28, 2013, 12:55 a.m.)
> 
> 
> Review request for Telepathy and Martin Klapetek.
> 
> 
> Description
> -------
> 
> Now playing shouldn't be disabled on pause
> 
> Fix requires corresponding commit in ktp-kded-module https://git.reviewboard.kde.org/r/110228/
> Needed to separate things done for every presence change
> from those actions only to be done on user-initiated presence changes
> 
> 
> Diffs
> -----
> 
>   global-presence-chooser.h 678681c92afcbfcc85245b809bd29e84477eacfa 
>   global-presence-chooser.cpp f5d59399a904195121ebac7266bef4176ac82bfb 
> 
> Diff: http://git.reviewboard.kde.org/r/110227/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ian Monroe
> 
>

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