[FFmpeg-devel] [PATCH] QCELP decoder

Diego Biurrun diego
Thu Oct 30 23:33:17 CET 2008


On Thu, Oct 30, 2008 at 03:26:05PM -0700, Kenan Gillet wrote:
>
> On Oct 30, 2008, at 3:11 PM, Michael Niedermayer wrote:
>
>> On Thu, Oct 30, 2008 at 02:50:45PM -0700, Kenan Gillet wrote:
>>>
>>> --- libavcodec/celp_filters.h	(revision 15754)
>>> +++ libavcodec/celp_filters.h	(working copy)
>>> @@ -79,8 +79,6 @@
>>>  * @param buffer_length amount of data to process
>>>  * @param filter_length filter length (10 for 10th order LP filter)
>>>  *
>>> - * @return 1 if overflow occurred, 0 - otherwise
>>> - *
>>
>> ok but
>>
>>>  * @note Output buffer must contain 10 samples of past
>>>  *       speech data before pointer.
>>>  *
>>
>> 10 samples ? isnt it filter_length samples?
>
> yes, patch attached that fix doxy for ff_celp_lp_synthesis_filter and  
> ff_celp_lp_synthesis_filterf

Applied.

Diego




More information about the ffmpeg-devel mailing list