[FFmpeg-devel] [PATCH] Add IFF metadata handling
Sebastian Vater
cdgs.basty
Fri Apr 30 14:03:15 CEST 2010
Vladimir Pantelic a ?crit :
> Sebastian Vater wrote:
>> Hi guys!
>>
>> Just did a small patch which adds new IFF metadata stuff.
>>
>> The original IFF demuxer just processed the comment metadata stuff,
>> therefore I added title, copyright and author metadata stuff to IFF
>> demuxer.
>>
>> It's really a small patch and shouldn't be of any problems.
>>
>
> + buf = av_malloc(data_size + 1);
> + if (!buf)
> + break;
> + get_buffer(pb, buf, data_size);
> + buf[data_size] = 0;
>
> could be factored out
Factored out? Do you mean putting it into a #define and just call the
define's?
--
Best regards,
:-) Basty/CDGS (-:
More information about the ffmpeg-devel
mailing list