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

2017-08-29 Thread Steven Liu
ffmpeg need a dash demuxer for demux the dash formats base on https://github.com/samsamsam-iptvplayer/exteplayer3/blob/master/tmp/ffmpeg/patches/3.2.2/01_add_dash_demux.patch TODO: 1. support multi bitrate dash v2 fixed: 1. from autodetect to disabled 2. from camelCase code style to ffmpeg co

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

2017-03-20 Thread Steven Liu
2017-03-19 18:12 GMT+08:00 Nicolas George : > Le nonidi 29 ventôse, an CCXXV, Rodger Combs a écrit : > > fragmentTimescale. > > fragment_timescale or fragment_time_scale; camelCase is not part of > FFmpeg's coding standards. > merged! > > > Use time_t or int64_t. > > Certainly not time_t. > > The

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

2017-03-20 Thread Steven Liu
2017-03-19 18:06 GMT+08:00 Rodger Combs : > A few initial comments inline: > > > On Mar 19, 2017, at 04:05, Steven Liu wrote: > > > > ffmpeg need a dash demuxer for demux the dash formats > > base on https://github.com/samsamsam-iptvplayer/exteplayer3/blob/ > master/tmp/ffmpeg/patches/3.2.2/0

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

2017-03-19 Thread Alexander Strasser
On 2017-03-19 11:01 +0100, wm4 wrote: > On Sun, 19 Mar 2017 10:57:21 +0100 > Clément Bœsch wrote: > > > > > If you want to have DASH enabled by default, make sure to have a native > > XML parser, or discuss the policy about detecting all the external > > libraries within the configure. > > > >

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

2017-03-19 Thread Nicolas George
Le nonidi 29 ventôse, an CCXXV, Rodger Combs a écrit : > fragmentTimescale. fragment_timescale or fragment_time_scale; camelCase is not part of FFmpeg's coding standards. > Use time_t or int64_t. Certainly not time_t. The rest looks fine at first glance. Regards, -- Nicolas George signat

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

2017-03-19 Thread Rodger Combs
A few initial comments inline: > On Mar 19, 2017, at 04:05, Steven Liu wrote: > > ffmpeg need a dash demuxer for demux the dash formats > base on > https://github.com/samsamsam-iptvplayer/exteplayer3/blob/master/tmp/ffmpeg/patches/3.2.2/01_add_dash_demux.patch > > Signed-off-by: Steven Liu

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

2017-03-19 Thread wm4
On Sun, 19 Mar 2017 10:57:21 +0100 Clément Bœsch wrote: > > If you want to have DASH enabled by default, make sure to have a native > XML parser, or discuss the policy about detecting all the external > libraries within the configure. > That's ridiculous. So NIH is encouraged because of a crap

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

2017-03-19 Thread Nicolas George
Le nonidi 29 ventôse, an CCXXV, Clement Boesch a écrit : > If you want to have DASH enabled by default, make sure to have a native > XML parser For the record, I have started something a few months back. Not very advanced, because other more pressing matters arose, but reasonably clean for a start

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

2017-03-19 Thread Clément Bœsch
On Sun, Mar 19, 2017 at 05:51:15PM +0800, Steven Liu wrote: > 2017-03-19 17:48 GMT+08:00 Clément Bœsch : > > > On Sun, Mar 19, 2017 at 05:05:55PM +0800, Steven Liu wrote: > > > ffmpeg need a dash demuxer for demux the dash formats > > > base on https://github.com/samsamsam-iptvplayer/exteplayer3/b

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

2017-03-19 Thread Steven Liu
2017-03-19 17:55 GMT+08:00 Paul B Mahol : > On 3/19/17, Steven Liu wrote: > > 2017-03-19 17:48 GMT+08:00 Clement Boesch : > > > >> On Sun, Mar 19, 2017 at 05:05:55PM +0800, Steven Liu wrote: > >> > ffmpeg need a dash demuxer for demux the dash formats > >> > base on https://github.com/samsamsam-i

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

2017-03-19 Thread Paul B Mahol
On 3/19/17, Steven Liu wrote: > 2017-03-19 17:48 GMT+08:00 Clement Boesch : > >> On Sun, Mar 19, 2017 at 05:05:55PM +0800, Steven Liu wrote: >> > ffmpeg need a dash demuxer for demux the dash formats >> > base on https://github.com/samsamsam-iptvplayer/exteplayer3/blob/ >> master/tmp/ffmpeg/patche

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

2017-03-19 Thread Steven Liu
2017-03-19 17:48 GMT+08:00 Clément Bœsch : > On Sun, Mar 19, 2017 at 05:05:55PM +0800, Steven Liu wrote: > > ffmpeg need a dash demuxer for demux the dash formats > > base on https://github.com/samsamsam-iptvplayer/exteplayer3/blob/ > master/tmp/ffmpeg/patches/3.2.2/01_add_dash_demux.patch > >

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

2017-03-19 Thread Clément Bœsch
On Sun, Mar 19, 2017 at 05:05:55PM +0800, Steven Liu wrote: > ffmpeg need a dash demuxer for demux the dash formats > base on > https://github.com/samsamsam-iptvplayer/exteplayer3/blob/master/tmp/ffmpeg/patches/3.2.2/01_add_dash_demux.patch > > Signed-off-by: Steven Liu > --- > configure

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

2017-03-19 Thread Steven Liu
2017-03-19 17:34 GMT+08:00 Carl Eugen Hoyos : > 2017-03-19 10:05 GMT+01:00 Steven Liu : > > > + --disable-xml2 disable XML parsing using the C library > libxml2 [autodetect] > > I believe several developers have argued in the past that they are against > adding libxml2 as an external de

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

2017-03-19 Thread Carl Eugen Hoyos
2017-03-19 10:05 GMT+01:00 Steven Liu : > + --disable-xml2 disable XML parsing using the C library libxml2 > [autodetect] I believe several developers have argued in the past that they are against adding libxml2 as an external dependency to FFmpeg. Instead of resending this (or a simi

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

2017-03-19 Thread Steven Liu
ffmpeg need a dash demuxer for demux the dash formats base on https://github.com/samsamsam-iptvplayer/exteplayer3/blob/master/tmp/ffmpeg/patches/3.2.2/01_add_dash_demux.patch Signed-off-by: Steven Liu --- configure|4 + libavformat/Makefile |1 + libavformat/allf