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
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
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
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
@@
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
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
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
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.
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
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
---
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
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
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
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
>> 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
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
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}
> > *
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
> > :
---
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
+++
---
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
---
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
+++
---
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
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
---
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
---
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
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
---
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
>> 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
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.
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
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
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
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:
>> 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
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
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 +
>
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
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
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
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
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
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
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
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 +
>>>
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
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
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
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
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
From: Rodger Combs
Reviewed-by: Tobias Rapp
---
doc/ffprobe.xsd | 1 +
ffprobe.c | 13 +
tests/ref/fate/concat-demuxer-extended-lavf-mxf | 2 +-
>> +/*
>> + * 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
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 +
>
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
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
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
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.
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
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
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++) {
>
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
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
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
___
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.
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
>
64 matches
Mail list logo