Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-08 Thread Clément Bœsch
On Wed, Sep 07, 2016 at 10:07:26PM -0300, James Almer wrote: > On 9/7/2016 6:14 AM, Michael Niedermayer wrote: > >> libavformat/utils.c |4 +++- > >> > tests/ref/fate/copy-trac4914 |4 ++-- > >> > tests/ref/fate/copy-trac4914-avi |4 ++-- > >> > 3 files changed, 7

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
>> for fmt in yuv420p nv12 bgr0 rgb0; do >> ./ffmpeg -f lavfi -i "testsrc=size=1920x1080:duration=10:rate=30" >> -c:v h264_nvenc -global_quality 20 -pix_fmt "$fmt" -y out_"${fmt}".mkv >> done > > You feed to nvenc only rgb? what testsrc only supports. Use testsrc2. pix_fmt should make sure

Re: [FFmpeg-devel] [PATCH] Fix potential integer overflow in mov_read_keys

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 02:38:48PM -0700, Sergey Volk wrote: > I just realized that count+1 itself might overflow if count==UINT_MAX, so I > guess it's better to subtract 1 from the right-hand side. Attached updated > patch. > > On Wed, Sep 7, 2016 at 2:21 PM, Sergey Volk

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Sven C. Dack wrote: It is not an issue. x11grab produces BGR0 and nvenc can handle it with the patch. It's giving me 100fp/s (up from 47fp/s) with a 1920x1080 monitor. I'd imagine people with 4K displays will be happy, too, although they will have to live with lower speeds of perhaps 30 fp/s.

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Sven C. Dack
Hallo, ich schicke Dir noch einen weitere Backtrace mit etwas mehr Details: (gdb) bt #0 0x7f19eb8e3a3e in ?? () from /lib/x86_64-linux-gnu/libc.so.6 #1 0x00fbc63e in av_image_copy_plane (height=8, bytewidth=7680, src_linesize=, src=0x7f19ea08f000 "", dst_linesize=out>, dst=) at

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Nicolas George
Le tridi 23 fructidor, an CCXXIV, Paul B Mahol a écrit : > You feed to nvenc only rgb? what testsrc only supports. Use testsrc2. That would be more efficient, but with testsrc, lavfi inserts the scale filter to convert into the requested format anyway. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Steven Liu
2016-09-08 17:46 GMT+08:00 Michael Niedermayer : > On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wrote: > > From: Aman Gupta > > > > --- > > doc/muxers.texi | 4 > > libavformat/hlsenc.c | 13 +++-- > > 2 files changed, 15

Re: [FFmpeg-devel] questions about multi-thread issues for ffmpeg

2016-09-08 Thread Ronald S. Bultje
Hi Andrew, On Thu, Sep 8, 2016 at 3:59 AM, qw wrote: > I have made one simple transcoding program by using ffmpeg lib, where > several av transcoding tasks are done in one process and each task > corresponds to one thread. > > But some ffmpeg function will report error, and

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Paul B Mahol
On 9/8/16, Timo Rothenpieler wrote: > Am 08.09.2016 um 02:29 schrieb Sven C. Dack: >> On 08/09/16 00:57, Hendrik Leppkes wrote: >>> The image copying code was refactored in an earlier patch to be >>> generic and not rely on hard-coding format info, hence the second part >>>

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wrote: > From: Aman Gupta > > --- > doc/muxers.texi | 4 > libavformat/hlsenc.c | 13 +++-- > 2 files changed, 15 insertions(+), 2 deletions(-) Isnt this redundant with -output_ts_offset ? or how do they

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
Am 08.09.2016 um 02:29 schrieb Sven C. Dack: > On 08/09/16 00:57, Hendrik Leppkes wrote: >> The image copying code was refactored in an earlier patch to be >> generic and not rely on hard-coding format info, hence the second part >> is not needed anymore. >> > > This is not quite accurate. It

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Timo Rothenpieler
Am 08.09.2016 um 02:29 schrieb Sven C. Dack: > On 08/09/16 00:57, Hendrik Leppkes wrote: >> The image copying code was refactored in an earlier patch to be >> generic and not rely on hard-coding format info, hence the second part >> is not needed anymore. >> > > This is not quite accurate. It

[FFmpeg-devel] questions about multi-thread issues for ffmpeg

