On Sun, Dec 18, 2011 at 1:40 AM, Luca Barbato <[email protected]> wrote:

> On 18/12/11 00:16, Asen Lekov wrote:
>
>> diff --git a/libavcodec/qcelpdec.c b/libavcodec/qcelpdec.c
>> index 20a0484..73d95d3 100644
>> --- a/libavcodec/qcelpdec.c
>> +++ b/libavcodec/qcelpdec.c
>> @@ -90,13 +90,13 @@ typedef struct
>>   */
>>  static av_cold int qcelp_decode_init(**AVCodecContext *avctx)
>>  {
>> -    QCELPContext *q = avctx->priv_data;
>> +    QCELPContext *q   = avctx->priv_data;
>>
>
> Why?
>
>       int i;
>>
>>      avctx->sample_fmt = AV_SAMPLE_FMT_FLT;
>>
>> -    for(i=0; i<10; i++)
>> -        q->prev_lspf[i] = (i+1)/11.;
>> +    for (i = 0; i<  10; i++)
>> +            q->prev_lspf[i] = (i + 1) / 11.;
>>
> Why?       ^^^^
>
> > +    int          i;
> > +    float        tmp_lspf, smooth, erasure_coeff;
> > +    const float  *predictors;
>
> Vertical alignment shouldn't be used in those cases.


Now i`ll fix it, the new patch as new mail or reply here?
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to