[FFmpeg-devel] [PATCH 04/21] libavcodec: added a structure AVData for decoding timed metadata

[Paul B Mahol]

On 8/23/16, Nicolas George <george at nsup.org> wrote:
> Le septidi 7 fructidor, an CCXXIV, erkki.seppala.ext at nokia.com a écrit :
>> From: Erkki Seppälä <erkki.seppala.ext at nokia.com>
>>
>> Also added avdata_alloc and avdata_free for dealing with it. AVData
>> can contain arbitrary binary data and comes with a format-field so far
>> unused.
>>
>> The purpose is that AVMEDIA_TYPE_DATA -kind codecs can store frames in
>> this format.
>
> Please consider using AVFrame itself instead. It has all the necessary
> fields. It has many more fields, but is it really an issue?

Yes, please use AVFrame.

>
> The strongest case for using AVFrame is uniformity. Thanks to wm4, we now
> have the same API for audio and video decoding. With that, code can be
> written to manipulate frames in a generic manner. Timed metadata would be
> handled the same way without extra code.
>
> Integration in libavfilter would be easier too.