Re: [FFmpeg-devel] [PATCH v2 04/18] avformat/movenc: deal with AVMEDIA_TYPE_DATA by using AV_CODEC_ID_META

On 09/06/2016 09:26 PM, Carl Eugen Hoyos wrote: 2016-09-06 15:07 GMT+02:00 Erkki Seppälä : Would it be even better to not copy the data tracks at all by default, so not set the .data_codec field for any format? This was my original question for which I do not know the answer: If you feel it d

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

> avctx->width << 1, avctx->height); > +} else if (frame->format == AV_PIX_FMT_RGBA || frame->format == > AV_PIX_FMT_RGB0) { > + av_image_copy_plane(buf, lockBufferParams->pitch, > + frame->data[0], frame->linesize[0], > + avctx->width << 2, avctx->height);

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

Hi! 2016-09-07 2:23 GMT+02:00 Sven C. Dack : > On 07/09/16 01:08, Sven C. Dack wrote: >> >> On 07/09/16 00:43, Carl Eugen Hoyos wrote: >>> >>> Should be AV_PIX_FMT_RGB0... As pointed out by Timo, this should be AV_PIX_FMT_0BGR32... >>> ... and AV_PIX_FMT_BGR0 ... and this AV_PIX_FMT_0RGB32: It

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

On Tue, Sep 06, 2016 at 06:26:59PM +0200, Michael Niedermayer wrote: > On Tue, Sep 06, 2016 at 05:45:46PM +0200, Hendrik Leppkes wrote: > > On Tue, Sep 6, 2016 at 5:10 PM, Michael Niedermayer > > wrote: > > > On Tue, Sep 06, 2016 at 02:39:11PM +0200, Hendrik Leppkes wrote: > > >> On Tue, Sep 6, 20

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

On 9/7/16, Carl Eugen Hoyos wrote: > Hi! > > 2016-09-07 2:23 GMT+02:00 Sven C. Dack : >> On 07/09/16 01:08, Sven C. Dack wrote: >>> >>> On 07/09/16 00:43, Carl Eugen Hoyos wrote: Should be AV_PIX_FMT_RGB0... > > As pointed out by Timo, this should be AV_PIX_FMT_0BGR32... > ... and A

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

On 07/09/16 09:23, Timo Rothenpieler wrote: avctx->width << 1, avctx->height); +} else if (frame->format == AV_PIX_FMT_RGBA || frame->format == AV_PIX_FMT_RGB0) { + av_image_copy_plane(buf, lockBufferParams->pitch, + frame->data[0], frame->linesize[0], +

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

On 07/09/16 10:40, Sven C. Dack wrote: It is necessary in order to make it work. The twist here is intentional as I pointed out earlier. If you do it the other way around as described in the documentation then you get false and missing colours. I'd like to keep in the transparency channel un

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

>> Also, why is the twist from AV_PIX_FMT_RGBA to NV_ENC_BUFFER_FORMAT_ABGR >> necessary? >> >> The nvenc header describes it as "8 bit Packed A8B8G8R8", so did they >> mess it up? > > It is necessary in order to make it work. The twist here is intentional > as I pointed out earlier. If you do it

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

Hi! > Am 07.09.2016 um 11:36 schrieb Paul B Mahol : > >> On 9/7/16, Carl Eugen Hoyos wrote: >> Hi! >> >> 2016-09-07 2:23 GMT+02:00 Sven C. Dack : On 07/09/16 01:08, Sven C. Dack wrote: > On 07/09/16 00:43, Carl Eugen Hoyos wrote: > > Should be AV_PIX_FMT_RGB0... >> >> A

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

Hi! > Am 07.09.2016 um 11:40 schrieb "Sven C. Dack" : > > On 07/09/16 09:23, Timo Rothenpieler wrote: > Otherwise will RGBA/BGRA have to > be converted into RGB0/BGR0 > and you will again get a performance penalty. What makes you think so? (The problem is that your encoder now reports "I sup

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

On 07/09/16 11:25, Carl Eugen Hoyos wrote: Hi! Am 07.09.2016 um 11:40 schrieb "Sven C. Dack" : On 07/09/16 09:23, Timo Rothenpieler wrote: Otherwise will RGBA/BGRA have to be converted into RGB0/BGR0 and you will again get a performance penalty. What makes you think so? I have tested it. Wh

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

2016-09-07 12:50 GMT+02:00 Sven C. Dack : > On 07/09/16 11:25, Carl Eugen Hoyos wrote: >> >>> Am 07.09.2016 um 11:40 schrieb "Sven C. Dack" : >>> >>> On 07/09/16 09:23, Timo Rothenpieler wrote: >>> Otherwise will RGBA/BGRA have to >>> be converted into RGB0/BGR0 >>> and you will again get a perform

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

Hi Most of the tickets on trac are about ffmpeg code. I belive this makes it hard for non developers to find things to contribute to, yet at the same time there are quite a few non devel tasks that need to be done and that arent done ... Can type = task be used for non devel tasks or should a key

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

Am 07.09.2016 um 13:27 schrieb Carl Eugen Hoyos: > 2016-09-07 12:50 GMT+02:00 Sven C. Dack : >> On 07/09/16 11:25, Carl Eugen Hoyos wrote: >>> Am 07.09.2016 um 11:40 schrieb "Sven C. Dack" : On 07/09/16 09:23, Timo Rothenpieler wrote: Otherwise will RGBA/BGRA have to be con

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

On 07/09/16 12:27, Carl Eugen Hoyos wrote: If your patch adds "RGBA" as supported colour space, the encoder (by definition) announces that it supports transparency. nvenc does not support transparency (if it would, it would be a - grave - bug to use "RGB0" in the patch). So please do not add it.

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

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 (the > > parser ignores the option, but it is semantically more correct). > > Good point, thanks for the feed

[FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

Signed-off-by: Michael Niedermayer --- doc/developer.texi | 6 ++ 1 file changed, 6 insertions(+) diff --git a/doc/developer.texi b/doc/developer.texi index 4d3a7ae..51e3da7 100644 --- a/doc/developer.texi +++ b/doc/developer.texi @@ -641,6 +641,12 @@ are notoriously left unchecked, which is

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

Hi, On Wed, Sep 7, 2016 at 7:51 AM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > doc/developer.texi | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/doc/developer.texi b/doc/developer.texi > index 4d3a7ae..51e3da7 100644 > --- a/doc/developer.texi > +++

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

On Wed, Sep 07, 2016 at 08:03:20AM -0400, Ronald S. Bultje wrote: > Hi, > > On Wed, Sep 7, 2016 at 7:51 AM, Michael Niedermayer > wrote: > > > Signed-off-by: Michael Niedermayer > > --- > > doc/developer.texi | 6 ++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/doc/developer.te

Re: [FFmpeg-devel] [PATCH] lavc/mediacodecdec_h264: fix SODB escaping

Hi, On 06/09/2016 16:53, Matthieu Bouron wrote: From: Matthieu Bouron Fixes escaping of consecutive 0x00, 0x00, 0x0{0-3} sequences. --- libavcodec/mediacodecdec_h264.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/mediacodecdec_h264.c b/libavcodec/med

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

On 07/09/16 12:40, Timo Rothenpieler wrote: libavutil/pixfmt.h defines AV_PIX_FMT_RGB0 and the other ones like this: packed RGB 8:8:8, 32bpp, XRGBXRGB... X=unused/undefined So I would expect the Alpha-Channel to be anything, and converting from RGBA to RGB0 to be a no-op "conversion". It is

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

Am 07.09.2016 um 15:26 schrieb Sven C. Dack: > On 07/09/16 12:40, Timo Rothenpieler wrote: >> libavutil/pixfmt.h defines AV_PIX_FMT_RGB0 and the other ones like this: >> >> packed RGB 8:8:8, 32bpp, XRGBXRGB... X=unused/undefined >> >> So I would expect the Alpha-Channel to be anything, and conver

Re: [FFmpeg-devel] [PATCH] lavc/mediacodecdec_h264: fix SODB escaping

On Wed, Sep 07, 2016 at 02:50:18PM +0200, Benoit Fouet wrote: > Hi, > > > On 06/09/2016 16:53, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > Fixes escaping of consecutive 0x00, 0x00, 0x0{0-3} sequences. > > --- > > libavcodec/mediacodecdec_h264.c | 6 +++--- > > 1 file changed, 3

Re: [FFmpeg-devel] [PATCH 2/2] lavc/mediacodecdec_h264: use h264_parse.h instead of h264dec.h

On Mon, Sep 05, 2016 at 03:57:54PM +0200, Matthieu Bouron wrote: > From: Matthieu Bouron > > ff_h264_decode_extradata is referenced by h264_parse.h and not > h264dec.h. > --- > libavcodec/mediacodecdec_h264.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/me

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

On 07/09/16 14:33, Timo Rothenpieler wrote: 0RGB/0BGR does not mean the alpha bits are zeroed. It means they are undefined, so you convert from ARGB to 0RGB by doing nothing. There is no performance to gain by supporting a format that falsely advertises support for an alpha channel. Also, the co

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

On 07/09/16 15:01, Sven C. Dack wrote: On 07/09/16 14:33, Timo Rothenpieler wrote: 0RGB/0BGR does not mean the alpha bits are zeroed. It means they are undefined, so you convert from ARGB to 0RGB by doing nothing. There is no performance to gain by supporting a format that falsely advertises sup

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

On 09/07/2016 04:51 AM, Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > doc/developer.texi | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/doc/developer.texi b/doc/developer.texi > index 4d3a7ae..51e3da7 100644 > --- a/doc/developer.texi > +++ b/doc/develop

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

On Wed, Sep 07, 2016 at 07:19:00AM -0700, Jonathan Campbell wrote: > On 09/07/2016 04:51 AM, Michael Niedermayer wrote: > > Signed-off-by: Michael Niedermayer > > --- > > doc/developer.texi | 6 ++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/doc/developer.texi b/doc/developer.te

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

On Wed, Sep 07, 2016 at 04:39:54PM +0200, Michael Niedermayer wrote: > On Wed, Sep 07, 2016 at 07:19:00AM -0700, Jonathan Campbell wrote: > > On 09/07/2016 04:51 AM, Michael Niedermayer wrote: > > > Signed-off-by: Michael Niedermayer > > > --- > > > doc/developer.texi | 6 ++ > > > 1 file cha

[FFmpeg-devel] lavc/mediacodec: add hevc decoder

Hello, The following patchset add support for hevc mediacodec decoding. The patchset applies on top of: * lavc/mediacodecdec_h264: move bsf variable declaration at the top of the function * lavc/mediacodecdec_h264: use h264_parse.h instead of h264dec.h which are going to be pushed in one

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

From: Matthieu Bouron --- libavcodec/hevc.h| 9 + libavcodec/hevc_ps.c | 27 +++ 2 files changed, 36 insertions(+) diff --git a/libavcodec/hevc.h b/libavcodec/hevc.h index be91010..6a3c750 100644 --- a/libavcodec/hevc.h +++ b/libavcodec/hevc.h @@ -387,6 +387

[FFmpeg-devel] [PATCH 2/2] lavc: add hevc mediacodec decoder

From: Matthieu Bouron --- configure | 3 + libavcodec/Makefile| 3 +- libavcodec/allcodecs.c | 2 + libavcodec/hevc_parse.c| 134 ++ libavcodec/hevc_

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

On Wed, Sep 07, 2016 at 02:27:07PM +0200, Michael Niedermayer wrote: > On Wed, Sep 07, 2016 at 08:03:20AM -0400, Ronald S. Bultje wrote: > > Hi, > > > > On Wed, Sep 7, 2016 at 7:51 AM, Michael Niedermayer > > wrote: > > > > > Signed-off-by: Michael Niedermayer > > > --- > > > doc/developer.tex

Re: [FFmpeg-devel] [PATCH] doc/developer: Add patchwork mentioning to "patch submission checklist"

On Wed, Sep 07, 2016 at 04:44:54PM +0200, Michael Niedermayer wrote: > On Wed, Sep 07, 2016 at 04:39:54PM +0200, Michael Niedermayer wrote: > > On Wed, Sep 07, 2016 at 07:19:00AM -0700, Jonathan Campbell wrote: > > > On 09/07/2016 04:51 AM, Michael Niedermayer wrote: > > > > Signed-off-by: Michael

Re: [FFmpeg-devel] [PATCH 1/2] tests/fate:Add FATE for hls_flags append option

2016-08-30 2:07 GMT+08:00 Michael Niedermayer : > On Mon, Aug 29, 2016 at 11:17:25PM +0800, Steven Liu wrote: > > 2016-08-29 22:18 GMT+08:00 Michael Niedermayer : > > > > > On Mon, Aug 29, 2016 at 07:52:23PM +0800, Steven Liu wrote: > > > > 2016-08-26 0:11 GMT+08:00 Michael Niedermayer >: > > > >

Re: [FFmpeg-devel] [PATCH 5/5] af_hdcd: tweak hdcd_analyze_prepare() a bit

applied On Mon, Sep 5, 2016 at 6:18 AM, Burt P wrote: > * use the actual sample rate > * use a more sensible frequency for the tone > * update fate test result > > Signed-off-by: Burt P > --- > libavfilter/af_hdcd.c | 9 ++--- > tests/fate/filter-audio.mak | 2 +- > 2 files changed, 7

Re: [FFmpeg-devel] [PATCH 4/5] af_hdcd: move decoding setup from init to config_input

applied -- Burt ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/5] af_hdcd: fix possible integer overflow

applied with fix: uint64_t sustain_reset = (uint64_t)cdt_ms * rate / 1000; Thank you, Michael. -- Burt On Tue, Sep 6, 2016 at 5:00 PM, Michael Niedermayer wrote: > On Mon, Sep 05, 2016 at 06:18:43AM -0500, Burt P wrote: >> Signed-off-by: Burt P >> --- >> libavfilter/af_hdcd.c | 9 --

Re: [FFmpeg-devel] [PATCH 1/5] af_hdcd: some types renamed to remove _t

applied -- Burt ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/5] af_hdcd: hdcd_analyze_gen() using int instead of float

applied as is for now. Thanks for the comment. I will look into a faster way of doing it. I only needed to change from the float version because it was giving different results for i686 than for amd64. I don't know why, exactly, but I do know that this version gives bit-perfect results everywhere

Re: [FFmpeg-devel] [PATCH 1/2] tests/fate:Add FATE for hls_flags append option

On Wed, Sep 07, 2016 at 11:10:48PM +0800, Steven Liu wrote: > 2016-08-30 2:07 GMT+08:00 Michael Niedermayer : > > > On Mon, Aug 29, 2016 at 11:17:25PM +0800, Steven Liu wrote: > > > 2016-08-29 22:18 GMT+08:00 Michael Niedermayer : > > > > > > > On Mon, Aug 29, 2016 at 07:52:23PM +0800, Steven Liu

Re: [FFmpeg-devel] [PATCH]lavc/avcodec: Improve av_parser_parse() documentation

2016-09-04 22:41 GMT+02:00 Michael Niedermayer : >> Fixes ticket #5809. >> --- >> libavcodec/avcodec.h |5 - >> 1 file changed, 4 insertions(+), 1 deletion(-) > > LGTM, but please wait a day or 2 to allow others to comment Patch applied. Thank you, Carl Eugen ___

Re: [FFmpeg-devel] [PATCH 1/2] tests/fate:Add FATE for hls_flags append option

2016-09-08 0:41 GMT+08:00 Michael Niedermayer : > On Wed, Sep 07, 2016 at 11:10:48PM +0800, Steven Liu wrote: > > 2016-08-30 2:07 GMT+08:00 Michael Niedermayer : > > > > > On Mon, Aug 29, 2016 at 11:17:25PM +0800, Steven Liu wrote: > > > > 2016-08-29 22:18 GMT+08:00 Michael Niedermayer > : > > > >

Re: [FFmpeg-devel] [PATCH]ffmpeg_opt: Suggest to use "file:..." if a protocol was not found

2016-09-06 12:57 GMT+02:00 Michael Niedermayer : >> Subject: [PATCH] ffmpeg_opt: Suggest to use "file:..." if a >> protocol was not found. >> >> Fixes Debian bug 785690. >> --- >> ffmpeg_opt.c |2 ++ >> 1 file changed, 2 insertions(+) > > LGTM > > please give others a day to comment before app

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

applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

Signed-off-by: Steven Liu --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 044854b..f7e8298 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -397,7 +397,7 @@ Muxers/Demuxers: gxf.c Reimar Doeffinger

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

On 07/09/16 17:52, Timo Rothenpieler wrote: applied ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel I've updated my git clone and it is now seg faulting. Sven $ ffmpeg -f x11grab -framerate 200

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

On Wed, 7 Sep 2016 13:31:37 +0200, Michael Niedermayer wrote: > Hi > > Most of the tickets on trac are about ffmpeg code. I belive this makes > it hard for non developers to find things to contribute to, yet at the > same time there are quite a few non devel tasks that need to be done > and that

[FFmpeg-devel] ffmpeg-memory leak

hello，Maybe I find a bug about the memory leak，I try to use the function“avformat_open_input”to open the file， after that，I use the function “avformat_close_input”，My memroy will increase several hundred KB，even several MB, if My media file's resolution is bigger, the memory leak will be bigg

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

Today, new commit landed in git [1], but third part of submitted patch is missing in that commit. Same part is missing in patchwork [2], too. Maybe committer redacted patch, and threw out unneeded third part. It looks like patchwork was unable to recognize handmade patch sent to ML. Patch author wa

Re: [FFmpeg-devel] [PATCH 2/2] lavc/mediacodecdec_h264: use h264_parse.h instead of h264dec.h

On 07.09.2016 15:38, Matthieu Bouron wrote: On Mon, Sep 05, 2016 at 03:57:54PM +0200, Matthieu Bouron wrote: From: Matthieu Bouron ff_h264_decode_extradata is referenced by h264_parse.h and not h264dec.h. --- libavcodec/mediacodecdec_h264.c | 2 +- 1 file changed, 1 insertion(+), 1 deletio

[FFmpeg-devel] requested feature

Hello just wondering if you guys would mind adding the ability to encode to gamecube adp format soon. thanks in advance for any help :) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 2/2] lavc/mediacodecdec_h264: use h264_parse.h instead of h264dec.h

