[FFmpeg-devel] [PATCH] Add supported_samplerates field to AVCodec
Mon Apr 21 23:46:28 CEST 2008
On date Monday 2008-04-21 19:11:13 +0200, Michael Niedermayer encoded:
> On Mon, Apr 21, 2008 at 05:51:25PM +0200, Stefano Sabatini wrote:
> > > > + const int *supported_samplerates; ///< array of supported audio samplerates, or NULL if unknown, array is terminated by -1
> > >
> > > i would prefer if the array where terminated by 0
> > I don't like this solution, in this way we can't distinguish if a
> > codec doesn't support a certain value or if we don't know or we don't
> > want to specify the valid values.
> > For example suppose we define a common function which checks if the
> > provided value for the samplerate is valid looking at the
> > supported_samplerates value, if it is set to NULL then the check is
> > skipped and will be eventually executed in the open_codec function,
> > while if the value is set to -1 (empty list of valid values) then it
> > will always fail.
> Learn C, especially the difference between pointer and pointed to value.
My bad, sorry for the misunderstanding.
New patch attached, regards.
FFmpeg = Fiendish & Friendly MultiPurpose ExchanGer
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1012 bytes
Desc: not available
More information about the ffmpeg-devel