[FFmpeg-devel] [PATCH][RFC] avcodec/avutil: Add timeline side data

[wm4]

On Wed, 28 Mar 2018 19:42:21 +0100
Derek Buitenhuis <derek.buitenhuis at gmail.com> wrote:

> On 3/28/2018 7:12 PM, Michael Niedermayer wrote:
> > 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  
> 
> Do you have an actual suggest though? Afaict, this is exactly how the recently
> pushed encryption support works with AVSubsampleEncryptionInfo entries in the
> AVEncryptionInfo struct. is this now not acceptable, even though we pushed
> that a few days ago after months of review?
> 
> I'm all ears to a solution...

You just need to change * to **. Then the size of the struct doesn't
matter anymore for array access and can be excluded from the ABI.

> > 
> > 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  
> 
> Codec switching is not related to timelines at all. It's entirely separate...
> 
> - Derek
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel