> On Feb. 12, 2015, 6:46 p.m., Martin Klapetek wrote: > > kpeople/datasourceplugin/im-persons-data-source.cpp, lines 182-183 > > <https://git.reviewboard.kde.org/r/122543/diff/1/?file=348508#file348508line182> > > > > Given that "found" is nowhere else used, wouldn't it be sufficient to > > just do > > > > ```if (it != m_contactVCards.constEnd()) {``` ?
Oh yes, I wanted to re-use it instead of the constFind below. I'll do the change directly when shipping it. - Aleix ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/122543/#review75933 ----------------------------------------------------------- On Feb. 12, 2015, 6:20 p.m., Aleix Pol Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/122543/ > ----------------------------------------------------------- > > (Updated Feb. 12, 2015, 6:20 p.m.) > > > Review request for Telepathy. > > > Repository: ktp-common-internals > > > Description > ------- > > Re-use the instances instead of creating them every time the cache is loaded. > > > Diffs > ----- > > kpeople/datasourceplugin/im-persons-data-source.cpp 77cb67f > > Diff: https://git.reviewboard.kde.org/r/122543/diff/ > > > Testing > ------- > > KTp Contact List plasmoid: > 1. Open the Contact List > 2. Go online > 3. Go offline > 4. Go online > > It doesn't assert anymore. > > > Thanks, > > Aleix Pol Gonzalez > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
