----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/112188/#review38292 -----------------------------------------------------------
KTp/logs-importer-private.cpp <http://git.reviewboard.kde.org/r/112188/#comment28311> I don't understand why we can't just check m_shouldStop here. Needs documenting at least. KTp/logs-importer.cpp <http://git.reviewboard.kde.org/r/112188/#comment28310> do we still need to delete d? - David Edmundson On Aug. 21, 2013, 4:18 p.m., Dan Vrátil wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/112188/ > ----------------------------------------------------------- > > (Updated Aug. 21, 2013, 4:18 p.m.) > > > Review request for Telepathy. > > > Description > ------- > > When LogsImporter is deleted, it just deletes the importer thread even when > it's running, which will obviously crash. > > This patch adds a safety guard (calling QThread::wait() before deleting it) > and a method to stop the thread prematurely so that we don't wait() for the > entire import to finish (just for current document to finish). > > > Diffs > ----- > > KTp/logs-importer-private.h 06c9c0d > KTp/logs-importer-private.cpp 7275b07 > KTp/logs-importer.cpp 926d265 > > Diff: http://git.reviewboard.kde.org/r/112188/diff/ > > > Testing > ------- > > > Thanks, > > Dan Vrátil > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
