[MPlayer-dev-eng] [PATCH] demux_real.c : make the mess even bigger

[Arpi]

Hi,

> Here is a small fix for demux_real . Of the two stream.dumps I uploaded
> to mplayerhq, one can be played (with correct sync too). The other can't
> perhaps it's a bug in the pnm:// streaming code and not demux_real.

it's possible. i've reportyed such problems to xine, and kept pnm.c synced
with their version. maybe they've fixed it already?

> The first problem was that there can be more headers than num_of_headers
hmm

> says. The second problem was with sync - the rm timestamp seems to be ok,
> so I don't know what that trick is for with sh_video->frametime (it made video
> too fast).
>
> btw who managed to (and how) rev.engeneer (?) this black magic in
> the rm fileformat ?? 

you mean the timestamps? it was me.
how? dumpes first 8 bytes in hex for each frame and look at it to find which
bits looks like a counter... easy.

> btw2 there is a hack in demux_real, because without that sound took
> a long time to catch up with the video. This solves it because until the first
> keyframe arrives, the video is still - and by that time audio will be in sync 
> with video. Or maybe not - could some one tell me what is the real reason 
> than?:)

which hack?

> --- demux_real.c	2003-02-05 23:51:17.000000000 +0100
> +++ demux_real.c.new	2003-02-05 23:50:56.000000000 +0100
> @@ -317,21 +317,22 @@

this patch is a big mess. could you send a clean patch, with no debug stuff,
out-commented timestamp calculation, no cosmetics?


A'rpi / Astral & ESP-team

--
Developer of MPlayer, the Movie Player for Linux - http://www.MPlayerHQ.hu
    "However, many people beg for its inclusion in Debian. Why?" - Gabucino
  "Because having new software in Debian is good." - Josselin Mouette
"Because having good software in Debian is new." - Gabucino