sounds good thanks On Wed, Sep 7, 2016 at 2:19 PM, Thomas Volkert wrote: > > On 07.09.2016 15:38, Matthieu Bouron wrote: > >> On Mon, Sep 05, 2016 at 03:57:54PM +0200, Matthieu Bouron wrote: >> >>> From: Matthieu Bouron >>> >>> ff_h264_decode_extradata is referenced by h264_parse.h and not >>>

Re: [FFmpeg-devel] [PATCH 2/2] lavc/mediacodecdec_h264: use h264_parse.h instead of h264dec.h

On Wed, Sep 07, 2016 at 08:19:15PM +0200, Thomas Volkert wrote: > > On 07.09.2016 15:38, Matthieu Bouron wrote: > > On Mon, Sep 05, 2016 at 03:57:54PM +0200, Matthieu Bouron wrote: > > > From: Matthieu Bouron > > > > > > ff_h264_decode_extradata is referenced by h264_parse.h and not > > > h264de

[FFmpeg-devel] [PATCH] avformat/hlsenc: implement program_date_time

TODO: docs, version bump Signed-off-by: Michael Niedermayer --- libavformat/hlsenc.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index c0b5ef2..a376312 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -64

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

On Wed, Sep 07, 2016 at 09:48:55AM -0800, Lou Logan wrote: > On Wed, 7 Sep 2016 13:31:37 +0200, Michael Niedermayer wrote: > > > Hi > > > > Most of the tickets on trac are about ffmpeg code. I belive this makes > > it hard for non developers to find things to contribute to, yet at the > > same ti

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

