Re: [FFmpeg-devel] TR-03 implementation

2016-11-12 Thread Carlos Fernandez Sanz
On Sat, Nov 12, 2016 at 8:17 PM, Ali KIZIL wrote: > If someone wants to try, he can give a try, release a patch, it can be > discussed in a discussion level. I really find saying such comments like " > We don't need more broadcasting shit in ffmpeg." and "and it is why > FFmpeg is full of bro

Re: [FFmpeg-devel] [PATCH 3/4] V14 - SCTE-35 support in hlsenc

2016-10-31 Thread Carlos Fernandez Sanz
Hi, Just a ping in case this can be revised and/or applied soonish - thanks. CC'ing Carl Eugen and Thilo since I could discuss this a bit in person at the GSoC summit. Thanks both for your time by the way. Carlos On Tue, Oct 18, 2016 at 5:36 PM, Carlos Fernandez Sanz wrote: > From

Re: [FFmpeg-devel] [PATCH 1/4] V14 - Adding SCTE-35 CUI codec

2016-10-22 Thread Carlos Fernandez Sanz
On Sat, Oct 22, 2016 at 4:15 AM, Kieran Kunhya wrote: > > Getting patches merged by sheer attrition is a shameful way of "running" an > open source project. I think ignoring follow-up questions from the patch submitter, like you are doing with mine, it's plain rude. Basically your position is "S

Re: [FFmpeg-devel] [PATCH 1/4] V14 - Adding SCTE-35 CUI codec

2016-10-21 Thread Carlos Fernandez Sanz
On Fri, Oct 21, 2016 at 5:17 PM, Kieran Kunhya wrote: > > So all the objections to this patchset are now irrelevant are they? Just curious - what are the objections to this patchset and where were they raised? I don't think there's _anything_ posted in this mailing list that hasn't been corrected

Re: [FFmpeg-devel] [PATCH 3/4] V13 - SCTE-35 support in hlsenc

2016-10-19 Thread Carlos Fernandez Sanz
On Sun, Oct 16, 2016 at 10:59 AM, Carlos Fernandez Sanz wrote: > > All other things corrected (I think) - will send a patch soon. Thanks! I submitted V14 yesterday - can you take a look and comment and/or merge? Thanks ___ ffmpeg-devel mailin

[FFmpeg-devel] [PATCH 3/4] V14 - SCTE-35 support in hlsenc

2016-10-18 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 104 -- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 697 insertions(+), 22 deletio

[FFmpeg-devel] [PATCH 1/4] V14 - Adding SCTE-35 CUI codec

2016-10-18 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 ++ 2 files changed, 7 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index d72ee07..e68dd93 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avco

[FFmpeg-devel] [PATCH 0/4] V14 - SCTE-35 support

2016-10-18 Thread Carlos Fernandez Sanz
- Addresses all things mentioned by Anssi Hannula about HLS Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 + libavformat/Makefile| 2 +- liba

[FFmpeg-devel] [PATCH 4/4] V14 - Correct Indentation

2016-10-18 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 01e3237..8d99154 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -391,

[FFmpeg-devel] [PATCH 2/4] V14 - SCTE-35 extraction from mpegts

2016-10-18 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 47 +-- 1 file changed, 45 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..932ffc1 100644 --- a/libavformat/mpegts

Re: [FFmpeg-devel] [PATCH 3/4] V13 - SCTE-35 support in hlsenc

2016-10-16 Thread Carlos Fernandez Sanz
On Sat, Oct 15, 2016 at 1:44 AM, Anssi Hannula wrote: > > Function pointers are not useful here as they always point to the same > functions. Function pointer are kept there so that scte_35.c function are not exposed to users using ffmpeg library. > > Also, the members of this "interface" struct

Re: [FFmpeg-devel] [PATCH 0/4] V12 - SCTE-35 support

2016-10-04 Thread Carlos Fernandez Sanz
On Tue, Oct 4, 2016 at 12:11 PM, Josh de Kock wrote: > Putting SCTE35 support in another library like Upipe was suggested as well. This is something that I don't understand. Should FFmpeg just ignore the existance of this very important standard in the professional video world? Drive users to so

Re: [FFmpeg-devel] [PATCH 2/4] V12 - SCTE-35 extraction from mpegts

