[FFmpeg-soc] G.723.1 Encoder: Compute adaptive codebook contribution, obtain residual
Mohamed Naufal
naufal11 at gmail.com
Thu Sep 30 21:58:34 CEST 2010
On 21 August 2010 16:22, 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 {}
>
>> +/**
>> + * Subtract the adaptive codebook contribution from the input
>> + * to obtain the residual.
>> + *
>> + * @param buf target vector
>> + */
>
> This is missing doxygen documentation for the other parameters. Please
> do not add incomplete documentation. Try running 'doxygen Doxyfile' and
> fix all the warnings your code generates.
>
Fixed locally.
Thanks
Naufal
More information about the FFmpeg-soc
mailing list