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

[Balatoni Denes]

Hi!

> > 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.
that part is actually documented :)
i originally meant the video packet parser inside a data packet

> > 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?
that with I / P frames

> > --- 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?
what is uncommented, is uncommented for a reason (timestamp calculation is 
wrong, it made video play too fast)
there is no debug stuff afaict 
or you mean the number of packets reached 0 ?
that is no debug, that is a bug there, commented ;)
cosmetics?
that is one line changed by accident - deal with it, I mean it
can't be hard there are 100 printf s and gotos in the code otherwise


bye
Denes





>
>
> A'rpi / Astral & ESP-team