[mythtv-firehose] mythtv/devel/mythtv-rec2 commit: 35fd63cf0 by Daniel Kristjansson (daniel-kristjansson)

[MythTV]

      Author:  Daniel Kristjansson <danielk at cuymedia.net>
 Change Date:  2011-03-22T17:58:35-07:00
   Push Date:  2012/04/10 00:41:52 -0700
  Repository:  mythtv
      Branch:  devel/mythtv-rec2
New Revision:  35fd63cf01e68ada84694728f3ea68f462c5f3da
   Changeset:  https://github.com/MythTV/mythtv/commit/35fd63cf0

Log:

Threading Scheduler cleanups. See [a5a2a9c0192f3ab76829b351ab42346006295361].

 * Gets rid of the use of QThread::terminate() in scheduler shutdown (unsafe).
 * Makes sure the thread is running when needed using a conditional.
 * Makes Scheduler a QThread itself since it was already a QObject.

The first change is for a problem that existed before the port, I just happened to catch it because I was doing a code review. The second change is for a problem introduced by the port (very unlikely to cause any problems in this case). The third change should not affect functionality at all.

Modified:

   mythtv/programs/mythbackend/scheduler.cpp
   mythtv/programs/mythbackend/scheduler.h