2016-10-03 Thread Carlos Fernandez Sanz
On Sat, Oct 1, 2016 at 10:50 AM, Marton Balint wrote: > > Empty line. Thanks, corrected in V13. If all OK, can this patchset be merged? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 3/4] V13 - SCTE-35 support in hlsenc

2016-10-03 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 108 +-- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 701 insertions(+), 22 deleti

[FFmpeg-devel] [PATCH 1/4] V13 - Adding SCTE-35 CUI codec

2016-10-03 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 2 ++ libavcodec/codec_desc.c | 6 ++ 2 files changed, 8 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index d72ee07..9064006 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avc

[FFmpeg-devel] [PATCH 4/4] V13 - Correct Indentation

2016-10-03 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 44259ec..83d6881 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -406,

[FFmpeg-devel] [PATCH 2/4] V13 - SCTE-35 extraction from mpegts

2016-10-03 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 47 +-- 1 file changed, 45 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..932ffc1 100644 --- a/libavformat/mpegts

[FFmpeg-devel] [PATCH 0/4] V13 - SCTE-35

2016-10-03 Thread Carlos Fernandez Sanz
- Same as V12 except for an empty that shouldn't have been there. Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 2 + libavcodec/codec_desc.c | 6 + libavformat/Makefile| 2 +-

[FFmpeg-devel] [PATCH 0/4] V12 - SCTE-35 support

2016-10-01 Thread Carlos Fernandez Sanz
- Addresses new comments such as like idx value checking and filename checking Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 2 + libavcodec/codec_desc.c | 6 + libavformat/Makefil

[FFmpeg-devel] [PATCH 4/4] V12 - Correct Indentation

2016-10-01 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index a3224ef..1aec53f 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -406,

[FFmpeg-devel] [PATCH 3/4] V12 - SCTE-35 support in hlsenc

2016-10-01 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 112 +-- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 705 insertions(+), 22 deleti

[FFmpeg-devel] [PATCH 2/4] V12 - SCTE-35 extraction from mpegts

2016-10-01 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 48 +++- 1 file changed, 47 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..c13d6d8 100644 --- a/libavformat/mpegts

[FFmpeg-devel] [PATCH 1/4] V12 - Adding SCTE-35 CUI codec

2016-10-01 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 2 ++ libavcodec/codec_desc.c | 6 ++ 2 files changed, 8 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index d72ee07..9064006 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avc

Re: [FFmpeg-devel] [PATCH 2/4] V11 - SCTE-35 extraction from mpegts

2016-10-01 Thread Carlos Fernandez Sanz
On Tue, Sep 27, 2016 at 3:13 PM, Marton Balint wrote: > > nothing useful in the header, it might make sense to create a data packet > only from the buffer _after_ the section_length field, skipping the last > CRC32 field as well. I've been looking into this, but it would break some things (like b

Re: [FFmpeg-devel] order T-shirts

2016-09-28 Thread Carlos Fernandez Sanz
On Tue, Sep 27, 2016 at 8:21 PM, Steven Liu wrote: >> > Hi guys, > what about the next step :-D Probably a good idea would be to have some ready for Google's Summer of Code summit... I know I'd love to get one there. ___ ffmpeg-devel mailing list ff

Re: [FFmpeg-devel] [PATCH 2/4] V10 - SCTE-35 extraction from mpegts

2016-09-27 Thread Carlos Fernandez Sanz
On Mon, Sep 26, 2016 at 3:25 PM, Michael Niedermayer wrote: > > this breaks fate: > make: *** [fate-h264-skip-nokey] Error 1 Thanks - solved in V11. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 4/4] V11 - Correct Indentation

2016-09-27 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 3303dc8..39d1fe9 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -373,

[FFmpeg-devel] [PATCH 3/4] V11 - SCTE-35 support in hlsenc

2016-09-27 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 108 +-- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 701 insertions(+), 22 deleti

[FFmpeg-devel] [PATCH 2/4] V11 - SCTE-35 extraction from mpegts

2016-09-27 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 48 ++-- 1 file changed, 46 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..d816450 100644 --- a/libavformat/mpegt

[FFmpeg-devel] [PATCH 1/4] V11 - Adding SCTE-35 CUI codec

