[FFmpeg-devel] [PATCH v17] avformat/dashdec: add dash demuxer base version

Steven Liu lingjiujianke at gmail.com
Wed Aug 30 15:21:54 EEST 2017


2017-08-30 19:50 GMT+08:00 Timo Rothenpieler <timo at rothenpieler.org>:
>>> Just wanted to point out, as stated on IRC, that dashenc already has a
>>> function that seems to safely solve the filename template issue, and
>>> looks
>>> like it can be moved into a common file + header, and reused as is:
>>>
>>>
>>> http://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavformat/dashenc.c;h=bdb332c71ab4aae87201c87c3d43938a07a9ed5c;hb=HEAD#l314
>>>
>> I have try to used that APIs, but it need some modify :
>> template $Number-%d$ will ok, but $Number$ will wrong, maybe mux
>> operation different to demux, or i misunderstand some thing?
>
>
> This looks to me like just $Number$ should work fine as well.
> It will expand to, basically, printf("%d", number);
>
> http://git.videolan.org/?p=ffmpeg.git;a=blob;f=libavformat/dashenc.c;h=bdb332c71ab4aae87201c87c3d43938a07a9ed5c;hb=HEAD#l289
>
> If there is some feature missing from it, it can most likely easily be added
> without breaking dashenc.

modify to it later after this patch merged :D
That modify will be clear than this patch
This patch is too big now, full review is too hard work.
Now i have use get_repl_pattern_and_format + get_segment_filename fix it.
>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>


More information about the ffmpeg-devel mailing list