2016-09-08 Thread qw
Hi, I have made one simple transcoding program by using ffmpeg lib, where several av transcoding tasks are done in one process and each task corresponds to one thread. But some ffmpeg function will report error, and sometimes the multi-thread version of transcoding program will crash

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Ligverd Haer
> > Common attributes for AdaptationSet and Representation shall either be in > > one of the elements but not in both. > > > > in particular, the attribute frameRate is duplicated in the node > > Representation > > There is a bug report outstanding for this: > https://trac.ffmpeg.org/ticket/5087

[FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Aman Gupta
From: Aman Gupta --- libavcodec/videotoolboxenc.c | 76 ++-- 1 file changed, 67 insertions(+), 9 deletions(-) diff --git a/libavcodec/videotoolboxenc.c b/libavcodec/videotoolboxenc.c index 4345ca3..859dde9 100644 ---

Re: [FFmpeg-devel] [PATCH] ffmpeg: drop format specific stream copy heuristics

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 10:07:26PM -0300, James Almer wrote: > On 9/7/2016 6:14 AM, Michael Niedermayer wrote: > >> libavformat/utils.c |4 +++- > >> > tests/ref/fate/copy-trac4914 |4 ++-- > >> > tests/ref/fate/copy-trac4914-avi |4 ++-- > >> > 3 files changed, 7

Re: [FFmpeg-devel] Possible incomplete commit "avcodec/nvenc: support RGB input"

2016-09-08 Thread Sven C. Dack
Just a guess, but you are using nv_surface->height, which appears to have been rounded up to 1088 while the actual size is 1080. The 1088 is then passed on to av_image_copy() where it probably tries to read 1920x1088 pixels from the source, but the source might not have that much memory and so

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Aman Gupta
On Thu, Sep 8, 2016 at 2:55 AM, Steven Liu wrote: > 2016-09-08 17:46 GMT+08:00 Michael Niedermayer : > > > On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wrote: > > > From: Aman Gupta > > > > > > --- > > > doc/muxers.texi

Re: [FFmpeg-devel] [PATCH 1/2] lavc/hevc: store VPS/SPS/PPS data

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 04:53:53PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > --- > libavcodec/hevc.h| 9 + > libavcodec/hevc_ps.c | 27 +++ > 2 files changed, 36 insertions(+) > > diff --git a/libavcodec/hevc.h

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

2016-09-08 Thread Nicolas George
Le duodi 22 fructidor, an CCXXIV, Rodger Combs a écrit : > --- > 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

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 05:10:14AM -0700, Aman Gupta wrote: > On Thu, Sep 8, 2016 at 2:55 AM, Steven Liu wrote: > > > 2016-09-08 17:46 GMT+08:00 Michael Niedermayer : > > > > > On Thu, Sep 08, 2016 at 10:47:42AM +0900, Aman Gupta wrote: > > > >

Re: [FFmpeg-devel] [PATCH] ffprobe: add -show_headers_first option

2016-09-08 Thread Stefano Sabatini
On date Sunday 2016-09-04 19:09:32 +0200, Nicolas George encoded: > Le nonidi 19 fructidor, an CCXXIV, Stefano Sabatini a écrit : > > This is meant to be used for generating output suitable for the > > ffprobe_default demuxer. > > --- > > ffprobe.c | 30 ++ > > 1 file

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 09:58:02AM +0300, Ligverd Haer wrote: > > > Common attributes for AdaptationSet and Representation shall either be in > > > one of the elements but not in both. > > > > > > in particular, the attribute frameRate is duplicated in the node > > > Representation > > > > There

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

2016-09-08 Thread Michael Niedermayer
On Wed, Sep 07, 2016 at 07:17:59PM -0500, Rodger Combs wrote: > --- > ffmpeg.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) ill leave these 2 patches for nicolas to review ... thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Asymptotically

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Steven Liu
2016-09-08 20:31 GMT+08:00 Michael Niedermayer : > On Thu, Sep 08, 2016 at 05:10:14AM -0700, Aman Gupta wrote: > > On Thu, Sep 8, 2016 at 2:55 AM, Steven Liu > wrote: > > > > > 2016-09-08 17:46 GMT+08:00 Michael Niedermayer

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 12:01 GMT+02:00 Andy Furniss : > I don't know what it is about x11grab/CSC with ffmpeg, but > on my old CPU gstreamer is twice as fast. x11grab or xcb? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Ligverd Haer
В письме от четверг, 8 сентября 2016 г. 14:35:27 MSK пользователь Michael Niedermayer написал: > patches should be sent to the mailing list, patches on trac tend to > be missed and forgotten Don't want to clog the ffmpeg-devel mailing list. I made a pull request on github, I was sent to

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: remove initial_offset option from segment.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 15:33 GMT+02:00 Steven Liu : > -{ "initial_offset", "set initial timestamp offset", > OFFSET(initial_offset), AV_OPT_TYPE_DURATION, > {.i64 = 0}, -INT64_MAX, INT64_MAX, E }, You generally cannot simply remove options from FFmpeg. (I did not look if the

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: implement initial_offset

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 08:42:05PM +0800, Steven Liu wrote: > 2016-09-08 20:31 GMT+08:00 Michael Niedermayer : > > > On Thu, Sep 08, 2016 at 05:10:14AM -0700, Aman Gupta wrote: > > > On Thu, Sep 8, 2016 at 2:55 AM, Steven Liu > > wrote: > > > > >

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 10:19 GMT+02:00 Aman Gupta : > +{ "a53cc", "Use A53 Closed Captions (if available)", OFFSET(a53_cc), > AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, VE }, Why is this disabled by default? Carl Eugen ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] [RFC] libavfilter HRTF/room acoustics filter [2/2]