2016-09-27 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 2 ++ libavcodec/codec_desc.c | 6 ++ 2 files changed, 8 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index b174116..e3103a1 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avc

[FFmpeg-devel] [PATCH 0/4] V11 - SCTE-35 support

2016-09-27 Thread Carlos Fernandez Sanz
- Addresses all new comments - fate now passes all tests Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 2 + libavcodec/codec_desc.c | 6 + libavformat/Makefile| 2 +- libavfo

Re: [FFmpeg-devel] [PATCH 2/4] V10 - SCTE-35 extraction from mpegts

2016-09-27 Thread Carlos Fernandez Sanz
On Tue, Sep 27, 2016 at 12:30 AM, Marton Balint wrote: >> +} else if (ts->pids[i] && ts->pids[i]->type == >> MPEGTS_SECTION) { >> +return ret; > > > Why do you need this hunk? I think you can delete it, and everything will > remain working. This loop flushes existing h

Re: [FFmpeg-devel] [PATCH 2/4] V9 - SCTE-35 extraction from mpegts

2016-09-26 Thread Carlos Fernandez Sanz
On Sun, Sep 4, 2016 at 9:48 AM, Marton Balint wrote: > > Sorry for the delay. Below some comments. Thanks for that. I've submitted a new patch that hopefully addresses everything. Definitely much cleaner now. Carlos ___ ffmpeg-devel mailing list ffmpeg

[FFmpeg-devel] [PATCH 2/4] V10 - SCTE-35 extraction from mpegts

2016-09-26 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 64 ++-- 1 file changed, 62 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..ad6f141 100644 --- a/libavformat/m

[FFmpeg-devel] [PATCH 4/4] V10 - Correct Indentation

2016-09-26 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index b210fe6..127b2ab 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -373,

[FFmpeg-devel] [PATCH 1/4] V10 - Adding SCTE-35 CUI codec

2016-09-26 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 3 ++- libavcodec/codec_desc.c | 6 ++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index b174116..0eee2ab 100644 --- a/libavcodec/avcodec.h +++

[FFmpeg-devel] [PATCH 3/4] V10 - SCTE-35 support in hlsenc

2016-09-26 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 110 --- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 702 insertions(+), 23 deleti

[FFmpeg-devel] [PATCH 0/4] V10 - SCTE-35

2016-09-26 Thread Carlos Fernandez Sanz
- Addressed all issues (I think) raised by cus. Major refactor from v9 Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 3 +- libavcodec/codec_desc.c | 6 + libavformat/Makefile|

Re: [FFmpeg-devel] [PATCH 3/4] V8 - SCTE-35 support in hlsenc

2016-09-01 Thread Carlos Fernandez Sanz
On Wed, Aug 31, 2016 at 4:10 PM, Steven Liu wrote: > > Overflow here Thanks - corrected in V9 (sent a couple hours ago). Looks OK other than this? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 4/4] V9 - Correct Indentation

2016-09-01 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 5e2fcac..39a6621 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -370,

[FFmpeg-devel] [PATCH 3/4] V9 - SCTE-35 support in hlsenc

2016-09-01 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 110 --- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 702 insertions(+), 23 deleti

[FFmpeg-devel] [PATCH 2/4] V9 - SCTE-35 extraction from mpegts

2016-09-01 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 50 +- 1 file changed, 49 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..d000269 100644 --- a/libavformat/mpeg

[FFmpeg-devel] [PATCH 0/4] V9 - SCTE-35

2016-09-01 Thread Carlos Fernandez Sanz
* Solves overflow reported by Steven Liu Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 3 +- libavcodec/codec_desc.c | 6 + libavformat/Makefile| 2 +- libavformat/hlsenc.c

[FFmpeg-devel] [PATCH 1/4] V9 - Adding SCTE-35 CUI codec

2016-09-01 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 3 ++- libavcodec/codec_desc.c | 6 ++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index e2dad5d..00599c4 100644 --- a/libavcodec/avcodec.h +++

[FFmpeg-devel] [PATCH 0/4] V8 - SCTE-35

2016-08-31 Thread Carlos Fernandez Sanz
* Used PCR for PTS and DTS of SCTE packet Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 3 +- libavcodec/codec_desc.c | 6 + libavformat/Makefile| 2 +- libavformat/hlsenc.c

