[FFmpeg-devel] [PATCH] QCELP decoder

Vitor Sessak vitor1001
Mon Dec 1 23:07:46 CET 2008


Michael Niedermayer wrote:
> On Mon, Dec 01, 2008 at 12:45:09PM -0800, Kenan Gillet wrote:
>> Hi,
>> On Mon, Dec 1, 2008 at 8:24 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
>>> On Mon, Dec 01, 2008 at 08:17:52AM -0800, Kenan Gillet wrote:
>>>> On Dec 1, 2008, at 4:52 AM, Michael Niedermayer wrote:
>>>>
>>>>> On Sun, Nov 30, 2008 at 05:04:07PM -0800, Kenan Gillet wrote:
>>>>>> Hi,
>>>>>> On Nov 30, 2008, at 7:50 AM, Michael Niedermayer wrote:
>>>>>>
>>>>>>> On Sat, Nov 29, 2008 at 10:39:58AM -0800, Kenan Gillet wrote:
>>> [...]
>>>>> [...]
>>>>>>>> +    /**
>>>>>>>> +     * reserved bits on all bitrate but bitrate 1/2 packets
>>>>>>> this is unclear, field that is on all but ... , vs. field that
>>>>>>> exists always but
>>>>>>> is reserved on all but .....
>>>>>>>
>>>>>> is "reserved bits only set for bitrate 1, 1/4 and 1/8" better ?
>>>>> no, because setting them means error IIRC
>>>>> also IIRC there is no indication of the existence of other fields
>>>>> for the
>>>>> other rates ...
>>>>>
>>>> reserved bits only present in bitrate 1, 1/4 and 1/8 packets
>>>>
>>>> ?
>>> ok
>>>
>>> [...]
>> here is an updated round 14 of the patches,
>> which is getting smaller and smaller :)

Ok'ed parts applied.

-Vitor




More information about the ffmpeg-devel mailing list