[MPlayer-dev-eng] [PATCH] fix for -srate bug

[D Richard Felker III]

On Thu, Oct 21, 2004 at 01:22:20PM +0200, Diego Biurrun wrote:
> D Richard Felker III writes:
> > On Tue, Oct 19, 2004 at 01:37:02AM +0200, Diego Biurrun wrote:
> > > Giacomo Comes writes:
> > > > 
> > > > This should go in the documentation, or mencoder should be fixed and be able to use
> > > > -af resample without -srate.
> > > 
> > > What is the problem, -af should work with MEncoder.
> > 
> > it doesn't. when the broken af was first committed it was designed for
> > the output end to be overridden by the negotiated parameters between
> > the codec and the ao driver. mplayer was fixed but mencoder never was.
> > so even if you specify resample, mencoder will reconfigure your
> > resample filter to resample to the original rate... :(
> 
> OK, let me try to distill this into documentation.  Currently in the
> man page we have
> 
> resample[=srate[:sloppy][:type]]
>   Changes the sample rate of the audio stream to an integer srate in Hz.
>   It only supports the 16-bit little-endian format.
> 
> IIUC I would just have to mark this MPlayer only, right?  Like so:

no. it's not mplayer-only!! it works with either, but you must
force-override the output samplerate to keep mencoder from undoing
your filter chain. so, something like:

-af resample=44100:0:0 -srate 44100

adding -af-adv force=3 (disable) couldn't hurt either..

rich