[MPlayer-dev-eng] [PATCH] (2nd) Fix demux_avi >=2GB overflow & enhance avi info output

eviv bulgroz ebulgroz at yahoo.com
Thu Dec 26 12:49:28 CET 2002


Oops, oh well ... ;-)
I saw a few minor differences like no a/v dwLength fix (I guess it's intended), no audio bps
printed, no tidle ~ printed when using fallback and also a few more relevant ones :
- uses video time length instead of audio time length to calculate audio size in fallback ?
- differences in fallback for priv->numberofframes calculation
- run time not printed (nice to have though, at least in verbose)

I prepared a small patch for the last three items


--- Arpi <arpi at thot.banki.hu> wrote:
> Hi,
> 
> > Here is second try to that demux_avi patch :
> 
> eh a bit too late
> i've already applied your first version with my heavy changes.
> 
> > 
> > - fix dwLength now counted in samples, not in chunks,
> >   use priv->audio_block_size for audio dwlength calc.
> > - audio_sh != NULL now always tested 
> > - fallback for a/v size calc. included again 
> >   and fixed in audio size calc.to use sh_audio->wf->nAvgBytesPerSec.
> > (prev. email)
> > - fix fallback in priv->numberofframes calc. (new)
> > - tilda "~" printed when a/v size calc. fallback used.

__________________________________________________
Do you Yahoo!?
Yahoo! Mail Plus - Powerful. Affordable. Sign up now.
http://mailplus.yahoo.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: demux_avi_fix_bps_overflow2.diff
Type: application/octet-stream
Size: 2837 bytes
Desc: demux_avi_fix_bps_overflow2.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20021226/0084ede7/attachment.obj>


More information about the MPlayer-dev-eng mailing list