> On June 25, 2015, 1:42 p.m., Sergio Luis Martins wrote: > > src/service/service.cpp, line 34 > > <https://git.reviewboard.kde.org/r/124176/diff/2/?file=381373#file381373line34> > > > > leaking d_ptr > > Martin Klapetek wrote: > Well it's a standalone process and this is its only class, so when that > process dies this is deleted by kernel and has nowhere to leak...no?
alright - Sergio Luis ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124176/#review81743 ----------------------------------------------------------- On June 25, 2015, 1:33 p.m., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/124176/ > ----------------------------------------------------------- > > (Updated June 25, 2015, 1:33 p.m.) > > > Review request for KDEPIM and Telepathy. > > > Repository: kpeople > > > Description > ------- > > This is a dbus service that runs in the background and returns contact ids > for queried contact details, for example asking for a phone number would > return a contact id from which a PersonData class can be constructed, thus > making all contact data available for display. > > This is very useful when eg. receiving a phone call, a tp approver starts up > showing the incoming call (this needs to be instant) and once it's shown up, > it should show a contact name for the number as soon as possible. However > loading all the datasources and populating the model can be slow and speed is > critical in this case. Therefore it's a dbus service which responds very fast > and there's no need to initialize anything as it can be loaded in the > background after system starts and it keeps itself always up to date. > > > Diffs > ----- > > autotests/CMakeLists.txt 233e7d9 > autotests/servicetest.h PRE-CREATION > autotests/servicetest.cpp PRE-CREATION > src/CMakeLists.txt 59bc915 > src/backends/abstractcontact.h ce22cbc > src/backends/abstractcontact.cpp f01236b > src/global.h e1d07ce > src/global.cpp b3595ca > src/service/CMakeLists.txt PRE-CREATION > src/service/main.cpp PRE-CREATION > src/service/org.kde.KPeople.service.in PRE-CREATION > src/service/service.h PRE-CREATION > src/service/service.cpp PRE-CREATION > > Diff: https://git.reviewboard.kde.org/r/124176/diff/ > > > Testing > ------- > > Unit test included; also tested with qdbus. > > > Thanks, > > Martin Klapetek > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
