Re: [FFmpeg-devel] [PATCH 3/3] fate: Add MXF D10/DV25 probe tests

2016-10-19 Thread Michael Niedermayer
On Wed, Oct 19, 2016 at 02:35:22PM +0200, Tobias Rapp wrote: > Signed-off-by: Tobias Rapp > --- > tests/fate/mxf.mak| 19 ++- > tests/ref/fate/mxf-probe-d10 | 3 +++ > tests/ref/fate/mxf-probe-dv25 | 4 > 3 files changed, 25

Re: [FFmpeg-devel] [PATCH 2/3] ffprobe: report field order for video streams

2016-10-19 Thread Michael Niedermayer
On Wed, Oct 19, 2016 at 02:35:21PM +0200, Tobias Rapp wrote: > From: Rodger Combs > > Reviewed-by: Tobias Rapp > --- > doc/ffprobe.xsd | 1 + > ffprobe.c | 13

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: init AVPacket::pos to FLVTAG offset

2016-10-19 Thread సుమన్
Thanks for taking time to review my patch! I appreciate it very much!! "Suman-" is my ID on github; Unless it is objectionable, I intend to use it for code submissions. Thanks! Suman On Wed, Oct 19, 2016 at 12:03 PM, Michael Niedermayer < mich...@niedermayer.cc> wrote: > On Tue, Oct 18, 2016

[FFmpeg-devel] [PATCH] removing comma at final enumeration items to fix pedantic warnings

2016-10-19 Thread Michael Behrisch
Signed-off-by: Michael Behrisch --- libavutil/log.h| 2 +- libavutil/pixfmt.h | 12 ++-- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/libavutil/log.h b/libavutil/log.h index 0acc1b9..f0a5738 100644 --- a/libavutil/log.h +++ b/libavutil/log.h @@

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Andreas Cadhalpun
On 19.10.2016 20:46, Sven C. Dack wrote: > No. This is exactly what I meant with wearing tin foil hats. Insults won't help you. > Just because a compiler includes information provided by header > files into the compilation process does this not imply a transfer > of ownership or copyright of

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Andreas Cadhalpun
On 19.10.2016 21:41, Sven C. Dack wrote: > You are misinterpreting it. The FAQ explicitly excludes external > components such as compilers, kernels and libraries. What is or is not a system component is quite debatable. However, it always depends on the specific operating system under

Re: [FFmpeg-devel] [PATCH] avformat: remove request_probe assert from ff_read_packet

2016-10-19 Thread Andreas Cadhalpun
On 19.10.2016 22:45, Michael Niedermayer wrote: > On Wed, Oct 19, 2016 at 07:27:59PM +0200, Andreas Cadhalpun wrote: >> f1f2b8b10efdab27848eeb2e2bac646eda08a175 >> 0001-avformat-prevent-triggering-request_probe-assert-in-.patch >> From 7912c6f200a37130844221a73941a7971afa6455 Mon Sep 17 00:00:00

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Sven C. Dack
On 19/10/16 20:23, Andreas Cadhalpun wrote: Read the FAQ: "What about the compiler, the toolchain?" Best regards, Andreas 1: http://gpl-violations.org/faq/sourcecode-faq/ You are misinterpreting it. The FAQ explicitly excludes external components such as compilers, kernels and libraries.

[FFmpeg-devel] [PATCH] rsd: limit number of channels

2016-10-19 Thread Andreas Cadhalpun
Negative values don't make sense and too large values can cause overflows. For AV_CODEC_ID_ADPCM_THP this leads to a too small extradata buffer being allocated, causing out-of-bounds writes. Signed-off-by: Andreas Cadhalpun --- libavformat/rsd.c | 4 +++- 1

[FFmpeg-devel] [PATCH] aiffdec: fix division by zero

2016-10-19 Thread Andreas Cadhalpun
This is similar to commit c143a9c. Signed-off-by: Andreas Cadhalpun --- libavformat/aiffdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/aiffdec.c b/libavformat/aiffdec.c index de82787..d96fc1d 100644 ---

