[FFmpeg-soc] G.723.1 Encoder: Compute adaptive codebook contribution, obtain residual

Ronald S. Bultje rsbultje at gmail.com
Sat Aug 21 14:29:12 CEST 2010


Hi,

On Aug 21, 2010, at 6:52 AM, Diego Biurrun <diego at biurrun.de> wrote:
> On Fri, Aug 20, 2010 at 11:58:35AM +0530, Mohamed Naufal wrote:
>> 
>> --- a/libavcodec/g723_1.c
>> +++ b/libavcodec/g723_1.c
>> +
>> +    if(!odd_frame) {
> 
> if (
> 
> Please review all your code so that it conforms to K&R style.
> 
>> +        if (pitch_lag == PITCH_MIN)
>> +            pitch_lag++;
>> +        else
>> +            pitch_lag = FFMIN(pitch_lag, PITCH_MAX - 5);
>> +    }
> 
> pointless {}

No, without it gcc won't know which if the else applies to.

Ronald


More information about the FFmpeg-soc mailing list