[mythtv] [PATCH] process cutlist without transcoding (final)

Geoffrey Hausheer ou401cru02 at sneakemail.com
Sat Jul 12 20:14:20 EDT 2003


Here is an updated version of the previous transcoder patch I sent
previously.  It fixes a critical bug which was corrupting transcoded
files when there were too many dropped frames in the original.

It also adds a new field to the settings table, 'TranscoderAutoRun',
which can now finally be seperated from 'TranscoderUseCutlist'.  It also
restes 'TranscoderUseCutlist' to default to '1'.  It has never been
enabled in mythtv, so noone should have cared before.  I can't think of
any good reason why it would be disabled, unless the user wants to keep
the commercials in his stream for posterity.

I also have a  fix for a corner case for when the cut ends on a keyframe
(without the fix nothing terrible happens, but we get an additional sync
frame with it, which may give better audio sync right after a cut).  The
fix is to change DoFastForward and DoRewind (in nuppeldecoder) to use
GetFrame (with video-only set), but I'm thinking maybe there's a reason
why this wasn't done originally.  I haven't included this patch, as it
would affect all fastforward/rewind cases, and thus I need to be really
sure it works correctly, but I thought I'd poll the developers as to
whether there is a reason this wasn't originally done?

As a side note, I believe the transcoder will fix many of the mencoder
issues with dropped frames, since it renumbers all frames sequentially
now.  I haven't tested it with mencoder tho, so that is just a guess.

.Geoff
-- 
  Geoffrey Hausheer
  XXXXXXXXXXXXXXXXXXXXX
-------------- next part --------------
A non-text attachment was scrubbed...
Name: transcode9b.patch
Type: application/unknown
Size: 9116 bytes
Desc: not available
Url : http://lists.snowman.net/pipermail/mythtv-dev/attachments/20030712/13726ef3/transcode9b.bin


More information about the mythtv-dev mailing list