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

Christof Buergi christof at buergi.lugs.ch
Mon Oct 14 21:13:40 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. ;-)

-- 
                     _ ___      http://www.p2501.ch/             / \
|/  /\   /\   /\  | \   |  |    christof at buergi.lugs.ch          \ /
|\ /--\ /  \ /  \ | /_  |  |    I'm breaking for DAUs, too!       x
                                Say NO to HTML in mail and news  / \



More information about the MPlayer-dev-eng mailing list