[FFmpeg-devel] [PATCH 3/4] V8 - SCTE-35 support in hlsenc

2016-08-31 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 110 --- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 702 insertions(+), 23 deleti

[FFmpeg-devel] [PATCH 4/4] V8 - Correct Indentation

2016-08-31 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 25092f7..a78fdae 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -370,

[FFmpeg-devel] [PATCH 1/4] V8 - Adding SCTE-35 CUI codec

2016-08-31 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 3 ++- libavcodec/codec_desc.c | 6 ++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index e2dad5d..00599c4 100644 --- a/libavcodec/avcodec.h +++

[FFmpeg-devel] [PATCH 2/4] V8 - SCTE-35 extraction from mpegts

2016-08-31 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 50 +- 1 file changed, 49 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..d000269 100644 --- a/libavformat/mpeg

Re: [FFmpeg-devel] [PATCH 1/4] V7 - Adding SCTE-35 CUI codec

2016-08-21 Thread Carlos Fernandez Sanz
On Sat, Aug 20, 2016 at 11:47 AM, Michael Niedermayer wrote: > dts are decode timestamps, that is for video thats when a packet OK, something like this? +AV_CODEC_ID_SCTE_35,/**< Contain no valid time stamp in DTS PTS of avpacket, avpacket data contain time stamp +

Re: [FFmpeg-devel] [PATCH 1/4] V7 - Adding SCTE-35 CUI codec

2016-08-20 Thread Carlos Fernandez Sanz
On Fri, Aug 19, 2016 at 11:48 AM, Michael Niedermayer wrote: > can you document in a comment briefly what is in the AVPackets > for most codecs that is clear, SCTE-35 is maybe a bit special > also what the dts/pts values of teh AVPackets mean Really don't know what kind of comments or where to a

[FFmpeg-devel] [PATCH 3/4] V7 - SCTE-35 support in hlsenc

2016-08-19 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 107 -- libavformat/scte_35.c | 527 ++ libavformat/scte_35.h | 86 4 files changed, 700 insertions(+), 22 deletio

[FFmpeg-devel] [PATCH 2/4] V7 - SCTE-35 extraction from mpegts

2016-08-19 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 43 ++- 1 file changed, 42 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..4185af0 100644 --- a/libavformat/mpegts.c ++

[FFmpeg-devel] [PATCH 4/4] V7 - Correct Indentation

2016-08-19 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 7c90157..a73c50d 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -360,

[FFmpeg-devel] [PATCH 1/4] V7 - Adding SCTE-35 CUI codec

2016-08-19 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 ++ 2 files changed, 7 insertions(+) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 6ac6646..6cdb579 100644 --- a/libavcodec/avcodec.h +++ b/libavcodec/avco

[FFmpeg-devel] [PATCH 0/4] v7 - SCTE-35

2016-08-19 Thread Carlos Fernandez Sanz
* Corrects bug that caused crash on file matrixbench_mpeg2.mpg Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 + libavformat/Makefile| 2 +- l

[FFmpeg-devel] [PATCH 2/4] v6 - SCTE-35 extraction from mpegts

2016-08-16 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 43 ++- 1 file changed, 42 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..4185af0 100644 --- a/libavformat/mpegts.c ++

[FFmpeg-devel] [PATCH 4/4] v6 - Correct Indentation

2016-08-16 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 7c90157..a73c50d 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -360,

[FFmpeg-devel] [PATCH 3/4] v6 - SCTE-35 support in hlsenc

2016-08-16 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 107 -- libavformat/scte_35.c | 525 ++ libavformat/scte_35.h | 86 + 4 files changed, 698 insertions(+), 22 deleti

[FFmpeg-devel] [PATCH 0/4] v6 - SCTE-35

2016-08-16 Thread Carlos Fernandez Sanz
* Patchset against current HEAD - v5 didn't build cleanly. Apologies. Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 + libavcodec/version.h|

[FFmpeg-devel] [PATCH 1/4] v6 - Adding SCTE-35 CUI codec

2016-08-16 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 ++ libavcodec/version.h| 2 +- 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index b43ee5a..d564178 100644 -

Re: [FFmpeg-devel] [PATCH 3/4] v5 - SCTE-35 support in hlsenc

2016-08-16 Thread Carlos Fernandez Sanz
On Tue, Aug 16, 2016 at 11:13 AM, Michael Niedermayer wrote: > > in which patch is the function ? Apologies - looks like it's me who had some leftovers and didn't do a proper clean before submitting. I'll send a new patch set right away... just tested with current HEAD. _

Re: [FFmpeg-devel] [PATCH 3/4] v5 - SCTE-35 support in hlsenc

2016-08-16 Thread Carlos Fernandez Sanz
On Tue, Aug 16, 2016 at 6:39 AM, Michael Niedermayer wrote: > On Mon, Aug 15, 2016 at 10:47:49AM -0700, Carlos Fernandez Sanz wrote: >> From: Carlos Fernandez > > seems to break build > > libavformat/scte_35.c: In function ‘ff_alloc_scte35_parser’: Maybe some left overs fr

Re: [FFmpeg-devel] [PATCH 4/4] v5 - Correct Indentation

2016-08-15 Thread Carlos Fernandez Sanz
On Mon, Aug 15, 2016 at 2:33 PM, Steven Liu wrote: > What dose this patch do? Addresses Michael's comment on the previous version: "this reindention is inconsistent, also you can if you like keep reindentions in a seperate patch. But if done they should be consistent" __

[FFmpeg-devel] [PATCH 2/4] v5 - SCTE-35 extraction from mpegts

2016-08-15 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/mpegts.c | 43 ++- 1 file changed, 42 insertions(+), 1 deletion(-) diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c index b31d233..4185af0 100644 --- a/libavformat/mpegts.c ++

[FFmpeg-devel] [PATCH 0/4] v5 - SCTE-35 support

2016-08-15 Thread Carlos Fernandez Sanz
* Corrected all (hopefully) issues pointed out for v4. Carlos Fernandez (4): Adding SCTE-35 CUI codec SCTE-35 extraction from mpegts SCTE-35 support in hlsenc Correct Indentation libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 + libavcodec/version.h| 4 +- libavfo

[FFmpeg-devel] [PATCH 4/4] v5 - Correct Indentation

2016-08-15 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/hlsenc.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index 7c90157..a73c50d 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -360,

[FFmpeg-devel] [PATCH 3/4] v5 - SCTE-35 support in hlsenc

2016-08-15 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 107 -- libavformat/scte_35.c | 525 ++ libavformat/scte_35.h | 86 + 4 files changed, 698 insertions(+), 22 deleti

[FFmpeg-devel] [PATCH 1/4] v5 - Adding SCTE-35 CUI codec

2016-08-15 Thread Carlos Fernandez Sanz
From: Carlos Fernandez Signed-off-by: Carlos Fernandez --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 ++ libavcodec/version.h| 4 ++-- 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index 06c2b89..a6b9a95 10064

[FFmpeg-devel] [PATCH 2/2] v4 - SCTE-35 support in hlsenc

2016-08-08 Thread Carlos Fernandez Sanz
From: carlos Signed-off-by: carlos --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 110 +--- libavformat/scte_35.c | 482 ++ libavformat/scte_35.h | 76 4 files changed, 646 insertions(+), 24 deletions(-) create mode

[FFmpeg-devel] [PATCH 1/2] v4 - SCTE extraction from mpegts

2016-08-08 Thread Carlos Fernandez Sanz
From: Carlos Signed-off-by: carlos --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 ++ libavformat/mpegts.c| 43 ++- 3 files changed, 49 insertions(+), 1 deletion(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h index

[FFmpeg-devel] [PATCH 0/2] v4 - SCTE-35 work

2016-08-08 Thread Carlos Fernandez Sanz
* Corrected reported issue (an unrelated change that made it into the patch) Carlos (1): SCTE extraction from mpegts carlos (1): SCTE-35 support in hlsenc libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 + libavformat/Makefile| 2 +- libavformat/hlsenc.c| 110 ++

[FFmpeg-devel] [PATCH 2/2] v3 - SCTE-35 support in hlsenc

2016-08-03 Thread Carlos Fernandez Sanz
From: carlos Signed-off-by: carlos --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 110 +--- libavformat/scte_35.c | 482 ++ libavformat/scte_35.h | 76 4 files changed, 646 insertions(+), 24 deletions(-) create mode

[FFmpeg-devel] [PATCH 1/2] v3 - SCTE extraction from mpegts

2016-08-03 Thread Carlos Fernandez Sanz
From: Carlos Signed-off-by: carlos --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 ++ libavformat/mpegts.c| 51 +++-- 3 files changed, 52 insertions(+), 6 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h

[FFmpeg-devel] [PATCH 0/2] v3 - SCTE 35 work

2016-08-03 Thread Carlos Fernandez Sanz
- Corrected last reported issue about a sample file missing audio after these patches were applied. - Created new streamtype with SCTE Carlos (1): SCTE extraction from mpegts carlos (1): SCTE-35 support in hlsenc libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 + libavformat/

Re: [FFmpeg-devel] [PATCH 1/2] v2 SCTE extraction from mpegts

2016-07-26 Thread Carlos Fernandez Sanz
On Mon, Jul 25, 2016 at 5:41 PM, Michael Niedermayer wrote: > This breaks > http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket1827/vc1_dts_mplayerAudioProblem_sb41_cut.m2ts I need some help with this. HDMV audio is not detected correctly, since we are giving priority to ISO based stream_type 8

[FFmpeg-devel] [PATCH 2/2] v2 SCTE-35 support in hlsenc

2016-07-25 Thread Carlos Fernandez Sanz
From: carlos Signed-off-by: carlos --- libavformat/Makefile | 2 +- libavformat/hlsenc.c | 110 +--- libavformat/scte_35.c | 482 ++ libavformat/scte_35.h | 76 4 files changed, 646 insertions(+), 24 deletions(-) create mode

[FFmpeg-devel] [PATCH 1/2] v2 SCTE extraction from mpegts

2016-07-25 Thread Carlos Fernandez Sanz
From: Carlos Signed-off-by: carlos --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 ++ libavformat/mpegts.c| 46 -- 3 files changed, 47 insertions(+), 6 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h in

[FFmpeg-devel] [PATCH 0/2] v2 - SCTE-35 support

2016-07-25 Thread Carlos Fernandez Sanz
These two patches add support for SCTE-35. First, parse it correct from MPEG, then segment appropiately. All things reported about v1 addressed: - Removed commented out code - Spaces where there was a tab - Should build on mingw64 (removed probable cause of error) - Made it a hls muxer dependency

[FFmpeg-devel] [PATCH 2/2] SCTE-35 support in hlsenc

2016-07-21 Thread Carlos Fernandez Sanz
From: carlos Signed-off-by: carlos --- libavformat/Makefile | 1 + libavformat/hlsenc.c | 179 +++--- libavformat/scte_35.c | 489 ++ libavformat/scte_35.h | 76 4 files changed, 680 insertions(+), 65 deletions(-) create

[FFmpeg-devel] [PATCH 0/2] SCTE-35 support

2016-07-21 Thread Carlos Fernandez Sanz
These two patches add support for SCTE-35. First, parse it correct from MPEG, then segment appropiately. Carlos (1): SCTE extraction from mpegts carlos (1): SCTE-35 support in hlsenc libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 + libavformat/Makefile| 1 + libavfor

[FFmpeg-devel] [PATCH 1/2] SCTE extraction from mpegts

2016-07-21 Thread Carlos Fernandez Sanz
From: Carlos Signed-off-by: carlos --- libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 6 ++ libavformat/mpegts.c| 46 -- 3 files changed, 47 insertions(+), 6 deletions(-) diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h in

[FFmpeg-devel] Start of SCTE-35 support [PATCH]

2016-07-06 Thread Carlos Fernandez Sanz
Hi, Apologies for resending - gmail sent the first attempt as a binary attachment. This patch starts the work on SCTE-35 support; for now it just detects SCTE-35 and if present sends the data to a dummy function. Since it's my first contribution to ffmpeg I preferred to start small and continue w

[FFmpeg-devel] Start of SCTE-35 support [PATCH]

2016-07-06 Thread Carlos Fernandez Sanz
Hi, This patch starts the work on SCTE-35 support; for now it just detects SCTE-35 and if present sends the data to a dummy function. Since it's my first contribution to ffmpeg I preferred to start small and continue working as small pieces of work are accepted or commented. Carlos 0001-SCTE-ex