[FFmpeg-devel] [PATCH 4/4] ffprobe: report field order for video streams

2016-10-03 Thread Rodger Combs
--- ffprobe.c | 13 + tests/ref/fate/concat-demuxer-extended-lavf-mxf | 2 +- tests/ref/fate/concat-demuxer-extended-lavf-mxf_d10 | 2 +- tests/ref/fate/concat-demuxer-simple1-lavf-mxf | 2 +-

[FFmpeg-devel] [PATCH] libavformat/utils: ignore outlier subtitle and data stream end times as well

2016-10-03 Thread Rodger Combs
--- libavformat/utils.c | 15 --- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index 3acb260..ce68408 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -2524,7 +2524,7 @@ static int has_duration(AVFormatContext

[FFmpeg-devel] [PATCH] lavf/segment: provide a virtual AVIOContext representing all the segments

2016-09-29 Thread Rodger Combs
This allows the use of muxers like matroska, which attempt to seek even when an AVIOContext doesn't set `seekable`, without concern for a rouge seek leading the muxer to overwrite the wrong data in a later segment. --- doc/muxers.texi | 17 libavformat/segment.c | 276

[FFmpeg-devel] [PATCH] lavf/mpegtsenc: fix autobsf when the first NAL is 0x1 bytes

2016-09-25 Thread Rodger Combs
--- libavformat/mpegtsenc.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index fd849e5..ac27b81 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -1790,11 +1790,15 @@ static int

Re: [FFmpeg-devel] [PATCH] lavc/parser: export field order if not already set

2016-09-20 Thread Rodger Combs
> On Sep 20, 2016, at 10:18, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > > 2016-09-20 13:13 GMT+02:00 Rodger Combs <rodger.co...@gmail.com>: >> Some codecs set this in the parser, but not the decoder > >> +#define FILL(name) if(s->name > 0

[FFmpeg-devel] [PATCH] lavc/parser: export field order if not already set

2016-09-20 Thread Rodger Combs
Some codecs set this in the parser, but not the decoder --- libavcodec/parser.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/parser.c b/libavcodec/parser.c index 2c8fc69..30cfc55 100644 --- a/libavcodec/parser.c +++ b/libavcodec/parser.c @@ -182,6 +182,11 @@ int

[FFmpeg-devel] [PATCH] lavf/matroska: expose stream encoding as global side-data

2016-09-12 Thread Rodger Combs
Some demuxers can't handle Matroska compression, so this lets API users check if a file uses it and determine whether those players will fail. Currently only exports the first encoding; the format technically supports multiple, but I can't find an example of this and lavf doesn't support it. ---

Re: [FFmpeg-devel] [PATCH 1/2] lavf/mov: fix parsing QuickTime meta after the first track

2016-09-12 Thread Rodger Combs
> On Sep 10, 2016, at 05:10, Michael Niedermayer <mich...@niedermayer.cc> wrote: > > On Fri, Sep 09, 2016 at 08:06:39PM -0500, Rodger Combs wrote: >> I'm not entirely sure why found_hdlr_mdta existed to begin with, so cc-ing >> Tinglin Liu (who originally wrote the

Re: [FFmpeg-devel] [PATCH 2/2] lavf/mov: strip com.apple.quicktime prefix in meta; parse creation date

2016-09-12 Thread Rodger Combs
> On Sep 10, 2016, at 05:55, Michael Niedermayer <mich...@niedermayer.cc> wrote: > > On Fri, Sep 09, 2016 at 08:06:40PM -0500, Rodger Combs wrote: >> --- >> libavformat/mov.c | 16 >> 1 file changed, 16 insertions(+) >> >> diff

Re: [FFmpeg-devel] [PATCH 2/3] lavf/mov: improve `tref/chap` chapter handling

2016-09-12 Thread Rodger Combs
> On Sep 10, 2016, at 20:41, Michael Niedermayer <mich...@niedermayer.cc> wrote: > > On Fri, Sep 09, 2016 at 09:07:31PM -0500, Rodger Combs wrote: >> 3 parts: >> - Supports multiple chapter streams >> - Exports regular text chapter streams as opaque data. Thi

[FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-10 Thread Rodger Combs
--- ffmpeg.c | 4 +--- ffmpeg_opt.c | 3 +++ 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index d858407..111d844 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -366,7 +366,7 @@ static BOOL WINAPI CtrlHandler(DWORD fdwCtrlType) void term_init(void) { #if

[FFmpeg-devel] [PATCH 2/2] tests: add -nostdin flag when calling ffmpeg

2016-09-10 Thread Rodger Combs
This fixes a long-standing issue where running FATE in parallel could result in the terminal being left misconfigured, particularly if a test failed or was canceled wtih ^C. --- tests/fate/vpx.mak| 10 +- tests/regression-funcs.sh | 2 +- 2 files changed, 6 insertions(+), 6

[FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-10 Thread Rodger Combs
--- ffmpeg.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c b/ffmpeg.c index d858407..08a7a3d 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -366,7 +366,7 @@ static BOOL WINAPI CtrlHandler(DWORD fdwCtrlType) void term_init(void) { #if HAVE_TERMIOS_H -

[FFmpeg-devel] [PATCH 2/2] tests: add -nostdin flag when calling ffmpeg

2016-09-10 Thread Rodger Combs
This fixes a long-standing issue where running FATE in parallel could result in the terminal being left misconfigured, particularly if a test failed or was canceled wtih ^C. --- tests/fate/vpx.mak| 10 +- tests/regression-funcs.sh | 2 +- 2 files changed, 6 insertions(+), 6

[FFmpeg-devel] [PATCH 3/3] lavc/webvttenc: XML-escape text output

2016-09-10 Thread Rodger Combs
The test output was previously incorrect, so the reference is updated. --- libavcodec/webvttenc.c | 7 ++- tests/ref/fate/sub-webvttenc | 20 ++-- 2 files changed, 16 insertions(+), 11 deletions(-) diff --git a/libavcodec/webvttenc.c b/libavcodec/webvttenc.c index

[FFmpeg-devel] [PATCH 2/3] ffprobe: switch xml_escape_str to use av_bprint_escape

2016-09-10 Thread Rodger Combs
This now escapes single-quotes, so the test reference is updated. --- ffprobe.c | 25 ++--- tests/ref/fate/ffprobe_xml | 2 +- 2 files changed, 7 insertions(+), 20 deletions(-) diff --git a/ffprobe.c b/ffprobe.c index 657867d..70f2ce6 100644 --- a/ffprobe.c

[FFmpeg-devel] [PATCH 1/3] lavu/bprint: add XML escaping

2016-09-10 Thread Rodger Combs
--- libavutil/avstring.h | 28 libavutil/bprint.c | 43 +++ 2 files changed, 71 insertions(+) diff --git a/libavutil/avstring.h b/libavutil/avstring.h index dd28769..8e97314 100644 --- a/libavutil/avstring.h +++

[FFmpeg-devel] [PATCH] lavc/ass_split: improve handling of complex ASS features

2016-09-10 Thread Rodger Combs
Specifically: - Skip writing drawings as text - Convert \h to a Unicode non-breaking space - Ignore comments and unknown tags The test references are update to reflect these changes. --- libavcodec/ass_split.c | 29 + tests/ref/fate/sub-textenc | 12

[FFmpeg-devel] [PATCH 1/2] ass_split: fix handling of streams with no [Events] or Format: line

2016-09-09 Thread Rodger Combs
--- libavcodec/ass_split.c | 36 +++- 1 file changed, 27 insertions(+), 9 deletions(-) diff --git a/libavcodec/ass_split.c b/libavcodec/ass_split.c index beaba7e..cdb1aa2 100644 --- a/libavcodec/ass_split.c +++ b/libavcodec/ass_split.c @@ -229,7 +229,7 @@ static

[FFmpeg-devel] [PATCH 2/2] ass_split: reindent

2016-09-09 Thread Rodger Combs
--- libavcodec/ass_split.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/libavcodec/ass_split.c b/libavcodec/ass_split.c index cdb1aa2..d3e8a8d 100644 --- a/libavcodec/ass_split.c +++ b/libavcodec/ass_split.c @@ -276,23 +276,23 @@ static

[FFmpeg-devel] [PATCH 09/11] fate/h264: make mp4toannexb test use auto-BSF

2016-09-09 Thread Rodger Combs
--- tests/fate/h264.mak | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/fate/h264.mak b/tests/fate/h264.mak index 472895f..b4d7f7a 100644 --- a/tests/fate/h264.mak +++ b/tests/fate/h264.mak @@ -205,7 +205,7 @@ FATE_H264-$(call DEMDEC, MOV, H264) +=

[FFmpeg-devel] [PATCH 10/11] fate/aac: add automatic bsf test

2016-09-09 Thread Rodger Combs
--- tests/fate/aac.mak | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index 3d64031..b02b768 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -241,6 +241,10 @@ FATE_AAC_LATM += fate-aac-latm_stereo_to_51

[FFmpeg-devel] [PATCH 11/11] fate/hevc: add automatic bsf test

2016-09-09 Thread Rodger Combs
--- tests/fate/hevc.mak | 11 +++ 1 file changed, 11 insertions(+) diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index 05266cd..bd09ab3 100644 --- a/tests/fate/hevc.mak +++ b/tests/fate/hevc.mak @@ -225,6 +225,17 @@ $(foreach N,$(HEVC_SAMPLES_444_12BIT),$(eval $(call

[FFmpeg-devel] [PATCH 08/11] lavf/rawenc: add automatic bitstream filtering for H264+HEVC

2016-09-09 Thread Rodger Combs
--- libavformat/rawenc.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/libavformat/rawenc.c b/libavformat/rawenc.c index c4d7a90..730e99a 100644 --- a/libavformat/rawenc.c +++ b/libavformat/rawenc.c @@ -20,8 +20,11 @@ * Foundation, Inc., 51 Franklin Street, Fifth

[FFmpeg-devel] [PATCH 07/11] lavf/movenc+dashenc: add automatic bitstream filtering

2016-09-09 Thread Rodger Combs
This is disabled by default when the empty_moov flag is enabled --- libavformat/dashenc.c | 43 +++- libavformat/movenc.c | 107 +++--- 2 files changed, 124 insertions(+), 26 deletions(-) diff --git a/libavformat/dashenc.c

[FFmpeg-devel] [PATCH 06/11] lavf/dashenc: add deinit function

2016-09-09 Thread Rodger Combs
--- libavformat/dashenc.c | 51 +-- 1 file changed, 17 insertions(+), 34 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 519f9c4..0848052 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -580,16

[FFmpeg-devel] [PATCH 05/11] lavf/movenc: add deinit function

2016-09-09 Thread Rodger Combs
--- libavformat/movenc.c | 76 ++-- 1 file changed, 32 insertions(+), 44 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 319ff57..1fe1eb6 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5567,21 +5567,18

[FFmpeg-devel] [PATCH 04/11] lavf/segment: fix writing separate header with auto BSF

2016-09-09 Thread Rodger Combs
--- libavformat/segment.c | 29 - 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 139761f..7e56f82 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -89,6 +89,7 @@ typedef struct

[FFmpeg-devel] [PATCH 02/11] lavf/mux: add avformat_init_output

2016-09-09 Thread Rodger Combs
This allows a consumer to run the muxer's init function without actually writing the header, which is useful in chained muxers that support automatic bitstream filtering. --- libavformat/avformat.h | 34 +++-- libavformat/internal.h | 10 libavformat/mux.c | 68

[FFmpeg-devel] [PATCH 03/11] lavf/segment: add deinit function

2016-09-09 Thread Rodger Combs
--- libavformat/segment.c | 47 --- 1 file changed, 20 insertions(+), 27 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 33a5cf0..139761f 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -627,8 +627,9 @@

[FFmpeg-devel] [PATCH 01/11] lavf: add a flag to enable/disable automatic bitstream filtering

2016-09-09 Thread Rodger Combs
This is mostly useful for muxers that wrap other muxers, such as dashenc and segment. The actual duplicated bitstream filtering is largely harmless, but delaying the header can cause problems when the muxer intended the header to be written to a separate file. --- libavformat/avformat.h | 1

[FFmpeg-devel] [PATCH] lavf/matroska: expose stream encoding as global side-data

2016-09-09 Thread Rodger Combs
Some demuxers can't handle Matroska compression, so this lets API users check if a file uses it and determine whether those players will fail. --- libavcodec/avcodec.h | 35 ++- libavcodec/avpacket.c | 1 + libavcodec/version.h | 4 ++--

[FFmpeg-devel] [PATCH 2/3] lavf/mov: improve `tref/chap` chapter handling

2016-09-09 Thread Rodger Combs
3 parts: - Supports multiple chapter streams - Exports regular text chapter streams as opaque data. This prevents consumers from showing chapters as if they were regular subtitle streams. - Exports video chapter streams as thumbnails, and provides the first one as an attached_pic. ---

[FFmpeg-devel] [PATCH 1/3] lavf: add AV_DISPOSITION_TIMED_THUMBNAILS

2016-09-09 Thread Rodger Combs
--- ffprobe.c | 1 + libavformat/avformat.h | 12 +--- tests/ref/fate/concat-demuxer-extended-lavf-mxf | 2 +- tests/ref/fate/concat-demuxer-extended-lavf-mxf_d10 | 2 +-

[FFmpeg-devel] [PATCH 3/3] lavf/mov: reindent

2016-09-09 Thread Rodger Combs
--- libavformat/mov.c | 158 +++--- 1 file changed, 79 insertions(+), 79 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 22ca809..b387311 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -4655,94 +4655,94 @@ static

[FFmpeg-devel] [PATCH] lavf/mov: strip com.apple.quicktime prefix in meta; parse creation date

2016-09-09 Thread Rodger Combs
--- libavformat/mov.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 54530e3..103927b 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -42,6 +42,7 @@ #include "libavutil/aes_ctr.h" #include "libavutil/sha.h"

[FFmpeg-devel] [PATCH 1/2] lavf/mov: fix parsing QuickTime meta after the first track

2016-09-09 Thread Rodger Combs
I'm not entirely sure why found_hdlr_mdta existed to begin with, so cc-ing Tinglin Liu (who originally wrote the patch) and Derek Buitenhuis (who signed off on it) hoping for some background. If these checks actually do have a purpose, then the `type == MKTAG('m','d','t','a')` check should be

[FFmpeg-devel] [PATCH 2/2] lavf/mov: strip com.apple.quicktime prefix in meta; parse creation date

2016-09-09 Thread Rodger Combs
--- libavformat/mov.c | 16 1 file changed, 16 insertions(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index 54530e3..b75acd2 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -42,6 +42,7 @@ #include "libavutil/aes_ctr.h" #include "libavutil/sha.h" #include

[FFmpeg-devel] [PATCH] configure: add linker export script support on Darwin

2016-09-09 Thread Rodger Combs
This isn't a "version script" in the usual sense, since it doesn't set symbol versions directly. Instead, the version for the whole .dylib is set in the linker flags, and we generate a list of symbol patterns to export. This allows us to keep our local symbols (e.g. ff_*) local on the platform.

[FFmpeg-devel] [PATCH 1/3] lavc/Makefile: g729dec: fix missing file

2016-09-09 Thread Rodger Combs
--- libavcodec/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 7396468..fa2318a 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -297,7 +297,7 @@ OBJS-$(CONFIG_G723_1_DECODER) += g723_1dec.o

[FFmpeg-devel] [PATCH 2/3] lavc/Makefile: add missing ADPCM_THP_LE objs

2016-09-09 Thread Rodger Combs
--- libavcodec/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index fa2318a..9c7302a 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -753,6 +753,7 @@ OBJS-$(CONFIG_ADPCM_SBPRO_4_DECODER) += adpcm.o adpcm_data.o

[FFmpeg-devel] [PATCH 3/3] lavc/wmv2dec: add #if around AVCodec def

2016-09-09 Thread Rodger Combs
--- libavcodec/wmv2dec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/wmv2dec.c b/libavcodec/wmv2dec.c index 20dbee5..477196e 100644 --- a/libavcodec/wmv2dec.c +++ b/libavcodec/wmv2dec.c @@ -455,6 +455,7 @@ int ff_wmv2_decode_mb(MpegEncContext *s, int16_t block[6][64])

Re: [FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-08 Thread Rodger Combs
> On Sep 8, 2016, at 07:49, Nicolas George <geo...@nsup.org> wrote: > > Le duodi 22 fructidor, an CCXXIV, Rodger Combs a écrit : >> --- >> ffmpeg.c | 6 +- >> 1 file changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/ffmpeg.

[FFmpeg-devel] [PATCH 1/2] ffmpeg: don't reconfigure terminal if we're not taking input from stdin

2016-09-07 Thread Rodger Combs
--- ffmpeg.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ffmpeg.c b/ffmpeg.c index d858407..1d793fe 100644 --- a/ffmpeg.c +++ b/ffmpeg.c @@ -366,7 +366,7 @@ static BOOL WINAPI CtrlHandler(DWORD fdwCtrlType) void term_init(void) { #if HAVE_TERMIOS_H -

[FFmpeg-devel] [PATCH 2/2] tests: add -nostdin flag when calling ffmpeg

2016-09-07 Thread Rodger Combs
This fixes a long-standing issue where running FATE in parallel could result in the terminal being left misconfigured, particularly if a test failed or was canceled wtih ^C. --- tests/fate/vpx.mak| 10 +- tests/regression-funcs.sh | 2 +- 2 files changed, 6 insertions(+), 6

[FFmpeg-devel] [PATCH] lavf/matroskaenc: use mkv_check_tag_name consistently

2016-09-06 Thread Rodger Combs
Previously, we used a different list of checks when deciding whether to write a set of tags at all than we did when deciding whether to write an individual tag in the set. This resulted in sometimes writing an empty tag master and seekhead. Now we use mkv_check_tag_name everywhere, so if a

Re: [FFmpeg-devel] [PATCH 2/3] lavf/matroskaenc: skip writing "duration" tags

2016-09-06 Thread Rodger Combs
> On Sep 6, 2016, at 05:10, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: > > 2016-09-06 5:26 GMT+02:00 Rodger Combs <rodger.co...@gmail.com>: >> --- >> libavformat/matroskaenc.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/li

Re: [FFmpeg-devel] [PATCH] lavc/audiotoolboxdec: fix OSX SDK detection

2016-09-05 Thread Rodger Combs
add eac3 decoder') > Cc: Rodger Combs <rodger.co...@gmail.com> > Signed-off-by: Dmitry Kalinkin <dmitry.kalin...@gmail.com> > --- > libavcodec/audiotoolboxdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/audiotoolboxdec.

Re: [FFmpeg-devel] [PATCH] lavc/audiotoolboxdec: fix OSX SDK detection

2016-09-05 Thread Rodger Combs
> On Sep 5, 2016, at 23:08, Dmitry Kalinkin <dmitry.kalin...@gmail.com> wrote: > > >> On 06 Sep 2016, at 00:05, Rodger Combs <rodger.co...@gmail.com> wrote: >> >> >>> On Sep 5, 2016, at 22:58, Dmitry Kalinkin <dmitry.kalin...@gmail.com>

[FFmpeg-devel] [PATCH 2/3] lavf/matroskaenc: skip writing "duration" tags

2016-09-05 Thread Rodger Combs
--- libavformat/matroskaenc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index decb66d..7deccaa 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -1314,6 +1314,7 @@ static int mkv_check_tag_name(const char

[FFmpeg-devel] [PATCH 1/3] lavf/matroskaenc: move skipped metadata keys to separate function

2016-09-05 Thread Rodger Combs
--- libavformat/matroskaenc.c | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 2a2877f..decb66d 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@ -1308,6 +1308,16 @@ static

[FFmpeg-devel] [PATCH 3/3] lavf/matroskaenc: use mkv_check_tag_name consistently; update test

2016-09-05 Thread Rodger Combs
--- libavformat/matroskaenc.c | 10 +- tests/fate/matroska.mak | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/matroskaenc.c b/libavformat/matroskaenc.c index 7deccaa..3eeb09b 100644 --- a/libavformat/matroskaenc.c +++ b/libavformat/matroskaenc.c @@

[FFmpeg-devel] [PATCH] lavf/matroskaenc: improve tag-skipping checks

2016-08-31 Thread Rodger Combs
- Add "duration" to list of skipped attributes (we handle that elsewhere) - Move the list to its own function and make its use consistent This fixes cases where we'd have a string in one list and not another, which could result in us writing an empty tag. --- libavformat/matroskaenc.c | 28

[FFmpeg-devel] [PATCH 4/5] lavf/segment: copy stream dispositions in output

2016-07-04 Thread Rodger Combs
--- libavformat/segment.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/segment.c b/libavformat/segment.c index 9c766a5..5aec018 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -184,6 +184,7 @@ static int segment_mux_init(AVFormatContext *s) }

[FFmpeg-devel] [PATCH 5/5] lavf/segment: write attached pictures to all segments by default

2016-07-04 Thread Rodger Combs
--- doc/muxers.texi | 4 libavformat/segment.c | 33 + libavformat/version.h | 2 +- 3 files changed, 38 insertions(+), 1 deletion(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index adf853e..fe95cc6 100644 --- a/doc/muxers.texi +++

[FFmpeg-devel] [PATCH 2/5] lavf/flacenc: support writing attached pictures

2016-07-04 Thread Rodger Combs
--- libavformat/flacenc.c | 253 +++--- 1 file changed, 218 insertions(+), 35 deletions(-) diff --git a/libavformat/flacenc.c b/libavformat/flacenc.c index 89b21e9..b7b3016 100644 --- a/libavformat/flacenc.c +++ b/libavformat/flacenc.c @@ -21,10 +21,13

[FFmpeg-devel] [PATCH 3/5] lavf/segment: add option to segment by chapter

2016-07-04 Thread Rodger Combs
--- doc/muxers.texi | 6 + libavformat/segment.c | 65 +++ libavformat/version.h | 2 +- 3 files changed, 67 insertions(+), 6 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index c2ca0ba..adf853e 100644 ---

[FFmpeg-devel] [PATCH 1/5] lavf: add cue sheet demuxer

2016-07-04 Thread Rodger Combs
02110-1301 USA + */ + +/** + * @file + * Cue sheet demuxer + * @author Rodger Combs <rodger.co...@gmail.com> + */ + +#include "avformat.h" +#include "internal.h" +#include "subtitles.h" +#include "url.h" +#include "libavutil/intreadwrite.h" +#i

[FFmpeg-devel] [PATCH 02/10] lavf/mux: add avformat_init_output

2016-06-25 Thread Rodger Combs
This allows a consumer to run the muxer's init function without actually writing the header, which is useful in chained muxers that support automatic bitstream filtering. --- libavformat/avformat.h | 34 +-- libavformat/internal.h | 10 libavformat/mux.c | 64

[FFmpeg-devel] [PATCH 06/10] lavf/dashenc: add deinit function

2016-06-25 Thread Rodger Combs
--- libavformat/dashenc.c | 51 +-- 1 file changed, 17 insertions(+), 34 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 519f9c4..0848052 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -580,16

[FFmpeg-devel] [PATCH 07/10] lavf/movenc+dashenc: add automatic bitstream filtering

2016-06-25 Thread Rodger Combs
This is disabled by default when the empty_moov flag is enabled --- libavformat/dashenc.c | 43 +++- libavformat/movenc.c | 107 +++--- 2 files changed, 124 insertions(+), 26 deletions(-) diff --git a/libavformat/dashenc.c

[FFmpeg-devel] [PATCH 03/10] lavf/segment: add deinit function

2016-06-25 Thread Rodger Combs
--- libavformat/segment.c | 47 --- 1 file changed, 20 insertions(+), 27 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 4c6c6d4..d22d550 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -627,8 +627,9 @@

[FFmpeg-devel] [PATCH 04/10] lavf/segment: fix writing separate header with auto BSF

2016-06-25 Thread Rodger Combs
--- libavformat/segment.c | 29 - 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index d22d550..d8877f0 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -89,6 +89,7 @@ typedef struct

[FFmpeg-devel] [PATCH 08/10] fate/h264: add automatic bsf test

2016-06-25 Thread Rodger Combs
--- tests/fate/h264.mak | 2 ++ tests/ref/fate/h264-autobsf-mp4toannexb | 1 + 2 files changed, 3 insertions(+) create mode 100644 tests/ref/fate/h264-autobsf-mp4toannexb diff --git a/tests/fate/h264.mak b/tests/fate/h264.mak index 4ae1305..e0a5446 100644 ---

[FFmpeg-devel] [PATCH 09/10] fate/aac: add automatic bsf test

2016-06-25 Thread Rodger Combs
--- tests/fate/aac.mak | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index 3d64031..b02b768 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -241,6 +241,10 @@ FATE_AAC_LATM += fate-aac-latm_stereo_to_51

[FFmpeg-devel] [PATCH 01/10] lavf: add a flag to enable/disable automatic bitstream filtering

2016-06-25 Thread Rodger Combs
This is mostly useful for muxers that wrap other muxers, such as dashenc and segment. The actual duplicated bitstream filtering is largely harmless, but delaying the header can cause problems when the muxer intended the header to be written to a separate file. --- libavformat/avformat.h | 1

[FFmpeg-devel] [PATCH 10/10] fate/hevc: add automatic bsf test

2016-06-25 Thread Rodger Combs
--- tests/fate/hevc.mak | 11 +++ 1 file changed, 11 insertions(+) diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index 05266cd..fa63afd 100644 --- a/tests/fate/hevc.mak +++ b/tests/fate/hevc.mak @@ -225,6 +225,17 @@ $(foreach N,$(HEVC_SAMPLES_444_12BIT),$(eval $(call

[FFmpeg-devel] [PATCH 05/10] lavf/movenc: add deinit function

2016-06-25 Thread Rodger Combs
--- libavformat/movenc.c | 70 ++-- 1 file changed, 30 insertions(+), 40 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index d614933..9668b9c 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5475,21 +5475,18

Re: [FFmpeg-devel] [PATCH 04/14] lavf/mux: run AVCodec::deinit if write_header fails

2016-06-12 Thread Rodger Combs
> On Jun 12, 2016, at 16:24, Marton Balint <c...@passwd.hu> wrote: > > > On Sun, 12 Jun 2016, Rodger Combs wrote: > >> --- >> libavformat/mux.c | 5 - >> 1 file changed, 4 insertions(+), 1 deletion(-) >> >> diff --git a/libavformat/mux.c b

[FFmpeg-devel] [PATCH 13/13] fate/hevc: add automatic bsf test

2016-06-12 Thread Rodger Combs
--- tests/fate/hevc.mak | 11 +++ 1 file changed, 11 insertions(+) diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index 05266cd..5ae5d3d 100644 --- a/tests/fate/hevc.mak +++ b/tests/fate/hevc.mak @@ -225,6 +225,17 @@ $(foreach N,$(HEVC_SAMPLES_444_12BIT),$(eval $(call

[FFmpeg-devel] [PATCH 11/13] fate/h264: add automatic bsf test

2016-06-12 Thread Rodger Combs
--- tests/fate/h264.mak | 2 ++ tests/ref/fate/h264-autobsf-mp4toannexb | 1 + 2 files changed, 3 insertions(+) create mode 100644 tests/ref/fate/h264-autobsf-mp4toannexb diff --git a/tests/fate/h264.mak b/tests/fate/h264.mak index e12263c..5307a29 100644 ---

[FFmpeg-devel] [PATCH 12/13] fate/aac: add automatic bsf test

2016-06-12 Thread Rodger Combs
--- tests/fate/aac.mak | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index 3d64031..b02b768 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -241,6 +241,10 @@ FATE_AAC_LATM += fate-aac-latm_stereo_to_51

[FFmpeg-devel] [PATCH 08/13] lavf/movenc: add deinit function

2016-06-12 Thread Rodger Combs
--- libavformat/movenc.c | 67 +++- 1 file changed, 29 insertions(+), 38 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 2f00091..2f6f8bf 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5284,21 +5284,18

[FFmpeg-devel] [PATCH 10/13] lavf/movenc+dashenc: add automatic bitstream filtering

2016-06-12 Thread Rodger Combs
This is disabled by default when the empty_moov flag is enabled --- libavformat/dashenc.c | 43 +++- libavformat/movenc.c | 107 +++--- 2 files changed, 124 insertions(+), 26 deletions(-) diff --git a/libavformat/dashenc.c

[FFmpeg-devel] [PATCH 09/13] lavf/dashenc: add deinit function

2016-06-12 Thread Rodger Combs
--- libavformat/dashenc.c | 51 +-- 1 file changed, 17 insertions(+), 34 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 519f9c4..0848052 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -580,16

[FFmpeg-devel] [PATCH 07/13] lavf/segment: fix writing separate header with auto BSF

2016-06-12 Thread Rodger Combs
--- libavformat/segment.c | 29 - 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index d22d550..d8877f0 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -89,6 +89,7 @@ typedef struct

[FFmpeg-devel] [PATCH 06/13] lavf/segment: add deinit function

2016-06-12 Thread Rodger Combs
--- libavformat/segment.c | 47 --- 1 file changed, 20 insertions(+), 27 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 4c6c6d4..d22d550 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -627,8 +627,9 @@

[FFmpeg-devel] [PATCH 04/13] lavf/mux: run AVCodec::deinit if write_header fails

2016-06-12 Thread Rodger Combs
--- libavformat/mux.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/mux.c b/libavformat/mux.c index dd3de24..071eac1 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -484,8 +484,11 @@ int avformat_write_header(AVFormatContext *s, AVDictionary

[FFmpeg-devel] [PATCH 03/13] lavf: add a flag to enable/disable automatic bitstream filtering

2016-06-12 Thread Rodger Combs
This is mostly useful for muxers that wrap other muxers, such as dashenc and segment. The actual duplicated bitstream filtering is largely harmless, but delaying the header can cause problems when the muxer intended the header to be written to a separate file. --- libavformat/avformat.h | 1

[FFmpeg-devel] [PATCH 01/13] lavf: deprecate av_apply_bitstream_filters

2016-06-12 Thread Rodger Combs
--- libavformat/avformat.h | 3 +++ libavformat/utils.c| 4 2 files changed, 7 insertions(+) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index f66c39b..b4fe626 100644 --- a/libavformat/avformat.h +++ b/libavformat/avformat.h @@ -2869,8 +2869,11 @@ int

[FFmpeg-devel] [PATCH 06/14] lavf/segment: add deinit function

2016-06-12 Thread Rodger Combs
--- libavformat/segment.c | 47 --- 1 file changed, 20 insertions(+), 27 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index 4c6c6d4..d22d550 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -627,8 +627,9 @@

[FFmpeg-devel] [PATCH 01/14] lavf: deprecate av_apply_bitstream_filters

2016-06-12 Thread Rodger Combs
--- libavformat/avformat.h | 3 +++ libavformat/utils.c| 4 2 files changed, 7 insertions(+) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index f66c39b..b4fe626 100644 --- a/libavformat/avformat.h +++ b/libavformat/avformat.h @@ -2869,8 +2869,11 @@ int

[FFmpeg-devel] [PATCH 02/14] lavf: update auto-bsf to new BSF API

2016-06-12 Thread Rodger Combs
--- libavformat/internal.h | 5 +++-- libavformat/mux.c | 45 +- libavformat/segment.c | 6 +++-- libavformat/utils.c| 59 +- 4 files changed, 91 insertions(+), 24 deletions(-) diff --git

[FFmpeg-devel] [PATCH 09/14] lavf/dashenc: add deinit function

2016-06-12 Thread Rodger Combs
--- libavformat/dashenc.c | 51 +-- 1 file changed, 17 insertions(+), 34 deletions(-) diff --git a/libavformat/dashenc.c b/libavformat/dashenc.c index 519f9c4..0848052 100644 --- a/libavformat/dashenc.c +++ b/libavformat/dashenc.c @@ -580,16

[FFmpeg-devel] [PATCH 08/14] lavf/movenc: add deinit function

2016-06-12 Thread Rodger Combs
--- libavformat/movenc.c | 65 ++-- 1 file changed, 28 insertions(+), 37 deletions(-) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index 2f00091..acb0e25 100644 --- a/libavformat/movenc.c +++ b/libavformat/movenc.c @@ -5284,21 +5284,18

[FFmpeg-devel] [PATCH 03/14] lavf: add a flag to enable/disable automatic bitstream filtering

2016-06-12 Thread Rodger Combs
This is mostly useful for muxers that wrap other muxers, such as dashenc and segment. The actual duplicated bitstream filtering is largely harmless, but delaying the header can cause problems when the muxer intended the header to be written to a separate file. --- libavformat/avformat.h | 1

[FFmpeg-devel] [PATCH 05/14] lavf/mux: add avformat_init_output

2016-06-12 Thread Rodger Combs
This allows a consumer to run the muxer's init function without actually writing the header, which is useful in chained muxers that support automatic bitstream filtering. --- libavformat/avformat.h | 34 +-- libavformat/internal.h | 10 libavformat/mux.c | 64

[FFmpeg-devel] [PATCH 04/14] lavf/mux: run AVCodec::deinit if write_header fails

2016-06-12 Thread Rodger Combs
--- libavformat/mux.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/mux.c b/libavformat/mux.c index dd3de24..071eac1 100644 --- a/libavformat/mux.c +++ b/libavformat/mux.c @@ -484,8 +484,11 @@ int avformat_write_header(AVFormatContext *s, AVDictionary

[FFmpeg-devel] [PATCH 11/14] fate/h264: add automatic bsf test

2016-06-12 Thread Rodger Combs
--- tests/fate/h264.mak | 2 ++ tests/ref/fate/h264-autobsf-mp4toannexb | 1 + 2 files changed, 3 insertions(+) create mode 100644 tests/ref/fate/h264-autobsf-mp4toannexb diff --git a/tests/fate/h264.mak b/tests/fate/h264.mak index e12263c..5307a29 100644 ---

[FFmpeg-devel] [PATCH 14/14] fate/hevc: add automatic bsf test

2016-06-12 Thread Rodger Combs
--- tests/fate/hevc.mak | 11 +++ 1 file changed, 11 insertions(+) diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index 05266cd..5ae5d3d 100644 --- a/tests/fate/hevc.mak +++ b/tests/fate/hevc.mak @@ -225,6 +225,17 @@ $(foreach N,$(HEVC_SAMPLES_444_12BIT),$(eval $(call

[FFmpeg-devel] [PATCH 12/14] fate/aac: add automatic bsf test

2016-06-12 Thread Rodger Combs
--- tests/fate/aac.mak | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index 3d64031..b02b768 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -241,6 +241,10 @@ FATE_AAC_LATM += fate-aac-latm_stereo_to_51

[FFmpeg-devel] [PATCH 13/14] fate/vp9: add automatic bsf test

2016-06-12 Thread Rodger Combs
--- tests/fate/vpx.mak | 5 + 1 file changed, 5 insertions(+) diff --git a/tests/fate/vpx.mak b/tests/fate/vpx.mak index f0bcfac..29cd2a7 100644 --- a/tests/fate/vpx.mak +++ b/tests/fate/vpx.mak @@ -133,5 +133,10 @@ FATE_VP9-$(CONFIG_IVF_DEMUXER) += fate-vp9-05-resize fate-vp9-05-resize:

[FFmpeg-devel] [PATCH 07/14] lavf/segment: fix writing separate header with auto BSF

2016-06-12 Thread Rodger Combs
--- libavformat/segment.c | 29 - 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index d22d550..d8877f0 100644 --- a/libavformat/segment.c +++ b/libavformat/segment.c @@ -89,6 +89,7 @@ typedef struct

[FFmpeg-devel] [PATCH 10/14] lavf/movenc+dashenc: add automatic bitstream filtering

2016-06-12 Thread Rodger Combs
This is disabled by default when the empty_moov flag is enabled --- libavformat/dashenc.c | 43 - libavformat/movenc.c | 105 +++--- 2 files changed, 123 insertions(+), 25 deletions(-) diff --git a/libavformat/dashenc.c

[FFmpeg-devel] [PATCH] lavc/audiotoolbox: fix mappings for surround-direct and surround-center channels

2016-06-09 Thread Rodger Combs
--- libavcodec/audiotoolboxdec.c | 2 +- libavcodec/audiotoolboxenc.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/audiotoolboxdec.c b/libavcodec/audiotoolboxdec.c index 2748e8d..3f12a46 100644 --- a/libavcodec/audiotoolboxdec.c +++

Re: [FFmpeg-devel] [PATCH] lavc/audiotoolboxenc: fix dropped frames on iOS

2016-06-09 Thread Rodger Combs
One comment below; otherwise this LGTM. > On Jun 2, 2016, at 01:32, Rick Kern wrote: > > AudioConverterFillComplexBuffer() doesn't always call its callback. A frame > queue is used to prevent skipped audio samples. > > Signed-off-by: Rick Kern > --- >

[FFmpeg-devel] [PATCH] lavf/srtdec: fix probing files with negative first timestamps

2016-06-09 Thread Rodger Combs
--- libavformat/srtdec.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavformat/srtdec.c b/libavformat/srtdec.c index 585aa6a..9ab7a4e 100644 --- a/libavformat/srtdec.c +++ b/libavformat/srtdec.c @@ -52,7 +52,10 @@ static int srt_probe(AVProbeData *p) /* Check if

[FFmpeg-devel] [PATCH] lavf/srtdec: fix probing files with negative first timestamps

2016-06-06 Thread Rodger Combs
--- libavformat/srtdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/srtdec.c b/libavformat/srtdec.c index 585aa6a..06061a8 100644 --- a/libavformat/srtdec.c +++ b/libavformat/srtdec.c @@ -52,7 +52,7 @@ static int srt_probe(AVProbeData *p) /* Check if the

[FFmpeg-devel] [PATCH 1/2] lavfi/vf_overlay: change variable name to avoid shadowing

2016-05-24 Thread Rodger Combs
--- libavfilter/vf_overlay.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/libavfilter/vf_overlay.c b/libavfilter/vf_overlay.c index 37f19ea..71f4db7 100644 --- a/libavfilter/vf_overlay.c +++ b/libavfilter/vf_overlay.c @@ -401,7 +401,7 @@

[FFmpeg-devel] [PATCH 2/2] lavfi/vf_overlay: support NV12

2016-05-24 Thread Rodger Combs
--- libavfilter/vf_overlay.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/libavfilter/vf_overlay.c b/libavfilter/vf_overlay.c index 71f4db7..e01b924 100644 --- a/libavfilter/vf_overlay.c +++ b/libavfilter/vf_overlay.c @@ -125,6 +125,7 @@ typedef struct

<    1   2   3   4   5   6   7   8   >