Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-05-01 Thread Gorilla Maguila
We will upload a new patch with some fixes. 2015-05-01 4:41 GMT+02:00 Michael Niedermayer : > On Thu, Apr 30, 2015 at 02:37:39PM +0200, Gorilla Maguila wrote: > > New patch with some fixes: > > > > - Corrected style and formatting. > > - No ugly casts. > > - New hds_probe function > > - No forwar

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-30 Thread Michael Niedermayer
On Thu, Apr 30, 2015 at 02:37:39PM +0200, Gorilla Maguila wrote: > New patch with some fixes: > > - Corrected style and formatting. > - No ugly casts. > - New hds_probe function > - No forward declarations in f4fbox.c. I couldn't get rid of the forward > declaration in amfmetadata.c due to circula

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-30 Thread Gorilla Maguila
New patch with some fixes: - Corrected style and formatting. - No ugly casts. - New hds_probe function - No forward declarations in f4fbox.c. I couldn't get rid of the forward declaration in amfmetadata.c due to circular dependencies (Ideas welcome) - Other minor fixes. TODO: - Fragment caching

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-28 Thread Gorilla Maguila
I can't think of a better way of avoiding forward declaration due to the nested nature of the F4F Boxes and AMF data. 2015-04-28 15:32 GMT+02:00 Clément Bœsch : > > +static int f4fbox_parse_single_box(AVIOContext *in, void *opaque); > > Again, isn't this avoidable? > +static int amf_metadata_

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-28 Thread Gorilla Maguila
I will try to fix all the problems, although I'm not the original author of most of the code, I just added better fragments/segment calculation for live streams, and other small changes. 2015-04-28 15:32 GMT+02:00 Clément Bœsch : > On Tue, Apr 28, 2015 at 02:48:42PM +0200, Gorilla Maguila wrote:

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-28 Thread Clément Bœsch
On Tue, Apr 28, 2015 at 02:48:42PM +0200, Gorilla Maguila wrote: [...] > From 5bb2e85f2e7c4dfeb3569225e263ddc6a4f127cd Mon Sep 17 00:00:00 2001 > From: Developer Mobdro > Date: Tue, 28 Apr 2015 14:38:35 +0200 > Subject: [PATCH] hds demuxer > > --- > configure | 8 + > libavform

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-28 Thread Gorilla Maguila
I will work on it and I will upload a new patch soon. 2015-04-28 15:00 GMT+02:00 Carl Eugen Hoyos : > Gorilla Maguila gmail.com> writes: > > > +static int hds_probe(AVProbeData *p) > > +{ > > +if(p->filename && av_stristr(p->filename, ".f4m")) > > +return AVPROBE_SCORE_MAX; > > +

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-28 Thread wm4
On Tue, 28 Apr 2015 14:48:42 +0200 Gorilla Maguila wrote: > From 5bb2e85f2e7c4dfeb3569225e263ddc6a4f127cd Mon Sep 17 00:00:00 2001 > From: Developer Mobdro > Date: Tue, 28 Apr 2015 14:38:35 +0200 > Subject: [PATCH] hds demuxer > > --- > +mdat->data = av_mallocz(sizeof(uint8_t)*data_size);

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-28 Thread Carl Eugen Hoyos
Gorilla Maguila gmail.com> writes: > +static int hds_probe(AVProbeData *p) > +{ > +if(p->filename && av_stristr(p->filename, ".f4m")) > +return AVPROBE_SCORE_MAX; > +return 0; > +} Remove this function, instead add ".f4m" as .extentions to the AVInputFormat. Carl Eugen ___

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-28 Thread Carl Eugen Hoyos
Gorilla Maguila gmail.com> writes: > New patch with some fixes: Thank you for working on this! > +if(type != AMF_DATA_TYPE_STRING) { Please add a space after "if", same below. > +av_log(NULL, AV_LOG_ERROR, "amfmetadata Expected type 2 Please pass the context to this function and

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2015-04-28 Thread Gorilla Maguila
New patch with some fixes: - use of av_freep - use of context in av_log when possible - corrected else-if format - improved fragment/segment calculation for live streams 2014-09-18 12:01 GMT+02:00 Gorilla Maguila : > I'll try to correct the patch and post it back. > > 2014-09-17 19:34 GMT+02:00

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2014-09-18 Thread Gorilla Maguila
I'll try to correct the patch and post it back. 2014-09-17 19:34 GMT+02:00 Carl Eugen Hoyos : > Gorilla Maguila gmail.com> writes: > > > http://lists.ffmpeg.org/pipermail/ffmpeg-devel/2014-February/154612.html > > > > I attach a new patch with improvements: > > I wanted to write an answer with a

Re: [FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2014-09-17 Thread Carl Eugen Hoyos
Gorilla Maguila gmail.com> writes: > http://lists.ffmpeg.org/pipermail/ffmpeg-devel/2014-February/154612.html > > I attach a new patch with improvements: I wanted to write an answer with a few minor remarks but just saw that several issues from the original review are still valid: http://thre

[FFmpeg-devel] [PATCH] Adobe HTTP Dynamic Streaming (HDS) demuxer improvements

2014-09-17 Thread Gorilla Maguila
After contacting the original author of the the patch posted in: http://lists.ffmpeg.org/pipermail/ffmpeg-devel/2014-February/154612.html I attach a new patch with improvements: - Better codec detection in amfmetata - Better live streams handling (Get current segment, calculate next segment..etc