2016-09-08 Thread Michael Niedermayer
On Tue, Aug 02, 2016 at 06:18:33AM -0700, Yue Shi Lai wrote: > This is 2/2 of the patch. > af_hrtf_mitkemar.c |14076 > + > 1 file changed, 14076 insertions(+) > c4c72f78adf6aebf20ccd1883388bb69d07b9bf9 >

[FFmpeg-devel] [PATCH 1/2] avformat/segment: remove initial_offset option from segment.c

2016-09-08 Thread Steven Liu
ffmpeg have a generic solution working with all muxer named output_ts_offset Signed-off-by: LiuQi --- libavformat/segment.c | 18 ++ 1 file changed, 2 insertions(+), 16 deletions(-) diff --git a/libavformat/segment.c b/libavformat/segment.c index

[FFmpeg-devel] [PATCH 2/2] doc/muxers: remove initial_offset from segment muxer

2016-09-08 Thread Steven Liu
Signed-off-by: LiuQi --- doc/muxers.texi | 4 1 file changed, 4 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index c39a6a0..4fd4133 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -1355,10 +1355,6 @@ will start with near-zero timestamps. It is meant to

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 15:34 GMT+02:00 Ligverd Haer : > В письме от четверг, 8 сентября 2016 г. 14:35:27 MSK пользователь Michael > Niedermayer написал: > >> patches should be sent to the mailing list, patches on trac tend to >> be missed and forgotten > > Don't want to clog the ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] doc/muxers: remove initial_offset from segment muxer

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 15:33 GMT+02:00 Steven Liu : > -@item initial_offset @var{offset} > -Specify timestamp offset to apply to the output packet timestamps. The > -argument must be a time duration specification, and defaults to 0. This should be merged with the other patch. (If the

[FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
ffmpeg have a generic solution working with all muxer named output_ts_offset, output_ts_offset will instead of initial_offset Signed-off-by: LiuQi --- libavformat/segment.c | 5 + 1 file changed, 5 insertions(+) diff --git a/libavformat/segment.c b/libavformat/segment.c

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Ligverd Haer
> Please send your updated patch as attachment to a > reply in this mailing list thread, > > Please tell us if documentation pointed you to github or trac to > post patches, we try hard to have patch submission (only) on > this mailing list. > > Carl Eugen diff --git a/libavformat/dashenc.c

Re: [FFmpeg-devel] [RFC] marking non developer tickets

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 03:23:36PM +0200, Carl Eugen Hoyos wrote: > 2016-09-07 13:31 GMT+02:00 Michael Niedermayer : > > > Can type = task be used for non devel tasks > > No objections although I don't understand how #5832 can be "fixed" > and I don't like tickets that by

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Carl Eugen Hoyos wrote: 2016-09-08 12:01 GMT+02:00 Andy Furniss : I don't know what it is about x11grab/CSC with ffmpeg, but on my old CPU gstreamer is twice as fast. x11grab or xcb? I guess xcb as I don't --enable anything related with configure.

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 16:17 GMT+02:00 Andy Furniss : > Carl Eugen Hoyos wrote: >> >> 2016-09-08 12:01 GMT+02:00 Andy Furniss : >> >>> I don't know what it is about x11grab/CSC with ffmpeg, but >>> on my old CPU gstreamer is twice as fast. >> >> x11grab or xcb? > > I

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 16:01 GMT+02:00 Steven Liu : > +if (seg->initial_offset > 0) { > +av_log(s, AV_LOG_WARNING, "NOTE: the option initial_offset will be > deprecated soon," "is deprecated" even if you decide not to ignore the value set. Carl Eugen

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Sven C. Dack
On 08/09/16 15:17, Andy Furniss wrote: Carl Eugen Hoyos wrote: 2016-09-08 12:01 GMT+02:00 Andy Furniss : I don't know what it is about x11grab/CSC with ffmpeg, but on my old CPU gstreamer is twice as fast. x11grab or xcb? I guess xcb as I don't --enable anything

Re: [FFmpeg-devel] [PATCH 1/2] lavc/hevc: store VPS/SPS/PPS data

2016-09-08 Thread Matthieu Bouron
On Thu, Sep 8, 2016 at 2:28 PM, Michael Niedermayer wrote: > On Wed, Sep 07, 2016 at 04:53:53PM +0200, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > --- > > libavcodec/hevc.h| 9 + > > libavcodec/hevc_ps.c | 27

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Aman Gupta
On Thursday, September 8, 2016, Carl Eugen Hoyos wrote: > 2016-09-08 10:19 GMT+02:00 Aman Gupta >: > > > +{ "a53cc", "Use A53 Closed Captions (if available)", > OFFSET(a53_cc), AV_OPT_TYPE_BOOL, {.i64 = 0}, 0, 1, VE }, > > Why is this

Re: [FFmpeg-devel] [PATCH] added expr evaluation to drawtext - fontsize

2016-09-08 Thread Michael Niedermayer
On Tue, Sep 06, 2016 at 10:27:24AM -0700, Brett Harrison wrote: > This patch addresses your concerns. > > On Fri, Sep 2, 2016 at 5:05 PM, Michael Niedermayer > wrote: > > > On Fri, Sep 02, 2016 at 03:31:21PM -0700, Brett Harrison wrote: > > > Addressed the following

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Carl Eugen Hoyos wrote: 2016-09-08 19:16 GMT+02:00 Andy Furniss : It is a bit faster xcb is slower? Yes (unless I am mixing things up) --disable-libxcb --enable-x11grab is faster than autodetect ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 15:48 GMT+02:00 Aman Gupta : > On Thursday, September 8, 2016, Carl Eugen Hoyos wrote: > >> 2016-09-08 10:19 GMT+02:00 Aman Gupta >: >> >> > +{ "a53cc", "Use A53 Closed Captions (if available)", >>

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Nicolas George
Le tridi 23 fructidor, an CCXXIV, Carl Eugen Hoyos a écrit : > xcb is slower? Looking at the code, I see that xcbgrab allocates and frees the shared memory segment for each frame, while x11grab only allocates it once at the beginning. xcbgrab does that so that it can wrap the segment in a

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Carl Eugen Hoyos wrote: 2016-09-08 16:17 GMT+02:00 Andy Furniss : Carl Eugen Hoyos wrote: 2016-09-08 12:01 GMT+02:00 Andy Furniss : I don't know what it is about x11grab/CSC with ffmpeg, but on my old CPU gstreamer is twice as fast. x11grab or

Re: [FFmpeg-devel] [PATCH] fate/als: add floating point decoding test

2016-09-08 Thread Michael Niedermayer
On Sun, Jul 17, 2016 at 07:25:31PM +0530, Umair Khan wrote: > On Sun, Jul 17, 2016 at 1:50 PM, Thilo Borgmann > wrote: > > Hi, > > > >> From e172e333807b4b3b2558a1ffa735ade79a3f3e36 Mon Sep 17 00:00:00 2001 > >> From: Umair Khan > >> Date: Sun, 17 Jul

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Carl Eugen Hoyos
2016-09-08 19:16 GMT+02:00 Andy Furniss : > It is a bit faster xcb is slower? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] adding RGBA and BGRA to nvenc.c

2016-09-08 Thread Andy Furniss
Sven C. Dack wrote: On 08/09/16 15:17, Andy Furniss wrote: Carl Eugen Hoyos wrote: 2016-09-08 12:01 GMT+02:00 Andy Furniss : I don't know what it is about x11grab/CSC with ffmpeg, but on my old CPU gstreamer is twice as fast. x11grab or xcb? I guess xcb as I don't

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 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.c b/ffmpeg.c >> index d858407..1d793fe 100644 >> ---

Re: [FFmpeg-devel] [FFmpeg-cvslog] avcodec/alsdec: Fix reading 0 mantisse bits

2016-09-08 Thread James Almer
> ffmpeg | branch: master | Michael Niedermayer | > Thu Sep 8 22:02:44 2016 +0200| [037422178d7f1d0dd09e1ce424dd61a69e77668b] | > committer: Michael Niedermayer > > avcodec/alsdec: Fix reading 0 mantisse bits > > Fixes assertion failure > Fixes: >

[FFmpeg-devel] [PATCH 2/2] Normalize VP8 and VP9 tags when writing IVF.

2016-09-08 Thread Alex Converse
VP9-in-ISOM uses vp08 and vp09 tags, while ivf uses VP80 and VP90. --- libavformat/ivfenc.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c index 5dbcd97..1735606 100644 --- a/libavformat/ivfenc.c +++

[FFmpeg-devel] [PATCH 1/2] Parse VP9 when coming out of MP4.

2016-09-08 Thread Alex Converse
Fixes Netflix VP9 DASH samples. --- libavformat/mov.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/mov.c b/libavformat/mov.c index a7595c5..6e80b93 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -2161,6 +2161,7 @@ static int mov_finalize_stsd_codec(MOVContext *c,

Re: [FFmpeg-devel] dash encoder. Correct generated manifest for MPEG-DASH MPD Validator and calculate current bandwidth for eath chunk. Now works correctly with dash.sj

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 04:49:12PM +0300, Ligverd Haer wrote: > > Please send your updated patch as attachment to a > > reply in this mailing list thread, > > > > Please tell us if documentation pointed you to github or trac to > > post patches, we try hard to have patch submission (only) on > >

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
2016-09-08 22:52 GMT+08:00 Carl Eugen Hoyos : > 2016-09-08 16:01 GMT+02:00 Steven Liu : > > > +if (seg->initial_offset > 0) { > > +av_log(s, AV_LOG_WARNING, "NOTE: the option initial_offset will > be > > deprecated soon," > > "is

[FFmpeg-devel] [PATCH] avcodec/mlz: Check output chars before using it

2016-09-08 Thread Michael Niedermayer
Fixes hypothetical integer overflow Signed-off-by: Michael Niedermayer --- libavcodec/mlz.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/libavcodec/mlz.c b/libavcodec/mlz.c index 039635d..a2d1b89 100644 --- a/libavcodec/mlz.c

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Aman Gupta
I tried to switch from -initial_offset to -output_ts_offset, but it does not work as expected. In Safari, the generated HLS stream stalls out and stops playing after the first segment. Aman On Thu, Sep 8, 2016 at 5:54 PM, Michael Niedermayer wrote: > On Fri, Sep 09,

Re: [FFmpeg-devel] [PATCH 1/2] Parse VP9 when coming out of MP4.

2016-09-08 Thread James Almer
On 9/8/2016 8:41 PM, Alex Converse wrote: > Fixes Netflix VP9 DASH samples. > --- > libavformat/mov.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/mov.c b/libavformat/mov.c > index a7595c5..6e80b93 100644 > --- a/libavformat/mov.c > +++ b/libavformat/mov.c > @@ -2161,6

Re: [FFmpeg-devel] [PATCH 1/2] Parse VP9 when coming out of MP4.

2016-09-08 Thread Ronald S. Bultje
Hi, On Thu, Sep 8, 2016 at 9:50 PM, James Almer wrote: > On 9/8/2016 8:41 PM, Alex Converse wrote: > > Fixes Netflix VP9 DASH samples. > > --- > > libavformat/mov.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/libavformat/mov.c b/libavformat/mov.c > > index

Re: [FFmpeg-devel] [PATCH] avformat/mov: Enable stream parsing for VP9.

2016-09-08 Thread Ronald S. Bultje
Hi, On Wed, Sep 7, 2016 at 7:37 AM, Ronald S. Bultje wrote: > Hi, > > On Tue, Sep 6, 2016 at 10:39 PM, Matthew Gregan wrote: > >> At 2016-09-06T22:18:18-0400, Ronald S. Bultje wrote: >> > I think the patch is fine, but I wonder if it should set it to _FULL

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Steven Liu
2016-09-09 9:59 GMT+08:00 Aman Gupta : > Yea this doesn't work at all. I wish someone had bothered to check before > the patch was proposed or merged. > > With "-ss 20 -i file -segment_time 2 -segment_start_number 10 > -initial_offset 20": > > #EXTM3U > #EXT-X-VERSION:3 >

[FFmpeg-devel] [PATCH] avformat/segment: fix the duration error of use output_ts_offset

2016-09-08 Thread Steven Liu
This patch can merge with 1da00be009aa74400042bf470b9a5ffbd82a1c5e i have checked this modify: ./ffmpeg -i ~/facebook.mp4 -c copy -f segment -segment_time 2 -output_ts_offset 80 -segment_list output-test.m3u8 -v debug output-test-%03d.ts #EXTM3U #EXT-X-VERSION:3 #EXT-X-MEDIA-SEQUENCE:0

Re: [FFmpeg-devel] [PATCH] MAINTAINERS: Add myself for hlsenc

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 12:56:19AM +0800, Steven Liu wrote: > Signed-off-by: Steven Liu > --- > MAINTAINERS | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) applied maybe the entry should be split between encryption for hlsenc and the rest of hlsenc thx [...]

Re: [FFmpeg-devel] [PATCH 2/2] Normalize VP8 and VP9 tags when writing IVF.

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 04:41:03PM -0700, Alex Converse wrote: > VP9-in-ISOM uses vp08 and vp09 tags, while ivf uses VP80 and VP90. > --- > libavformat/ivfenc.c | 13 - > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/libavformat/ivfenc.c b/libavformat/ivfenc.c >

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Aman Gupta
Yea this doesn't work at all. I wish someone had bothered to check before the patch was proposed or merged. With "-ss 20 -i file -segment_time 2 -segment_start_number 10 -initial_offset 20": #EXTM3U #EXT-X-VERSION:3 #EXT-X-MEDIA-SEQUENCE:0 #EXT-X-ALLOW-CACHE:YES #EXT-X-TARGETDURATION:3

[FFmpeg-devel] Fwd: Re: [PATCH v3] lavf : scale_vaapi : add denoise/sharpless support

2016-09-08 Thread Jun Zhao
cc Mark Forwarded Message Subject: Re: [FFmpeg-devel] [PATCH v3] lavf : scale_vaapi : add denoise/sharpless support Date: Mon, 5 Sep 2016 09:33:43 +0800 From: Jun Zhao To: ffmpeg-devel@ffmpeg.org On 2016/8/31 6:48, Mark Thompson wrote: > On 30/08/16

[FFmpeg-devel] Fwd: Re: [PATCH v3] lavf : scale_vaapi : add denoise/sharpless support

2016-09-08 Thread Jun Zhao
cc Mark Forwarded Message Subject: Re: [FFmpeg-devel] [PATCH v3] lavf : scale_vaapi : add denoise/sharpless support Date: Mon, 5 Sep 2016 09:52:22 +0800 From: Jun Zhao To: FFmpeg development discussions and patches On

[FFmpeg-devel] [PATCH v4] lavf : scale_vaapi : add denoise/sharpless support

2016-09-08 Thread Jun Zhao
v4 : - fix sharpless typo(sharpless -> sharpness) - when don't support denoise/sharpness, report the error and return - fix denoise/sharpness params buffer leak in error handle v3 : fix sharpless mapping issue v2 : fix filter support flag check logic issue From

Re: [FFmpeg-devel] [PATCH 1/2] avformat/segment: give a warning message for remove initial_offset option

2016-09-08 Thread Michael Niedermayer
On Fri, Sep 09, 2016 at 02:59:52AM +0800, Steven Liu wrote: > 2016-09-08 22:52 GMT+08:00 Carl Eugen Hoyos : > > > 2016-09-08 16:01 GMT+02:00 Steven Liu : > > > > > +if (seg->initial_offset > 0) { > > > +av_log(s, AV_LOG_WARNING, "NOTE: the

Re: [FFmpeg-devel] [PATCH 1/2] lavc/hevc: store VPS/SPS/PPS data

2016-09-08 Thread Michael Niedermayer
On Thu, Sep 08, 2016 at 04:18:26PM +0200, Matthieu Bouron wrote: > On Thu, Sep 8, 2016 at 2:28 PM, Michael Niedermayer > wrote: > > > On Wed, Sep 07, 2016 at 04:53:53PM +0200, Matthieu Bouron wrote: > > > From: Matthieu Bouron > > > > > >

Re: [FFmpeg-devel] [PATCH] lavc/videotoolboxenc: implement a53cc

2016-09-08 Thread Aman Gupta
On Thu, Sep 8, 2016 at 9:18 AM, Carl Eugen Hoyos wrote: > 2016-09-08 15:48 GMT+02:00 Aman Gupta : > > On Thursday, September 8, 2016, Carl Eugen Hoyos > wrote: > > > >> 2016-09-08 10:19 GMT+02:00 Aman Gupta