[FFmpeg-devel] [PATCH] broaden ff_set_fixed_vector

Ronald S. Bultje rsbultje
Wed Jan 27 02:59:41 CET 2010


Hi,

On Jan 26, 2010, at 8:51 PM, M?ns Rullg?rd <mans at mansr.com> wrote:
> "Ronald S. Bultje" <rsbultje at gmail.com> writes:
>> /** Sparse representation for the algebraic codebook (fixed) vector  
>> */
>> typedef struct {
>>     int      n;
>> -    int      x[10];
>> -    float    y[10];
>> +    struct {
>> +        unsigned int single:1;
>> +        unsigned int x:31;
>> +        float        y;
>> +    } pulses[10];
>>     int      pitch_lag;
>>     float    pitch_fac;
>> } AMRFixed;
>
> Why the bit-fields?  GCC often generates incredibly stupid code for
> bit-fields.

I'm hoping gcc is smarter than that?

Ronald



More information about the ffmpeg-devel mailing list