[FFmpeg-devel] [PATCH 2/2] avformat/ivfenc: use the av1_metadata bsf to insert Temporal Delimiter OBUs if needed

[James Almer]

On 10/3/2018 8:01 PM, Mark Thompson wrote:
> On 03/10/18 01:18, James Almer wrote:
>> Signed-off-by: James Almer <jamrial at gmail.com>
>> ---
>>  libavformat/ivfenc.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c
>> index 66441a2a43..adf72117e9 100644
>> --- a/libavformat/ivfenc.c
>> +++ b/libavformat/ivfenc.c
>> @@ -97,6 +97,8 @@ static int ivf_check_bitstream(struct AVFormatContext *s, const AVPacket *pkt)
>>  
>>      if (st->codecpar->codec_id == AV_CODEC_ID_VP9)
>>          ret = ff_stream_add_bitstream_filter(st, "vp9_superframe", NULL);
>> +    else if (st->codecpar->codec_id == AV_CODEC_ID_AV1)
>> +        ret = ff_stream_add_bitstream_filter(st, "av1_metadata", "td=insert");
>>  
>>      return ret;
>>  }
>>
> 
> I'm not quite seeing why this is wanted - could you explain it a bit further?
> 
> (Since IVF is packetised into temporal units already, it seems to me that having TDs or not in the file won't change anything from the point of view of the consumer.)

No, but ivf afaik doesn't suggest or mandate the removal of TDs, unlike
mp4 and matroska, so i figured it would be best to put them back in
place for such remuxing scenarios.

I don't mind dropping this patch.

> 
> Thanks,
> 
> - Mark
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>