[libav-devel] [PATCH] lavc: move some flac-specific options to its private context.

Ronald S. Bultje rsbultje at gmail.com
Mon Apr 4 17:30:39 CEST 2011


Hi,

On Mon, Apr 4, 2011 at 3:21 PM, Justin Ruggles <justin.ruggles at gmail.com> wrote:
> On 04/02/2011 01:19 PM, Anton Khirnov wrote:
>> +#if FF_API_FLAC_GLOBAL_OPTS
>>      /**
>>       * Determines which LPC analysis algorithm to use.
>>       * - encoding: Set by user
>>       * - decoding: unused
>>       */
>> -    enum AVLPCType lpc_type;
>> +    attribute_deprecated enum AVLPCType lpc_type;
>
> Should we also deprecate the AVLPCType enum and move it to lpc.h without
> the AV prefix?  I don't think private options need to have public enums
> for their values.

Does AVOption attempt to solve this in any way? I.e., does AVOption
have a mechanism to link a string to a numerical value to create a
"enum-style" list of (random) integers and (meaningful) strings?

Ronald


More information about the libav-devel mailing list