[mythtv] Scheduling bug with instant record.

Chris Pinkham cpinkham at bc2va.org
Wed Nov 19 09:07:28 EST 2003


> On Wed, Nov 19, 2003 at 12:09:31AM -0500, Chris Pinkham wrote:
> > I assume we might need some locking or some wrapper functions since the
> > scheduler sometimes just returns pointers to its own internal lists.  I'll
> 
> Yeah, locking will definitely be needed.  The cleanest way will
> probably be to put the lock inside the scheduler and have the
> appropriate scheduler methods return copies of the internal lists.

So, how's this sound:

Add locks around anything that touches scheduledList and recordingList
then make new overload/wrapper methods for things like
Scheduler::getAllScheduled() that take a programinfo list as an argument.
Inside the wrapper/overloaded method, the appropriate lock will be grabbed;
then the local scheduler list will be retrieved and copied into the
list given in the argument; then the lock will be released.

Make sense or am I missing anything?

-- 

Chris



More information about the mythtv-dev mailing list