[mythtv] [PATCH] Control duplicates by channel ID

[David Engel]

On Tue, Aug 07, 2007 at 01:56:18PM -0500, David Shay wrote:
> > Yes, the AddHistory should also be qualified by
> >
> >   nextRecording->recstartts <= schedTime
> 
> After some more thought, I wondered if maybe the check should be a
> call to IsSameTimeslot instead?  I'll try some things out tonight.

I don't follow.  In PruneRedundants(), any call to AddHistory() must
be qualified by the current time in order to match how it would have
been called in RunScheduler().  IOW, anything in that might happen in
the future is still conjecture at that point.  Only after the
scheduler would have acted on something, does it become a mater or
record.

> > Yes, only to get accurate status.  Previously, the scheduler kept a
> > day's worth of state in memory for past recordings.  That worked fine
> > as long as the scheduler stayed up.  If the master backend crash or
> > was restarted, however, all of that state was lost and the EPG would
> > display things as if they had never been scheduled.
> 
> Is that status only used in the EPG, though, or where else might it be
> of importance?

Technically, it's important to any user of FromProgram().  It just so
happens the EPG is currently the only user which shows past programs.
All of the other users currently restrict their views to only current
and future programs.

David
-- 
David Engel
david at istwok.net