> 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 > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
