[FFmpeg-devel] [PATCH] avformat/mov: update extensions

Gyan ffmpeg at gyani.pro
Mon Jan 27 18:09:39 EET 2020



On 27-01-2020 09:04 pm, Carl Eugen Hoyos wrote:
> Am Mo., 27. Jan. 2020 um 11:31 Uhr schrieb Gyan Doshi <ffmpeg at gyani.pro>:
>> Added all extensions used by the mov muxer family
>> ---
>>   libavformat/mov.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavformat/mov.c b/libavformat/mov.c
>> index 589576b529..21a5a0180e 100644
>> --- a/libavformat/mov.c
>> +++ b/libavformat/mov.c
>> @@ -8073,7 +8073,7 @@ AVInputFormat ff_mov_demuxer = {
>>       .long_name      = NULL_IF_CONFIG_SMALL("QuickTime / MOV"),
>>       .priv_class     = &mov_class,
>>       .priv_data_size = sizeof(MOVContext),
>> -    .extensions     = "mov,mp4,m4a,3gp,3g2,mj2",
>> +    .extensions     = "mov,mp4,m4a,3gp,3g2,mj2,psp,m4v,m4b,ism,ismv,isma,f4v",
> Is this supposed to fix anything?
> I ask because iirc the demuxer does not work for files not detected or do I
> misremember?

Keeping the list up to date.  FATE passed. Do you have cases where this 
leads to a false positive?

Primary motivation is to replace the unwieldy format short name with am 
actually short & representative string. I've locally added 
av_find_input_format2 which will look at extensions as well so that 
existing commands with `-f mp4` ..etc continue to work,

Gyan


More information about the ffmpeg-devel mailing list