On Wed, Sep 07, 2016 at 09:43:26PM +0200, Michael Niedermayer wrote: > On Wed, Sep 07, 2016 at 09:48:55AM -0800, Lou Logan wrote: > > On Wed, 7 Sep 2016 13:31:37 +0200, Michael Niedermayer wrote: > > > > > Hi > > > > > > Most of the tickets on trac are about ffmpeg code. I belive this makes > > >

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

On 07/09/2016 18:48, Lou Logan wrote: If you want to use "task" for this then that would be fine with me. Alternatively, a wiki page could be setup similar to: Are all these on the trac? It might be worth adding a 'small' keyword

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

Actual allocation size is computed as (count + 1)*sizeof(meta_keys), so we need to check that (count + 1) won't cause overflow. From cfc0f5a099284c95476d5c020dca05fb743ff5ae Mon Sep 17 00:00:00 2001 From: Sergey Volk Date: Wed, 7 Sep 2016 14:05:35 -0700 Subject: [PATCH] Fix potential integer overf

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

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 wrote: > Actual allocation size is computed as (count + 1)*sizeof(meta_keys), so > we need to

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

Your patch was truncated before being applied [1], someone needs to revert and apply it properly. [1] - http://ffmpeg.org/pipermail/ffmpeg-devel/2016-September/199079.html ​ ___ ffmpeg-d

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

