[Ffmpeg-devel] [PATCH] accept only existing bitrates in amr.c
Diego Biurrun
diego
Thu Mar 29 11:38:34 CEST 2007
On Tue, Mar 27, 2007 at 12:12:32PM +0200, Benoit Fouet wrote:
>
> Michael Niedermayer wrote:
> >
> > On Tue, Mar 06, 2007 at 09:53:32AM +0100, Benoit Fouet wrote:
> >
> >> Benoit Fouet wrote:
> >>
> >>> M?ns Rullg?rd wrote:
> >>>
> >>>> Michael Niedermayer <michaelni at gmx.at> writes:
> >>>>
> >>>>> i prefer using the return value for both enum and -1 error yes, enum==int
> >>>>> anyway
> >>>>> or if you recently have been brainwashed by some CS teachers then just add
> >>>>> a enum entry for error and give it the value -1
> >>>>>
> >>>> An enum isn't required to be int. The compiler may choose any integer
> >>>> type capable of representing the full range of the enum values. The
> >>>> enumeration constants have type int, but that's another matter.
> >>>>
> >>> well, the enumerated has a unsigned type in our case...
> >>> so, i can either have two different behaviours for NB and WB, or change
> >>> "enum Mode mode" to "int mode" in AMR context...
> >>> i don't know what is better to do...
> >>>
> >> for you to see the difference, new patch attached...
> >
> > patch ok
> >
> patch against latest tree attached, can someone apply it, please ?
Applied.
Diego
More information about the ffmpeg-devel
mailing list