[FFmpeg-devel] [PATCH] dcstr: fix division by zero

2016-10-19 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- libavformat/dcstr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/dcstr.c b/libavformat/dcstr.c index 69fae41..d5d2281 100644 --- a/libavformat/dcstr.c +++ b/libavformat/dcstr.c @@ -47,7

[FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Yogender Kumar Gupta
Added capability check for Temporal AQ Thanks, Yogender 0001-nvenc-Add-capability-check-for-Temporal-AQ.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] lavf/mov: Add support for edit list parsing.

2016-10-19 Thread wm4
On Tue, 18 Oct 2016 13:08:07 -0700 Sasi Inguva wrote: > With the current state of code, we will only encounterdemuxer > outputting non-monotonic DTS when there are multiple edit lists . Avoiding > non-monotonic DTS for mulitple edit lists will not be possible

Re: [FFmpeg-devel] [PATCH] avfilter/firequalizer: add cubic_interpolate function on gain

2016-10-19 Thread Muhammad Faiz
On 10/18/16, Muhammad Faiz wrote: > smoother version of gain_interpolate > > Signed-off-by: Muhammad Faiz > --- > doc/filters.texi | 2 ++ > libavfilter/af_firequalizer.c | 49 > +-- > 2 files changed, 49

[FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Yogender Kumar Gupta
>> I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3. limitation work-around for scale npp), but there seems to be no maintainer present at the moment to pick them up... Tempoal AQ HEVC not supported yet. Thanks, Yogender

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 12:00, Hendrik Leppkes wrote: On Wed, Oct 19, 2016 at 12:20 PM, Sven C. Dack wrote: On 19/10/16 11:01, Yogender Kumar Gupta wrote: I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3. limitation work-around

Re: [FFmpeg-devel] [PATCH 4/4] lavc/mediacodec: use more meaningful filenames

2016-10-19 Thread Matthieu Bouron
On Tue, Oct 18, 2016 at 03:26:46PM +0200, Matthieu Bouron wrote: > On Wed, Oct 12, 2016 at 03:22:42PM +0200, Matthieu Bouron wrote: > > From: Matthieu Bouron > > > > Adds the following changes: > > * mediacodecdec.{c,h} -> mediacodecdec_common.{c,h} > > *

Re: [FFmpeg-devel] [PATCH] lavf/img2dec: disable parsing if frame_size is specified

2016-10-19 Thread Matthieu Bouron
On Mon, Mar 07, 2016 at 11:59:38AM +, Carl Eugen Hoyos wrote: > Matthieu Bouron gmail.com> writes: > > > else { > > s->is_pipe = 1; > > -st->need_parsing = AVSTREAM_PARSE_FULL; > > +st->need_parsing = s->frame_size > 0 ? AVSTREAM_PARSE_NONE > > :

[FFmpeg-devel] [PATCH 6/8] avfilter/vf_hwupload_cuda: use new hwdevice allocation API

2016-10-19 Thread Timo Rothenpieler
--- libavfilter/vf_hwupload_cuda.c | 57 -- 1 file changed, 5 insertions(+), 52 deletions(-) diff --git a/libavfilter/vf_hwupload_cuda.c b/libavfilter/vf_hwupload_cuda.c index c1c..dba496f 100644 --- a/libavfilter/vf_hwupload_cuda.c +++

[FFmpeg-devel] [PATCH 7/8] avfilter/vf_hwupload_cuda: check ff_formats_ref for errors

2016-10-19 Thread Timo Rothenpieler
--- libavfilter/vf_hwupload_cuda.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_hwupload_cuda.c b/libavfilter/vf_hwupload_cuda.c index dba496f..14688df 100644 --- a/libavfilter/vf_hwupload_cuda.c +++ b/libavfilter/vf_hwupload_cuda.c @@ -54,6 +54,8

[FFmpeg-devel] [PATCH 5/8] avcodec/nvenc: use dynamically loaded CUDA

2016-10-19 Thread Timo Rothenpieler
--- configure | 1 + libavcodec/nvenc.c | 129 + libavcodec/nvenc.h | 46 ++- 3 files changed, 26 insertions(+), 150 deletions(-) diff --git a/configure b/configure index 8c5742f..78113e2 100755 --- a/configure +++

[FFmpeg-devel] [PATCH 8/8] configure: cuda is no longer nonfree, enable and autodetect by default

2016-10-19 Thread Timo Rothenpieler
--- configure | 24 ++-- 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/configure b/configure index 78113e2..b6a0276 100755 --- a/configure +++ b/configure @@ -299,8 +299,8 @@ External library support: The following libraries provide various hardware

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 11:01, Yogender Kumar Gupta wrote: I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3. limitation work-around for scale npp), but there seems to be no maintainer present at the moment to pick them up... Tempoal AQ HEVC not