2016-09-08 3:34 GMT+08:00 Michael Niedermayer : > TODO: docs, version bump > > Signed-off-by: Michael Niedermayer > --- > libavformat/hlsenc.c | 23 +++ > 1 file changed, 23 insertions(+) > > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c > index c0b5ef2..a376312 10

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

On Wed, Sep 07, 2016 at 10:24:15PM +0100, Josh de Kock wrote: > On 07/09/2016 18:48, Lou Logan wrote: > >If you want to use "task" for this then that would be fine with me. > > > >Alternatively, a wiki page could be setup similar to: > >

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

On Wed, Sep 7, 2016 at 7:35 PM, Zoran Turalija wrote: > Today, new commit landed in git [1], but third part of submitted patch is > missing in that commit. Same part is missing in patchwork [2], too. Maybe > committer redacted patch, and threw out unneeded third part. It looks like > patchwork was

Re: [FFmpeg-devel] [PATCH v5 5/5] avformat/tee: Use BSF list API

On 09/05/2016 05:42 PM, Jan Sebechlebsky wrote: On 09/01/2016 09:42 PM, sebechlebsky...@gmail.com wrote: From: Jan Sebechlebsky Signed-off-by: Jan Sebechlebsky --- Changes since the last version of the patch: - added check for avcodec_parameters_copy return value - fixed stray space

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

