----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/106700/#review19799 -----------------------------------------------------------
Ship it! Nice trick with the loop! Just use Q_FOREACH instead of foreach and ship it! - Dan Vratil On Oct. 2, 2012, 9:45 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/106700/ > ----------------------------------------------------------- > > (Updated Oct. 2, 2012, 9:45 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > Don't use an entire model to loop through contacts. > > Should be faster, and simpler. > > > Diffs > ----- > > src/contactrunner.h 55e7e9de017dca23fa1d6551d4ce99997d05cad3 > src/contactrunner.cpp 48e01bf10b902932631b6e987c191d4d868e7d82 > > Diff: http://git.reviewboard.kde.org/r/106700/diff/ > > > Testing > ------- > > > Thanks, > > David Edmundson > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
