[mythtv-firehose] mythtv/master commit: 1c9f2801d by Daniel Kristjansson (daniel-kristjansson)

MythTV noreply at mythtv.org
Sat Jul 23 15:23:58 UTC 2011

      Author:  Daniel Kristjansson <danielk at cuymedia.net>
 Change Date:  2011-07-23T08:05:15-07:00
   Push Date:  2011/07/23 08:23:51 -0700
  Repository:  mythtv
      Branch:  master
New Revision:  1c9f2801d3a3127707ac26423f0bc72f21e795e2
   Changeset:  https://github.com/MythTV/mythtv/commit/1c9f2801d


Idiot proof logging thread dtors.

Basically this is the same thing I applied to all the threads that were ported from pthread to QThread.

It's an error if the threads are still running when we enter the thread dtor, this makes sure the threads are stopped if they we somehow missed waiting on them to stop before calling the dtor.

I experienced the error condition only with LoggerThread, but the consequences of deleting a thread's stack while it is still running are dire and the fix quite simple and safe so I applied it to both threads.



More information about the mythtv-firehose mailing list