On Thu, Sep 08, 2016 at 06:38:03AM +0800, Steven Liu wrote: > 2016-09-08 3:34 GMT+08:00 Michael Niedermayer : > > > TODO: docs, version bump > > > > Signed-off-by: Michael Niedermayer > > --- > > libavformat/hlsenc.c | 23 +++ > > 1 file changed, 23 insertions(+) > > > > diff

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

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 deleti

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

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 doesn't explain the seg. fault. This didn't happen in my patch

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

--- 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 -if(!run_as_d

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

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 insertions(+), 5 deletions(-) >> > 2dc146e807cbdbdbca653a22d827920e8c05b3c8

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

From: Aman Gupta --- doc/muxers.texi | 4 libavformat/hlsenc.c | 13 +++-- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index fd7ee50..e88dbf8 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -389,6 +389,10 @@ to @var{w

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

2016-09-08 9:47 GMT+08:00 Aman Gupta : > From: Aman Gupta > > --- > doc/muxers.texi | 4 > libavformat/hlsenc.c | 13 +++-- > 2 files changed, 15 insertions(+), 2 deletions(-) > > diff --git a/doc/muxers.texi b/doc/muxers.texi > index fd7ee50..e88dbf8 100644 > --- a/doc/muxers

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

On Tuesday, September 6, 2016 1:56:57 PM CDT Ligverd Haer wrote: > В письме от вторник, 6 сентября 2016 г. 12:26:49 MSK пользователь Carl Eugen > > Attributes > > profiles > width > height > sar > frameRate > audioSamplingRate > mimeType > segmentProfiles > codecs > maximumSAPPeriod > startWithSA