----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/114621/#review46058 -----------------------------------------------------------
KTp/persistent-contact.cpp <https://git.reviewboard.kde.org/r/114621/#comment32860> I found the const, but forgot to remove my fixme. Oops. Done now. - David Edmundson On Dec. 22, 2013, 6:49 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/114621/ > ----------------------------------------------------------- > > (Updated Dec. 22, 2013, 6:49 p.m.) > > > Review request for Telepathy. > > > Repository: ktp-common-internals > > > Description > ------- > > Create an account directly in PersistentContact > > This saves having to create an AccountManager to load contact info which in > turn saves having to load every account from DBus in order to get one > account's info. > > > Will make the API neater too as we will no longer have to wait and load an > AccountManager first. > > > Diffs > ----- > > KTp/persistent-contact.h f5367a3 > KTp/persistent-contact.cpp 919fad1 > > Diff: https://git.reviewboard.kde.org/r/114621/diff/ > > > Testing > ------- > > Using (my unpushed) KTp KPeople::ContactMonitor the KPeople personwidget > example loads presence _instantly_. Bustle confirms it is using a lot less > traffic, without libpurple spamming constantly if I am not accessing a > libpurple account. > > I also expect this to be the preferred way to get the ContactPtr in KTp's > kpeople UI/actions plugins. > > Theoretically this will also make the plasma quick chat applet laod faster - > but will require some changes before that actually happens. > > > Thanks, > > David Edmundson > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
