Review Request 112188: Don't delete LogsImporter thread while it's still running

Dan Vrátil dvratil at redhat.com
Wed Aug 21 16:18:52 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/112188/
-----------------------------------------------------------

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130821/6c2d3a64/attachment.html>


More information about the KDE-Telepathy mailing list