Am 20.10.2016 um 04:18 schrieb 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 patc
Hi,
On Wed, Oct 19, 2016 at 5:11 PM, Michael Behrisch wrote:
> Signed-off-by: Michael Behrisch
> ---
> libavutil/log.h| 2 +-
> libavutil/pixfmt.h | 12 ++--
> 2 files changed, 7 insertions(+), 7 deletions(-)
FWIW I still have doubts about this.
Ronald
_
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 should use git
send-email, not just copy
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
> index 0acc1b9..f
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 o
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 69fae41..d5d2281 100644
> --- a/libavformat
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 ?
either way, block_duration is f
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 version of Video
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 +
> compat/cuda/dynlink_nvcuvid.h | 316 +
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 Cadhalp
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 file changed, 3 insertions(+), 1 delet
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 +
> tests/ref/fate/concat-demuxer-extended-l
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
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
@@ -44,7 +44,7 @@ typede
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 insertions(+), 1 deletion(-)
> crea
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
consideratio
On Wed, Oct 19, 2016 at 07:27:59PM +0200, Andreas Cadhalpun wrote:
> 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)
> > retu
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 +47,7 @@ static int dcstr_read_header(A
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.
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 this
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 a
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
--- a/libavformat/aiffdec.c
+++ b/libavformat/aiffdec.c
@@
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
> -
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 correspo
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 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
___
ffmpeg-devel mailing list
ffm
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.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/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 b
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 license with no such pe
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 #inc
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.
[
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 100644
--- a/libavfil
Signed-off-by: Muhammad Faiz
---
doc/filters.texi | 5 +
libavfilter/avf_showcqt.c | 45 ++---
libavfilter/avf_showcqt.h | 2 ++
3 files changed, 41 insertions(+), 11 deletions(-)
diff --git a/doc/filters.texi b/doc/filters.texi
index 14c9b0
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 +
>>> compat/cuda/dynlink_cuviddec.h | 808
>>> +
>> +/*
>> + * 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 optio
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 m
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 regarding Nvidia stuff (1. check for temp aq,
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 +
compat/cud
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 +
> compat/cuda/dynlink_nvcuvid.h | 316 +
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 at
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 +-
tests/ref/fate/concat-demuxer-extended-lavf-mxf_d10 |
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
create mode 100644 tests/ref/f
---
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
+++ b/
---
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 accele
---
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
+++ b/libavfilter/vf
---
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 "libavutil/co
---
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 in
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
---
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 compat/
>> 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
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 for scale npp), but
ther
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: 9FF2128B147EF6730BADF133611EC7
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 who would give up es
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 seems to be no maintaine
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 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 supported
>> 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'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 10:22, Timo Rothenpieler wrote:
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
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 sca
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}
> > * mediacodecdec_h2645.c -> mediacodecde
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
> > : AVSTREAM_
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 th
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, Oct 18, 2016 at 10:10 PM, Sasi Inguva <
isasi-at-google@ffmpeg.org> wrote:
> I am also happy to fix any breaking files, if they don't fall under the
> above category.
>
>
>
The change seems to have caused resulting files to have incorrect duration
in some cases (remuxing XDCAM material
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 with how the
> edit list support
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 insertions(+), 2 deletions(-)
>
appl
70 matches
Mail list logo