[FFmpeg-devel] [PATCH 4/8] avfilter/vf_scale_npp: use dynamically loaded CUDA

2016-10-19 Thread Timo Rothenpieler
--- libavfilter/vf_scale_npp.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavfilter/vf_scale_npp.c b/libavfilter/vf_scale_npp.c index 35abe39..c84282f 100644 --- a/libavfilter/vf_scale_npp.c +++ b/libavfilter/vf_scale_npp.c @@ -29,7 +29,7 @@ #include

[FFmpeg-devel] [PATCH 2/8] avutil/hwcontext_cuda: use dynamically loaded CUDA

2016-10-19 Thread Timo Rothenpieler
--- configure | 2 + libavutil/hwcontext_cuda.c | 99 - libavutil/hwcontext_cuda.h | 5 ++ libavutil/hwcontext_cuda_internal.h | 37 ++ libavutil/version.h | 2 +- 5 files changed, 120

[FFmpeg-devel] [PATCH 3/8] avcodec/cuvid: use dynamically loaded CUDA/CUVID

2016-10-19 Thread Timo Rothenpieler
And remove the now obsolete compat headers. --- compat/cuda/cuviddec.h | 827 - compat/cuda/nvcuvid.h | 321 --- libavcodec/cuvid.c | 78 +++-- 3 files changed, 47 insertions(+), 1179 deletions(-) delete mode 100644

[FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Timo Rothenpieler
--- compat/cuda/dynlink_cuda.h | 88 + compat/cuda/dynlink_cuviddec.h | 808 + compat/cuda/dynlink_loader.h | 254 + compat/cuda/dynlink_nvcuvid.h | 316 4 files changed, 1466 insertions(+) create mode 100644

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Yogender Kumar Gupta
>> This patch LGTM, minus the one spurious space at the end of the if. What is the procedure to submit, do I do a git push Thanks, Yogender ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Timo Rothenpieler
I'm not exactly maintaining libnpp, I think it was merged from libav, but the patch looks fine regardless. No idea why I never got these mails, must have mass-marked them as read and then deleted all unread mails. Will apply the old patches as this one does not show up on patchwork.

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Hendrik Leppkes
On Wed, Oct 19, 2016 at 12:20 PM, Sven C. Dack wrote: > On 19/10/16 11:01, Yogender Kumar Gupta wrote: I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. >> >> option for temp aq in hevc, 3. limitation work-around for scale npp), but >> there

Re: [FFmpeg-devel] [PATCH] lavf/mov: support gopro hero moments udta tag

2016-10-19 Thread Michael Niedermayer
On Tue, Oct 18, 2016 at 03:47:55PM +0200, Jean Caillé wrote: > From: Jean Caillé > > --- > libavformat/mov.c | 19 +++ > 1 file changed, 19 insertions(+) applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Those

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 09:34, Yogender Kumar Gupta wrote: Added capability check for Temporal AQ Thanks, Yogender I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. option for temp aq in hevc, 3. limitation work-around for scale npp), but there seems to be no maintainer present at

Re: [FFmpeg-devel] [PATCH] avformat/movenc: Skip duration check for discontinuous fragments

