> On Sept. 9, 2013, 12:15 p.m., David Edmundson wrote: > > KTp/Models/contacts-list-model.cpp, line 210 > > <http://git.reviewboard.kde.org/r/112599/diff/1/?file=188403#file188403line210> > > > > I think there may be potential to emit this when only the first contact > > manager out of several is loaded. > > > >
I'm not sure I follow - d->initialized is set to true immediately after this, so it will emitted only once...? - Dan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112599/#review39629 ----------------------------------------------------------- On Sept. 9, 2013, 12:03 p.m., Dan Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112599/ > ----------------------------------------------------------- > > (Updated Sept. 9, 2013, 12:03 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > Lorem Ipsum. > > > Diffs > ----- > > KTp/Models/contacts-list-model.h 876a485 > KTp/Models/contacts-list-model.cpp 9a1b532 > KTp/Models/contacts-model.h 345378b > KTp/Models/contacts-model.cpp 8a1b0d6 > > Diff: http://git.reviewboard.kde.org/r/112599/diff/ > > > Testing > ------- > > > Thanks, > > Dan Vrátil > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
