[mythtv-commits] mythtv commit: r23415 by cpinkham

[mythtv at cvs.mythtv.org]

      Author: cpinkham
        Date: 2010-02-01 03:49:37 +0000 (Mon, 01 Feb 2010)
New Revision: 23415
   Changeset: http://svn.mythtv.org/trac/changeset/23415

Modified:

   trunk/mythtv/libs/libmythtv/jobqueue.cpp
   trunk/mythtv/libs/libmythtv/jobqueue.h

Log:

Fix a regression in the JobQueue from [21369].  During the QMap key
conversion from chanid+starttime to jobID, the JobQueue lost the
ability to check for another job running for the same recording
and multiple jobs could be running at the same time for a given
recording.  This patch fixes this regression so that multiple
jobs for a given recording are run serially instead of in parallel.

Also add a locking in some places around reads of the runningJobs QMap.
These haven't been there since the beginning of the JobQueue, but we
need to protect our reads to make sure we're not reading while the
QMap is being updated in another thread.

References #7731 since the regression above was mentioned in this ticket.