[FFmpeg-devel] [PATCH] QCELP decoder

[Vitor Sessak]

Michael Niedermayer wrote:
> On Thu, Nov 20, 2008 at 10:46:49AM -0800, Kenan Gillet wrote:
>> On Sat, Nov 15, 2008 at 3:10 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>>> On Fri, Nov 14, 2008 at 03:32:51PM -0800, Kenan Gillet wrote:
>>>> Hi,
>>>> On Fri, Nov 14, 2008 at 2:27 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
>>>>> On Fri, Nov 14, 2008 at 12:17:50PM -0800, Kenan Gillet wrote:
>>>>>> On Nov 14, 2008, at 2:14 AM, Michael Niedermayer wrote:
> [...]
>>>> +
>>>> +/// @defgroup qcelp_unpacked_data_frame QCELP unpacked data frame
>>>> +/// @{
>>>> +    uint8_t           cbsign[16];
>>>> +    uint8_t           cbgain[16];
>>>> +    uint8_t           cindex[16];
>>>> +    uint8_t           plag[4];
>>>> +    uint8_t           pfrac[4];
>>>> +    uint8_t           pgain[4];
>>>> +    uint8_t           lspv[10];               /*!< LSP for RATE_OCTAVE, LSPV for other rates */
>>>> +    uint8_t           reserved;               /*!< on all but rate 1/2 packets */
>>>> +/// @}
>>>> +
>>>> +    uint8_t           erasure_count;
>>>> +    uint8_t           octave_count;           /*!< count the consecutive RATE_OCTAVE frames */
>>>> +    float             prev_lspf[10];
>>>> +    float             predictor_lspf[10];     /*!< LSP predictor,
>>>> +                                                  only use for RATE_OCTAVE and I_F_Q */
>>>> +    float             pitch_synthesis_filter_mem[303];
>>>> +    float             pitch_pre_filter_mem[303];
>>>> +    float             rnd_fir_filter_mem[180];
>>>> +    float             formant_mem[170];
>>>> +    float             last_codebook_gain;
>>>> +    int               prev_g1[2];
>>>> +    int               prev_framerate;
>>>> +    float             prev_pitch_gain[4];
>>>> +    uint8_t           prev_pitch_lag[4];
>>>> +    uint16_t          first16bits;
>>>> +} QCELPContext;
>>> i somehow think this struct does not belong in qcelpdata.h
>>> but rather qcelpdec.c
>>>

[...]

> 
> ok

Applied ok'ed parts, together with the typo fix.

-Vitor