[MPlayer-dev-eng] [PATCH] fixes subtitle dropouts

Christof Buergi christof at buergi.lugs.ch
Mon Oct 14 21:17:47 CEST 2002


This is the first patch I ever submit, so please don't shoot me right
away. ;-)

All right: The patch adresses two issues which I found, when I analyzed
the input from some DVDs with known subtitle-dropouts:

1. The packet-size at the beginning of the packet, which is used to
check, whether we got all fragments, is sometimes one byte too long. It
seems to be always padded to an even number, while the actual size can
be odd.

2. The original algorythm used to assemble the fragments relies on the
timestamps to check, whether a new packet begins. This has proven to be
unrelieable on some disks. So instead, I use the timestamp only to
check, whether it's been too long (defined as 0,01sec) since the last
fragment, which is probably indicating a broken packet, and normaly
starting a new packet when the last one has been finished.

NOTE: I commented every change and added my initials, because this is
corporate practice at my workplace. I like this practice, because you
can see immediatly, who did what and why. But if you don't like it,
just erase it. ;-)

PS: Sorry for the double post. Clicked the wrong button. (Feeling sooo 
stupid)

-- 
                     _ ___      http://www.p2501.ch/             / \
|/  /\   /\   /\  | \   |  |    christof at buergi.lugs.ch          \ /
|\ /--\ /  \ /  \ | /_  |  |    I'm breaking for DAUs, too!       x
                                Say NO to HTML in mail and news  / \
-------------- next part --------------
A non-text attachment was scrubbed...
Name: spudec.patch
Type: text/x-diff
Size: 1186 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20021014/845dabfc/attachment.patch>


More information about the MPlayer-dev-eng mailing list