[MPlayer-dev-eng] [PATCH] wrong bitrate assignment when using mp3lame

[D Richard Felker III]

On Mon, Mar 07, 2005 at 04:27:30PM +0100, Nico Sabbi wrote:
> D Richard Felker III wrote:
> 
> >On Mon, Mar 07, 2005 at 08:45:58AM +0100, Nico Sabbi wrote:
> > 
> >
> >>This patch fixes the initial assignment of AvgBytesPerSecond to the 
> >>value specified with :br=
> >>when encoding with mp3lame; later this var is fixed as needed, but at 
> >>least two
> >>muxers will work correctly (mpeg and lavf).
> >>
> >>If no one is against it I will commit tonight.
> >>   
> >>
> >
> >Keep in mind it won't work unless you use cbr..
> >
> >Rich
> >
> > 
> >
> 
> can you explain why, please?

The br= parameter won't tell you anything unless you're encoding cbr
mode. Even with abr the average often comes out 10-15% lower (but
sometimes higher too) than the requested bitrate. Using this for a/v
sync is not acceptable..

Rich