[mythtv] [PATCH] HDTV Recorder Patch
Isaac Richards
ijr at case.edu
Thu Dec 2 00:57:46 UTC 2004
On Wednesday 01 December 2004 07:21 pm, Daniel Thor Kristjansson wrote:
> On Tue, 30 Nov 2004, Doug Larrick wrote:
> ]Sure sounds like it's not detecting keyframes properly. You might want to
> try ]my tsdump utility on a failing recording
> <http://jekyl.no-ip.org/doug/tsdump.C> ]and see if that program sees GOP or
> picture start.
>
> The recordings from PBS do not have GOP's, just picture starts. I'm
> testing a fallback to update the position map anyway. But I'm not
> sure why it's even recording this. I think maybe it's because we also
> pretend to have seen a GOP if we see 00 00 01 B3, the sequence header.
> Can we use the sequence headers instead of the GOP for the position map
> too?
Yeah, the old hdtvrecorder would treat sequence headers as gop starts if no
real gop headers were found. Looking over your patch, I think you've broken
it slightly:
else if (0xB8 == k1)
HandleGOP();
else if (0xB3 == k1 && !_gop_seen)
HandleGOP();
Which'll treat a sequence header as a gop a couple times, but HandleGOP will
set _gop_seen to true after a few calls, and not allow it to happen anymore.
Isaac
More information about the mythtv-dev
mailing list