[FFmpeg-devel] [PATCH] avformat: add apic to AVStream

[James Almer]

On 3/29/2021 4:38 AM, Nicolas George wrote:
> Lynne (12021-03-29):
>> Personally I don't really like 'apic'. Maybe 'pic_attachment' or 'attachment_pic'?
> 
> By the way, do we want it to be a field?
> 
> Attached pictures can be quite big. Do we want them to stay in AVStream
> forever, even when the application has already finished using them, or
> does not use them at all?
> 
> Regards,

The idea for attached_pic afaik was that it's always available and 
easily accessible to the user from the start, instead of just the one 
time the demuxing process would return it in an av_read_frame() call.
The doxy for AV_DISPOSITION_ATTACHED_PIC even mentions that if you seek 
the relevant packet may not be returned at all.