2016-10-19 Thread Michael Niedermayer
On Tue, Oct 18, 2016 at 06:33:59PM +0200, Michael Niedermayer wrote: > Found-by: Daemon404 > Signed-off-by: Michael Niedermayer > --- > libavformat/movenc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) applied [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Yogender Kumar Gupta
>> Temporal AQ is implemented in CUDA, not in silicon. So far does the API (Video SDK 7.0.1) accept the flag without giving any errors. The output however remains identical currently (tested on a GTX 960). You'll have to ask Nvidia why they haven't implemented it yet (i'ts probably too soon). I am

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Timo Rothenpieler
Am 19.10.2016 um 11:15 schrieb Sven C. Dack: > On 19/10/16 09:34, Yogender Kumar Gupta wrote: >> Added capability check for Temporal AQ >> >> Thanks, >> Yogender > I've send in 3 patches regarding Nvidia stuff (1. check for temp aq, 2. > option for temp aq in hevc, 3. limitation work-around for

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Philip Langdale
On Wed, 19 Oct 2016 14:00:29 +0200 Timo Rothenpieler wrote: > --- > compat/cuda/dynlink_cuda.h | 88 + > compat/cuda/dynlink_cuviddec.h | 808 > + > compat/cuda/dynlink_loader.h | 254 + >

Re: [FFmpeg-devel] [PATCH] dcstr: fix division by zero

2016-10-19 Thread Michael Niedermayer
On Wed, Oct 19, 2016 at 10:41:22PM +0200, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavformat/dcstr.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/dcstr.c b/libavformat/dcstr.c > index

Re: [FFmpeg-devel] [PATCH] removing comma at final enumeration items to fix pedantic warnings

2016-10-19 Thread Michael Niedermayer
On Wed, Oct 19, 2016 at 11:11:21PM +0200, Michael Behrisch wrote: > Signed-off-by: Michael Behrisch > --- > libavutil/log.h| 2 +- > libavutil/pixfmt.h | 12 ++-- > 2 files changed, 7 insertions(+), 7 deletions(-) > > diff --git a/libavutil/log.h b/libavutil/log.h

Re: [FFmpeg-devel] [PATCH] aiffdec: fix division by zero

2016-10-19 Thread Michael Niedermayer
On Wed, Oct 19, 2016 at 09:18:51PM +0200, Andreas Cadhalpun wrote: > This is similar to commit c143a9c. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/aiffdec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) can aiff work without block_align

[FFmpeg-devel] [PATCH] lavc/videotoolbox: fix avcc creation for h264 streams missing extradata

2016-10-19 Thread Aman Gupta
From: Aman Gupta ff_videotoolbox_avcc_extradata_create() was never being called if avctx->extradata_size==0, even though the function does not need or use the avctx->extradata. This manifested itself only on h264 streams in specific containers, and only on iOS. I guess the macOS

Re: [FFmpeg-devel] [PATCH] rsd: limit number of channels

2016-10-19 Thread Michael Niedermayer
On Wed, Oct 19, 2016 at 11:46:43PM +0200, Andreas Cadhalpun wrote: > Negative values don't make sense and too large values can cause > overflows. For AV_CODEC_ID_ADPCM_THP this leads to a too small extradata > buffer being allocated, causing out-of-bounds writes. > > Signed-off-by: Andreas

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: zero initialize codec_name in mov_parse_stsd_video()

2016-10-19 Thread Michael Niedermayer
On Wed, Oct 19, 2016 at 07:30:29PM +0200, Andreas Cadhalpun wrote: > On 19.10.2016 04:15, James Almer wrote: > > On 10/17/2016 9:57 PM, Michael Niedermayer wrote: > >> On Sun, Oct 16, 2016 at 09:34:50PM -0300, James Almer wrote: > >>> Fixes valgrind warning about "Conditional jump or move depends

Re: [FFmpeg-devel] [PATCH] removing comma at final enumeration items to fix pedantic warnings

2016-10-19 Thread Muhammad Faiz
On 10/20/16, Michael Behrisch wrote: > Signed-off-by: Michael Behrisch > --- > libavutil/log.h| 2 +- > libavutil/pixfmt.h | 12 ++-- > 2 files changed, 7 insertions(+), 7 deletions(-) If you have difficulty to send your patch directly (you

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Hendrik Leppkes
On Wed, Oct 19, 2016 at 3:32 PM, Sven C. Dack wrote: > On 19/10/16 13:50, Hendrik Leppkes wrote: >> >> On Wed, Oct 19, 2016 at 2:00 PM, Timo Rothenpieler >> wrote: >>> >>> --- >>> compat/cuda/dynlink_cuda.h | 88 + >>>

[FFmpeg-devel] [PATCH 3/3] fate: Add MXF D10/DV25 probe tests

2016-10-19 Thread Tobias Rapp
Signed-off-by: Tobias Rapp --- tests/fate/mxf.mak| 19 ++- tests/ref/fate/mxf-probe-d10 | 3 +++ tests/ref/fate/mxf-probe-dv25 | 4 3 files changed, 25 insertions(+), 1 deletion(-) create mode 100644 tests/ref/fate/mxf-probe-d10

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Hendrik Leppkes
On Wed, Oct 19, 2016 at 1:30 PM, Sven C. Dack wrote: > On 19/10/16 12:00, Hendrik Leppkes wrote: >> >> On Wed, Oct 19, 2016 at 12:20 PM, Sven C. Dack >> wrote: >>> >>> On 19/10/16 11:01, Yogender Kumar Gupta wrote: >> >> I've send in 3 patches

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Sven C. Dack
On 19/10/16 13:50, Hendrik Leppkes wrote: On Wed, Oct 19, 2016 at 2:00 PM, Timo Rothenpieler wrote: --- compat/cuda/dynlink_cuda.h | 88 + compat/cuda/dynlink_cuviddec.h | 808 + compat/cuda/dynlink_loader.h | 254

[FFmpeg-devel] [PATCH v2 0/3] Detect field_order based on video_line_map

2016-10-19 Thread Tobias Rapp
This patch series adds fate tests for probing MXF D10 (top field first) and DV25 (bottom field first) file properties. Patch #2 is taken from https://ffmpeg.org/pipermail/ffmpeg-devel/2016-October/200607.html and has been augmented with the missing XSD update. The fate samples can be downloaded

[FFmpeg-devel] [PATCH 1/3] avformat/mxfdec: Detect field_order based on video_line_map

2016-10-19 Thread Tobias Rapp
Read video_line_map from MXF generic picture essence descriptor and use it to derive the coded field order. Use field_dominance to derive the display field order from coded field order. If field_dominance is not available the default value "1" is used as defined in SMPTE S377-1. Fixes field_order

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

2016-10-19 Thread Tobias Rapp
From: Rodger Combs Reviewed-by: Tobias Rapp --- doc/ffprobe.xsd | 1 + ffprobe.c | 13 + tests/ref/fate/concat-demuxer-extended-lavf-mxf | 2 +-

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Timo Rothenpieler
>> +/* >> + * This file is part of FFmpeg. >> + * >> + * FFmpeg is free software; you can redistribute it and/or >> + * modify it under the terms of the GNU Lesser General Public >> + * License as published by the Free Software Foundation; either >> + * version 2.1 of the License, or (at your

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Hendrik Leppkes
On Wed, Oct 19, 2016 at 2:00 PM, Timo Rothenpieler wrote: > --- > compat/cuda/dynlink_cuda.h | 88 + > compat/cuda/dynlink_cuviddec.h | 808 > + > compat/cuda/dynlink_loader.h | 254 + >

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Sven C. Dack
On 19/10/16 15:25, Hendrik Leppkes wrote: You seem to be referring to the license of the cuvid files, which is not the license actually used by the CUDA headers. The CUDA headers are under a full proprietary license with no such permission. Are you saying the use of CUDA header files (with

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Hendrik Leppkes
On Wed, Oct 19, 2016 at 6:07 PM, Sven C. Dack wrote: > On 19/10/16 15:25, Hendrik Leppkes wrote: >> >> You seem to be referring to the license of the cuvid files, which is >> not the license actually used by the CUDA headers. The CUDA headers >> are under a full proprietary

[FFmpeg-devel] [PATCH] avfilter/firequalizer: use zero phase kernel

2016-10-19 Thread Muhammad Faiz
this makes real-valued kernel Signed-off-by: Muhammad Faiz --- libavfilter/af_firequalizer.c | 26 +++--- 1 file changed, 15 insertions(+), 11 deletions(-) diff --git a/libavfilter/af_firequalizer.c b/libavfilter/af_firequalizer.c index e799e34..4988717

Re: [FFmpeg-devel] [PATCH] lavf: add ffprobe demuxer

2016-10-19 Thread Moritz Barsnick
On Tue, Oct 18, 2016 at 13:33:27 +0200, Stefano Sabatini wrote: > > Latest patch in attachment with ffprobe demuxer disabled by default, > > and extended documentation. (I'm also attaching the ff_get_line2 patch > > which is used by this new patch). > Updated, depends on the ff_get_line2() patch.

Re: [FFmpeg-devel] [PATCH]nvenc: Add capability check for Temporal AQ

2016-10-19 Thread Sven C. Dack
On 19/10/16 14:24, Hendrik Leppkes wrote: If its entirely untested (due to being unsupported), it should not be made available, and instead be added once it is supported and has been confirmed to be working properly. There is also no reason to add it if we know it doesn't work, it would just be

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mov: zero initialize codec_name in mov_parse_stsd_video()

2016-10-19 Thread Andreas Cadhalpun
On 19.10.2016 04:15, James Almer wrote: > On 10/17/2016 9:57 PM, Michael Niedermayer wrote: >> On Sun, Oct 16, 2016 at 09:34:50PM -0300, James Almer wrote: >>> Fixes valgrind warning about "Conditional jump or move depends on >>> uninitialised value(s)" >>> >>> Signed-off-by: James Almer

Re: [FFmpeg-devel] [PATCH] avformat: remove request_probe assert from ff_read_packet

2016-10-19 Thread Andreas Cadhalpun
On 19.10.2016 05:29, Michael Niedermayer wrote: > hmm, i guess the patch is then ok > alternatively you could use i think: > @@ -803,7 +803,7 @@ int ff_read_packet(AVFormatContext *s, AVPacket *pkt) > return ret; > for (i = 0; i < s->nb_streams; i++) { >

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Sven C. Dack
On 19/10/16 17:18, Hendrik Leppkes wrote: Thats the general interpretation of the license situation. If you include non-free headers, your binary becomes non-free, hence why building with cuda currently requires the --enable-nonfree option. No. This is a generalization and cannot make sense. At

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Sven C. Dack
On 19/10/16 18:56, Andreas Cadhalpun wrote: I think you are missing the main problem here: FFmpeg is licensed under the LGPL 2.1, which states [1]: " 4. You may copy and distribute the Library [...] in object code or executable form [...] provided that you accompany it with the complete

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 ___

Re: [FFmpeg-devel] [PATCH 1/8] compat/cuda: add dynamic loader

2016-10-19 Thread Andreas Cadhalpun
On 19.10.2016 18:52, Sven C. Dack wrote: > On 19/10/16 17:18, Hendrik Leppkes wrote: >> Thats the general interpretation of the license situation. If you >> include non-free headers, your binary becomes non-free, hence why >> building with cuda currently requires the --enable-nonfree option. > No.

Re: [FFmpeg-devel] [PATCH] lavf/flvdec: init AVPacket::pos to FLVTAG offset

2016-10-19 Thread Michael Niedermayer
On Tue, Oct 18, 2016 at 03:13:23PM -0700, Suman- wrote: > Current code doesn't initialize AVPacket::pos. Made it point to FLVTAG so > flv_read_packet can decode from pos > --- > libavformat/flvdec.c| 1 + > tests/ref/seek/acodec-adpcm-swf | 46 >