[FFmpeg-devel] [PATCH][RFC] avcodec/avutil: Add timeline side data
Jan Ekström
jeebjp at gmail.com
Wed Mar 28 21:35:12 EEST 2018
On Wed, Mar 28, 2018 at 9:12 PM, Michael Niedermayer
<michael at niedermayer.cc> wrote:
>> + /**
>> + * An in-order array of entries for the given timeline.
>> + * Each entry contains information on which samples to display for a
>> + * particular edit.
>> + */
>> + AVTimelineEntry *entries;
>
> This is problematic as its non extensible. (unless iam missing something)
> Consider that a field is added to AVTimelineEntry, the entries array would
> have a larger element size and that would require all user apps to be rebuild
>
So you would prefer some sort of iterator? Or a size entry like with
some of the entries in AVEncryptionInfo? I have a feeling this was
based off of the following added into AVEncryptionInfo:
+ AVSubsampleEncryptionInfo *subsamples;
+ uint32_t subsample_count;
Which now would seem to be similarly not being extendable?
> Also, if you want to support quicktime more fully theres more needed.
> QT can for example switch even between codecs mid stream IIRC
> not sure we want to support this
>
Thankfully, that has little to do with virtual timelines (edit lists)
as far as I can tell?
Jan
More information about the ffmpeg-devel
mailing list