[FFmpeg-devel] [PATCH v2 3/5] avcodec/bsf: Add list BSF API

[Jan Sebechlebsky]

On 08/15/2016 03:40 AM, Michael Niedermayer wrote:
> On Fri, Aug 05, 2016 at 02:00:25PM +0200, sebechlebskyjan at gmail.com wrote:
>> From: Jan Sebechlebsky <sebechlebskyjan at gmail.com>
>>
>> ---
>>   Changes from last version:
>>   - fixed doxygen comments
>>   - added av_bsf_list_append2() function
>>   - changed names of array and length fields to follow naming convention
>>   - idx and flushed_idx is now unsigned
>>   - merged bsf_list_flush to bsf_list_filter to reduce code duplication
>>   - aligned structure fields initialization
>>   - added check for NULL to av_bsf_free()
>>   - fixed memleak in av_bsf_finalize in case there is only 1 filter
>>
>>   libavcodec/avcodec.h |  85 ++++++++++++++++
>>   libavcodec/bsf.c     | 279 +++++++++++++++++++++++++++++++++++++++++++++++++++
>>   2 files changed, 364 insertions(+)
> seems to break build or i did something silly
>
> CC      libavcodec/bsf.o
> libavcodec/bsf.c: In function ‘bsf_list_close’:
> libavcodec/bsf.c:348:18: error: ‘BSFListContext’ has no member named ‘item_name’
> make: *** [libavcodec/bsf.o] Error 1
>
>
That is my mistake, sorry for that - when I was splitting custom item 
name functionality to separate patch I left line freeing item name in 
the first patch. I will move it to the correct patch and resend.

Regards,
Jan