[FFmpeg-devel] [PATCH] dash: change default MP4 extension to .m4s

[Alfred E. Heggestad]

On 18/06/2019 04:02, Steven Liu wrote:
> Alfred E. Heggestad <alfred.heggestad at gmail.com> 于2019年6月17日周一 下午4:02写道：
>>
>>   From 923da82598bddd1ed05750427dbc71e607d296a2 Mon Sep 17 00:00:00 2001
>> From: "Alfred E. Heggestad" <alfred.heggestad at gmail.com>
>> Date: Mon, 17 Jun 2019 09:59:04 +0200
>> Subject: [PATCH] dash: change default MP4 extension to .m4s
>>
>> this was changed in commit 281a21ed50849e3c8c0d03005230e9fd07c24370
>> ---
>>    libavformat/dashenc.c | 2 +-
>>    1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c
>> index 3fd7e78166..a51a1da0ca 100644
>> --- a/libavformat/dashenc.c
>> +++ b/libavformat/dashenc.c
>> @@ -166,7 +166,7 @@ static struct format_string {
>>        const char *str;
>>    } formats[] = {
>>        { SEGMENT_TYPE_AUTO, "auto" },
>> -    { SEGMENT_TYPE_MP4, "mp4" },
>> +    { SEGMENT_TYPE_MP4, "m4s" },
>>        { SEGMENT_TYPE_WEBM, "webm" },
>>        { 0, NULL }
>>    };
>> --
>> 2.20.1 (Apple Git-117)
>>
>> _______________________________________________
>> ffmpeg-devel mailing list
>> ffmpeg-devel at ffmpeg.org
>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>> To unsubscribe, visit link above, or email
>> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> 
> 
> 
> LGTM
> 

the background for this is the extension for DASH media files
used to be *.m4s and it is now *.mp4


the patch is a suggestion and should be checked by the DASH experts

what is correct according to the standard ?

the media-file is not really an .mp4 file, it cannot be
played with e.g. ffplay:

   $ ffplay chunk-stream1-00001.m4s




/alfred