Re: [FFmpeg-devel] [PATCH] avutil/qsort: use do while form for AV_QSORT

2015-10-22 Thread Clément Bœsch
On Thu, Oct 22, 2015 at 10:53:10PM -0400, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > libavutil/qsort.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavutil/qsort.h b/libavutil/qsort.h > index 30edcc8..f079661 100644 > --- a/libavutil/q

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter

2015-10-22 Thread Sven Dueking
> -Ursprüngliche Nachricht- > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > von Michael Niedermayer > Gesendet: Donnerstag, 22. Oktober 2015 17:39 > An: FFmpeg development discussions and patches > Betreff: Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter >

Re: [FFmpeg-devel] [PATCH] Don't needlessly reinitialize ff_cos_## tables.

2015-10-22 Thread Dale Curtis
On Wed, Oct 21, 2015 at 6:52 PM, Dale Curtis wrote: > On Tue, Oct 20, 2015 at 11:50 PM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: >> >> the last element to be written should be checked, so that if >> initialization is done by 2 threads at the same time, neither can >> return from thi

Re: [FFmpeg-devel] [PATCH] vf_psnr/ssim: don't crash if stats_file is NULL.

2015-10-22 Thread Paul B Mahol
Dana 23. 10. 2015. 00:01 osoba "Ronald S. Bultje" napisala je: > > --- > libavfilter/vf_psnr.c | 24 +--- > libavfilter/vf_ssim.c | 24 +--- > 2 files changed, 26 insertions(+), 22 deletions(-) > > diff --git a/libavfilter/vf_psnr.c b/libavfilter/vf_psnr.c

[FFmpeg-devel] [PATCH] avutil/qsort: use do while form for AV_QSORT

2015-10-22 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- libavutil/qsort.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavutil/qsort.h b/libavutil/qsort.h index 30edcc8..f079661 100644 --- a/libavutil/qsort.h +++ b/libavutil/qsort.h @@ -27,7 +27,7 @@ * to construct input that require

Re: [FFmpeg-devel] [PATCH] avcodec/huffman: replace qsort with AV_QSORT

2015-10-22 Thread Ganesh Ajjanagadde
On Thu, Oct 22, 2015 at 9:28 PM, Timothy Gu wrote: > On Thu, Oct 22, 2015 at 5:01 PM Ganesh Ajjanagadde > wrote: >> >> Sample benchmark (x86-64, Haswell, GNU/Linux), fraps-v2 from FATE: >> new: >> 280110 decicycles in qsort, 1 runs, 0 skips >> 268260 decicycles in qsort, 2 runs,

Re: [FFmpeg-devel] [PATCH] avcodec/huffman: replace qsort with AV_QSORT

2015-10-22 Thread Timothy Gu
On Thu, Oct 22, 2015 at 5:01 PM Ganesh Ajjanagadde wrote: > Sample benchmark (x86-64, Haswell, GNU/Linux), fraps-v2 from FATE: > new: > 280110 decicycles in qsort, 1 runs, 0 skips > 268260 decicycles in qsort, 2 runs, 0 skips > > old: > 1469910 decicycles in qsort, 1 r

[FFmpeg-devel] [PATCH] avcodec/huffman: replace qsort with AV_QSORT

2015-10-22 Thread Ganesh Ajjanagadde
ff_huff_build_tree uses qsort underneath. AV_QSORT is substantially faster due to the inlining of the comparison callback. Furthermore, this code is reasonably performance critical, since in e.g the fraps codec, ff_huff_build_tree is called on every frame. This routine is also called in vp6 on ever

Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread Derek Buitenhuis
On 10/23/2015 12:19 AM, wm4 wrote: > Wrong, snprintf() always returns the number of characters the string > would have been, even if the buffer size is smaller. That'll teach me to reply past midnight. I am dumb at night. > Also, shouldn't this use some av_ wrapper? What about locale issues? > S

Re: [FFmpeg-devel] [PATCH] legal: remove Legal Threats section

2015-10-22 Thread Lou Logan
On Thu, 22 Oct 2015 15:19:02 -0800, Lou Logan wrote: > Signed-off-by: Lou Logan > --- > src/legal | 11 --- > 1 file changed, 11 deletions(-) Pushed. Thanks. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listi

Re: [FFmpeg-devel] [PATCH] legal: remove Legal Threats section

2015-10-22 Thread Ronald S. Bultje
Hi, On Thu, Oct 22, 2015 at 7:19 PM, Lou Logan wrote: > Signed-off-by: Lou Logan > --- > > I'm tired of looking at it, being reminded of such things, and it has > served its purpose long ago. > > --- > src/legal | 11 --- > 1 file changed, 11 deletions(-) > > diff --git a/src/legal b/s

Re: [FFmpeg-devel] [PATCH] legal: remove Legal Threats section

2015-10-22 Thread wm4
On Fri, 23 Oct 2015 00:22:45 +0100 Kieran Kunhya wrote: > On 23 October 2015 at 00:19, Lou Logan wrote: > > Signed-off-by: Lou Logan > > --- > > > > I'm tired of looking at it, being reminded of such things, and it has > > served its purpose long ago. > > > > --- > > src/legal | 11 ---

Re: [FFmpeg-devel] [PATCH] legal: remove Legal Threats section

2015-10-22 Thread Kieran Kunhya
On 23 October 2015 at 00:19, Lou Logan wrote: > Signed-off-by: Lou Logan > --- > > I'm tired of looking at it, being reminded of such things, and it has > served its purpose long ago. > > --- > src/legal | 11 --- > 1 file changed, 11 deletions(-) > > diff --git a/src/legal b/src/legal >

[FFmpeg-devel] [PATCH] legal: remove Legal Threats section

2015-10-22 Thread Lou Logan
Signed-off-by: Lou Logan --- I'm tired of looking at it, being reminded of such things, and it has served its purpose long ago. --- src/legal | 11 --- 1 file changed, 11 deletions(-) diff --git a/src/legal b/src/legal index c235d31..1037a69 100644 --- a/src/legal +++ b/src/legal @@ -1

Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread wm4
On Fri, 23 Oct 2015 00:10:20 +0100 Derek Buitenhuis wrote: > On 10/22/2015 11:04 PM, Tinglin Liu wrote: > > +} else if (data_type == 23 && str_size >= 4) { // BE float32 > > +union av_intfloat32 val; > > +val.i = avio_rb32(pb); > > I found we have a function to

[FFmpeg-devel] [PATCH] WIP: lavf/segment: provide a virtual AVIOContext representing all the segments

2015-10-22 Thread Rodger Combs
This allows the use of muxers like matroska, which attempt to seek even when an AVIOContext doesn't set `seekable`, without concern for a rouge seek leading the muxer to overwrite the wrong data in a later segment. --- doc/muxers.texi | 17 libavformat/segment.c | 263 +

Re: [FFmpeg-devel] [PATCH] web/src/legal: minor typo and grammer fixes

2015-10-22 Thread Ganesh Ajjanagadde
On Thu, Oct 22, 2015 at 7:11 PM, Lou Logan wrote: > On Thu, 22 Oct 2015 18:46:04 -0400, Ganesh Ajjanagadde wrote: > >> Signed-off-by: Ganesh Ajjanagadde >> --- >> src/legal | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/src/legal b/src/legal >> index c235d31..7be

Re: [FFmpeg-devel] [PATCH] web/src/legal: minor typo and grammer fixes

2015-10-22 Thread Ganesh Ajjanagadde
On Thu, Oct 22, 2015 at 7:00 PM, Kieran Kunhya wrote: > Yeah...sometimes it's best to just let the past lie there. I'd rather > see it gone completely to be honest . I am in no position to judge this. Feel free to apply, remove the thing, whatever - just wanted to point out the minor typos if som

Re: [FFmpeg-devel] [PATCH] web/src/legal: minor typo and grammer fixes

2015-10-22 Thread Lou Logan
On Thu, 22 Oct 2015 18:46:04 -0400, Ganesh Ajjanagadde wrote: > Signed-off-by: Ganesh Ajjanagadde > --- > src/legal | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/legal b/src/legal > index c235d31..7be0291 100644 > --- a/src/legal > +++ b/src/legal > @@ -162,9

Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread Derek Buitenhuis
On 10/22/2015 11:04 PM, Tinglin Liu wrote: > +} else if (data_type == 23 && str_size >= 4) { // BE float32 > +union av_intfloat32 val; > +val.i = avio_rb32(pb); I found we have a function to to this: av_int2float(). > +if (snprintf(str, str_size_alloc,

Re: [FFmpeg-devel] [PATCH 02/11] avdevice/pulse_audio_common: add av_warn_unused_result

2015-10-22 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 12:45 PM, Ganesh Ajjanagadde wrote: > On Sat, Oct 17, 2015 at 12:36 PM, Michael Niedermayer > wrote: >> On Thu, Oct 15, 2015 at 10:22:16PM -0400, Ganesh Ajjanagadde wrote: >>> This does not trigger any warnings, but adds robustness. >>> >>> Signed-off-by: Ganesh Ajjanagadd

Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread Derek Buitenhuis
On 10/22/2015 11:28 PM, Tinglin Liu wrote: > 2) The key and value are stored separately for each key-value pair. 'keys' > atom stores the key name table, while 'ilst' atom stores the values > corresponding to the indices in the key table. And since they are stored in > two different atoms, I have t

Re: [FFmpeg-devel] [PATCH] web/src/legal: minor typo and grammer fixes

2015-10-22 Thread Kieran Kunhya
Yeah...sometimes it's best to just let the past lie there. I'd rather see it gone completely to be honest . Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] web/src/legal: minor typo and grammer fixes

2015-10-22 Thread Ganesh Ajjanagadde
Signed-off-by: Ganesh Ajjanagadde --- src/legal | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/legal b/src/legal index c235d31..7be0291 100644 --- a/src/legal +++ b/src/legal @@ -162,9 +162,9 @@ vigilant and diligent about collecting for MPEG-related technologies.

[FFmpeg-devel] [PATCH] avcodec/hap: set bits_per_coded_sample

2015-10-22 Thread Tom Butterworth
fixes issue where alpha is ignored in some players --- libavcodec/hapenc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/hapenc.c b/libavcodec/hapenc.c index 7daadce..cb5dcfa 100644 --- a/libavcodec/hapenc.c +++ b/libavcodec/hapenc.c @@ -242,16 +242,19 @@ static av_cold int hap

Re: [FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Hendrik Leppkes
On Thu, Oct 22, 2015 at 11:48 PM, Thierry Foucu wrote: > On Thu, Oct 22, 2015 at 2:29 PM, Hendrik Leppkes > wrote: > >> On Thu, Oct 22, 2015 at 11:15 PM, Thierry Foucu wrote: >> > On Thu, Oct 22, 2015 at 2:08 PM, Hendrik Leppkes >> > wrote: >> > >> >> On Thu, Oct 22, 2015 at 9:19 PM, Thierry Fo

Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread Tinglin Liu
Thank you Derek for the reply. Basically the metadata is stored in the structure like: |--meta |hdlr |keys |ilst 1) Firstly, we check if the handler type in the metadata handler atom is ‘mdta’. If it is, we set found_hdlr_mdta in ​ MOVContext be 1. 2) The key and value are stored se

Re: [FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Thierry Foucu
On Thu, Oct 22, 2015 at 2:29 PM, Hendrik Leppkes wrote: > On Thu, Oct 22, 2015 at 11:15 PM, Thierry Foucu wrote: > > On Thu, Oct 22, 2015 at 2:08 PM, Hendrik Leppkes > > wrote: > > > >> On Thu, Oct 22, 2015 at 9:19 PM, Thierry Foucu > wrote: > >> > --- > >> > libavformat/riff.c | 1 + > >> >

[FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread Tinglin Liu
The Apple dev specification: https://developer.apple.com/library/mac/documentation/QuickTime/QTFF/Metadata/Metadata.html Basically the structure is like: |--meta |hdlr |keys |ilst 1) The handler type in the metadata handler atom is ‘mdta’. 2) The key and value are stored separately fo

[FFmpeg-devel] [PATCH] vf_psnr/ssim: don't crash if stats_file is NULL.

2015-10-22 Thread Ronald S. Bultje
--- libavfilter/vf_psnr.c | 24 +--- libavfilter/vf_ssim.c | 24 +--- 2 files changed, 26 insertions(+), 22 deletions(-) diff --git a/libavfilter/vf_psnr.c b/libavfilter/vf_psnr.c index ca41939..36bbab2 100644 --- a/libavfilter/vf_psnr.c +++ b/libavfilter/v

[FFmpeg-devel] [PATCH] vf_psnr: don't crash if stats_file is NULL.

2015-10-22 Thread Ronald S. Bultje
--- libavfilter/vf_psnr.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/libavfilter/vf_psnr.c b/libavfilter/vf_psnr.c index ca41939..36bbab2 100644 --- a/libavfilter/vf_psnr.c +++ b/libavfilter/vf_psnr.c @@ -193,17 +193,19 @@ static av_cold int init

Re: [FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Thierry Foucu
On Thu, Oct 22, 2015 at 2:29 PM, Hendrik Leppkes wrote: > On Thu, Oct 22, 2015 at 11:15 PM, Thierry Foucu wrote: > > On Thu, Oct 22, 2015 at 2:08 PM, Hendrik Leppkes > > wrote: > > > >> On Thu, Oct 22, 2015 at 9:19 PM, Thierry Foucu > wrote: > >> > --- > >> > libavformat/riff.c | 1 + > >> >

Re: [FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Hendrik Leppkes
On Thu, Oct 22, 2015 at 11:15 PM, Thierry Foucu wrote: > On Thu, Oct 22, 2015 at 2:08 PM, Hendrik Leppkes > wrote: > >> On Thu, Oct 22, 2015 at 9:19 PM, Thierry Foucu wrote: >> > --- >> > libavformat/riff.c | 1 + >> > 1 file changed, 1 insertion(+) >> > >> > diff --git a/libavformat/riff.c b/l

Re: [FFmpeg-devel] [PATCH 1/1] avcodec/png: read and write stereo3d frame side data information

2015-10-22 Thread Gonzalo
El 22/10/15 a las 17:50, Kirill Gavrilov escribió: Use optional sTER chunk defining side-by-side stereo pair within "Extensions to the PNG 1.2 Specification", version 1.3.0. Specification: http://www.libpng.org/pub/png/spec/register/pngext-1.4.0-pdg.html Note that patch does not add any image

Re: [FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Thierry Foucu
On Thu, Oct 22, 2015 at 1:45 PM, Paul B Mahol wrote: > On 10/22/15, Thierry Foucu wrote: > > --- > > libavformat/riff.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/libavformat/riff.c b/libavformat/riff.c > > index a9197e7..7de8936 100644 > > --- a/libavformat/riff.c > > +++ b

Re: [FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Thierry Foucu
On Thu, Oct 22, 2015 at 2:08 PM, Hendrik Leppkes wrote: > On Thu, Oct 22, 2015 at 9:19 PM, Thierry Foucu wrote: > > --- > > libavformat/riff.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/libavformat/riff.c b/libavformat/riff.c > > index a9197e7..7de8936 100644 > > --- a/libav

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

2015-10-22 Thread Ganesh Ajjanagadde
On Thu, Oct 22, 2015 at 4:51 PM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: > >> To put an end to a long and tortuous thread, and >> due to the lack of relevant outstanding objections, >> pushed. > > To sum it up: > Several developers have explained to you that the > numbers yo

Re: [FFmpeg-devel] Using ffmpeg library

2015-10-22 Thread Lou Logan
On Thu, 22 Oct 2015 16:04:05 -0400, mohamed larbi gharib wrote: > Hi, > > Thank you for your help. > > I would like to get information on what library shall I use for parsing > data from a transport stream file MPEG2/h264. > > I would like to use ffprobe tools but as functions in my program (wr

Re: [FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Hendrik Leppkes
On Thu, Oct 22, 2015 at 9:19 PM, Thierry Foucu wrote: > --- > libavformat/riff.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/riff.c b/libavformat/riff.c > index a9197e7..7de8936 100644 > --- a/libavformat/riff.c > +++ b/libavformat/riff.c > @@ -487,5 +487,6 @@ const AVCod

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

2015-10-22 Thread Carl Eugen Hoyos
Ganesh Ajjanagadde mit.edu> writes: > To put an end to a long and tortuous thread, and > due to the lack of relevant outstanding objections, > pushed. To sum it up: Several developers have explained to you that the numbers you posted show that FFmpeg is now either slower or equally fast, you

[FFmpeg-devel] [PATCH 1/1] avcodec/png: read and write stereo3d frame side data information

2015-10-22 Thread Kirill Gavrilov
Use optional sTER chunk defining side-by-side stereo pair within "Extensions to the PNG 1.2 Specification", version 1.3.0. Specification: http://www.libpng.org/pub/png/spec/register/pngext-1.4.0-pdg.html Note that patch does not add any image resolution checks defined by spec. Regards, Kirill

Re: [FFmpeg-devel] [PATCH] vf_psnr: remove %0.2f format specifiers for stream summary line.

2015-10-22 Thread Paul B Mahol
On 10/22/15, Ronald S. Bultje wrote: > This makes output equally precise as vf_ssim. > --- > libavfilter/vf_psnr.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavfilter/vf_psnr.c b/libavfilter/vf_psnr.c > index a01b2c1..4f060c9 100644 > --- a/libavfilter/vf_psnr

Re: [FFmpeg-devel] [PATCH] vf_ssim: print per-channel dB values.

2015-10-22 Thread Paul B Mahol
On 10/22/15, Ronald S. Bultje wrote: > --- > libavfilter/vf_ssim.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavfilter/vf_ssim.c b/libavfilter/vf_ssim.c > index 5cd9361..e7a64f7 100644 > --- a/libavfilter/vf_ssim.c > +++ b/libavfilter/vf_ssim.c > @@ -346,7 +346,

Re: [FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Paul B Mahol
On 10/22/15, Thierry Foucu wrote: > --- > libavformat/riff.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/riff.c b/libavformat/riff.c > index a9197e7..7de8936 100644 > --- a/libavformat/riff.c > +++ b/libavformat/riff.c > @@ -487,5 +487,6 @@ const AVCodecGuid ff_codec_wav_

Re: [FFmpeg-devel] [PATCH] avfilter, swresample, swscale: use fabs, fabsf instead of FFABS

2015-10-22 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 7:53 AM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 7:30 AM, Michael Niedermayer > wrote: >> On Thu, Oct 15, 2015 at 06:38:10AM -0400, Ganesh Ajjanagadde wrote: >>> On Wed, Oct 14, 2015 at 6:53 AM, Hendrik Leppkes >>> wrote: >>> > On Wed, Oct 14, 2015 at 12:49 P

[FFmpeg-devel] Using ffmpeg library

2015-10-22 Thread mohamed larbi gharib
Hi, Thank you for your help. I would like to get information on what library shall I use for parsing data from a transport stream file MPEG2/h264. I would like to use ffprobe tools but as functions in my program (written in C) returning a structure with stream index codec_name codec_tag and code

[FFmpeg-devel] [PATCH] vf_psnr: remove %0.2f format specifiers for stream summary line.

2015-10-22 Thread Ronald S. Bultje
This makes output equally precise as vf_ssim. --- libavfilter/vf_psnr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_psnr.c b/libavfilter/vf_psnr.c index a01b2c1..4f060c9 100644 --- a/libavfilter/vf_psnr.c +++ b/libavfilter/vf_psnr.c @@ -322,10 +322,10 @@

[FFmpeg-devel] [PATCH] libavformat/riff.c: Adding support of PCM_S24LE in WAVEFORMATEXTENSIBLE

2015-10-22 Thread Thierry Foucu
--- libavformat/riff.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/riff.c b/libavformat/riff.c index a9197e7..7de8936 100644 --- a/libavformat/riff.c +++ b/libavformat/riff.c @@ -487,5 +487,6 @@ const AVCodecGuid ff_codec_wav_guids[] = { { AV_CODEC_ID_ATRAC3P, { 0xBF, 0xA

[FFmpeg-devel] [PATCH] vf_ssim: print per-channel dB values.

2015-10-22 Thread Ronald S. Bultje
--- libavfilter/vf_ssim.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavfilter/vf_ssim.c b/libavfilter/vf_ssim.c index 5cd9361..e7a64f7 100644 --- a/libavfilter/vf_ssim.c +++ b/libavfilter/vf_ssim.c @@ -346,7 +346,8 @@ static av_cold void uninit(AVFilterContext *ctx)

Re: [FFmpeg-devel] [PATCH] configure: add -Wstrict-prototypes when available

2015-10-22 Thread Ganesh Ajjanagadde
On Thu, Oct 22, 2015 at 2:45 PM, Hendrik Leppkes wrote: > On Thu, Oct 22, 2015 at 8:40 PM, Ganesh Ajjanagadde > wrote: >> GCC (and Clang) have this useful warning that is not enabled by -Wall or >> -Wextra. This will ensure that issues like those fixed in >> 4da52e3630343e8d3a79aef2cafcb6bf0b71e8

Re: [FFmpeg-devel] [PATCH] configure: add -Wstrict-prototypes when available

2015-10-22 Thread Hendrik Leppkes
On Thu, Oct 22, 2015 at 8:40 PM, Ganesh Ajjanagadde wrote: > GCC (and Clang) have this useful warning that is not enabled by -Wall or > -Wextra. This will ensure that issues like those fixed in > 4da52e3630343e8d3a79aef2cafcb6bf0b71e8da > will trigger warnings. > I did get this warning on GCC 5.2

Re: [FFmpeg-devel] [PATCH] Adds support parsing the QuickTime Metadata Keys.

2015-10-22 Thread Derek Buitenhuis
On 10/20/2015 7:29 PM, Tinglin Liu wrote: > The Apple dev specification: > https://developer.apple.com/library/mac/documentation/QuickTime/QTFF/Metadata/Metadata.html > --- > libavformat/isom.h | 3 +++ > libavformat/mov.c | 77 > +- > 2 files

[FFmpeg-devel] [PATCH] configure: add -Wstrict-prototypes when available

2015-10-22 Thread Ganesh Ajjanagadde
GCC (and Clang) have this useful warning that is not enabled by -Wall or -Wextra. This will ensure that issues like those fixed in 4da52e3630343e8d3a79aef2cafcb6bf0b71e8da will trigger warnings. Signed-off-by: Ganesh Ajjanagadde --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/c

Re: [FFmpeg-devel] [PATCH 04/11] libavformat/mxfdec.c: Try to increment current edit before rejecting a klv that spans onto next edit unit.

2015-10-22 Thread Alexis Ballier
On Wed, 21 Oct 2015 23:45:07 +0200 Tomas Härdin wrote: > On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > > Some files such as those from tickets #2817 & #2776 claim to have > > constant edit unit size but, in fact, have some of them that are > > smaller. This confuses the demuxer that

Re: [FFmpeg-devel] [PATCH 01/11] libavformat/mxfdec.c: klv_read_packet: Properly check klv_decode_ber_length return value.

2015-10-22 Thread Alexis Ballier
On Wed, 21 Oct 2015 23:31:29 +0200 Tomas Härdin wrote: > On Wed, 2015-10-21 at 18:00 +0200, Alexis Ballier wrote: > > klv_decode_ber_length cannot return -1, but can return > > AVERROR_INVALIDDATA. Store its return value in a signed integer > > (instead of unsigned KLVPacket.length) and forward t

Re: [FFmpeg-devel] [PATCH 5/5] ffmpeg: add abort_on option to allow aborting on empty output

2015-10-22 Thread Marton Balint
On Mon, 19 Oct 2015, Marton Balint wrote: On Mon, 19 Oct 2015, Michael Niedermayer wrote: On Sun, Oct 18, 2015 at 12:24:07AM +0200, Marton Balint wrote: Signed-off-by: Marton Balint --- doc/ffmpeg.texi | 8 ffmpeg.c| 7 +++ ffmpeg.h| 3 +++ ffmpeg_opt.c|

Re: [FFmpeg-devel] [PATCH] ffmpeg: fix ffmpeg.h trailing whitespace

2015-10-22 Thread Marton Balint
On Thu, 22 Oct 2015, Carl Eugen Hoyos wrote: Marton Balint passwd.hu> writes: How this passed through the commit hook? I'd also like to know but please push. Ok, pushed. Thanks, Marton ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org htt

Re: [FFmpeg-devel] [PATCH] WIP: lavf/segment: provide a virtual AVIOContext representing all the segments

2015-10-22 Thread Michael Niedermayer
On Thu, Oct 22, 2015 at 04:50:00AM -0500, Rodger Combs wrote: > This allows the use of muxers like matroska, which attempt to seek even > when an AVIOContext doesn't set `seekable`, without concern for a rouge > seek leading the muxer to overwrite the wrong data in a later segment. > --- > doc/mux

Re: [FFmpeg-devel] [PATCH] libavcodec/qsv.c: Re-design session control and internal allocation

2015-10-22 Thread Hendrik Leppkes
On Wed, Oct 14, 2015 at 11:47 AM, Hendrik Leppkes wrote: > On Wed, Oct 7, 2015 at 7:07 PM, Ivan Uskov wrote: >> Hello wm4, >> >> Wednesday, October 7, 2015, 7:40:45 PM, you wrote: >> >> w> There's no automagic way to get this done. >> >> w> Hardware accelerators like vaapi and vdpau need the same

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-22 Thread Michael Niedermayer
On Thu, Oct 22, 2015 at 08:53:07AM -0400, Ganesh Ajjanagadde wrote: > On Thu, Oct 22, 2015 at 8:42 AM, Ganesh Ajjanagadde wrote: > > On Thu, Oct 22, 2015 at 8:33 AM, Michael Niedermayer > > wrote: > >> On Thu, Oct 22, 2015 at 07:04:41AM -0400, Ganesh Ajjanagadde wrote: > >>> On Thu, Oct 22, 2015

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter

2015-10-22 Thread Michael Niedermayer
On Wed, Oct 21, 2015 at 11:26:30AM +0200, Sven Dueking wrote: > > > > -Ursprüngliche Nachricht- > > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > > von Moritz Barsnick > > Gesendet: Mittwoch, 14. Oktober 2015 11:23 > > An: FFmpeg development discussions and patch

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-22 Thread Ganesh Ajjanagadde
On Thu, Oct 22, 2015 at 8:42 AM, Ganesh Ajjanagadde wrote: > On Thu, Oct 22, 2015 at 8:33 AM, Michael Niedermayer > wrote: >> On Thu, Oct 22, 2015 at 07:04:41AM -0400, Ganesh Ajjanagadde wrote: >>> On Thu, Oct 22, 2015 at 5:49 AM, Carl Eugen Hoyos wrote: >>> > Ganesh Ajjanagadde gmail.com> writ

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-22 Thread Ganesh Ajjanagadde
On Thu, Oct 22, 2015 at 8:33 AM, Michael Niedermayer wrote: > On Thu, Oct 22, 2015 at 07:04:41AM -0400, Ganesh Ajjanagadde wrote: >> On Thu, Oct 22, 2015 at 5:49 AM, Carl Eugen Hoyos wrote: >> > Ganesh Ajjanagadde gmail.com> writes: >> > >> >> > This broke fate with -ftrapv >> > >> >> > The prob

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter

2015-10-22 Thread Sven Dueking
> -Ursprüngliche Nachricht- > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > von Hendrik Leppkes > Gesendet: Donnerstag, 22. Oktober 2015 14:26 > An: FFmpeg development discussions and patches > Betreff: Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter > > O

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-22 Thread Michael Niedermayer
On Thu, Oct 22, 2015 at 07:04:41AM -0400, Ganesh Ajjanagadde wrote: > On Thu, Oct 22, 2015 at 5:49 AM, Carl Eugen Hoyos wrote: > > Ganesh Ajjanagadde gmail.com> writes: > > > >> > This broke fate with -ftrapv > > > >> > The problem seems to be in av_gcd() and not the De-Bruijn version of > >> > f

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter

2015-10-22 Thread Hendrik Leppkes
On Thu, Oct 22, 2015 at 2:20 PM, Sven Dueking wrote: > > >> -Ursprüngliche Nachricht- >> Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag >> von Sven Dueking >> Gesendet: Donnerstag, 22. Oktober 2015 14:03 >> An: 'FFmpeg development discussions and patches' >> Betreff:

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter

2015-10-22 Thread Sven Dueking
> -Ursprüngliche Nachricht- > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > von Sven Dueking > Gesendet: Donnerstag, 22. Oktober 2015 14:03 > An: 'FFmpeg development discussions and patches' > Betreff: Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter > > >

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter

2015-10-22 Thread Sven Dueking
> -Ursprüngliche Nachricht- > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > von Hendrik Leppkes > Gesendet: Donnerstag, 22. Oktober 2015 14:00 > An: FFmpeg development discussions and patches > Betreff: Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter > > O

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter

2015-10-22 Thread Hendrik Leppkes
On Thu, Oct 22, 2015 at 1:29 PM, Sven Dueking wrote: > > >> -Ursprüngliche Nachricht- >> Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag >> von Carl Eugen Hoyos >> Gesendet: Mittwoch, 14. Oktober 2015 12:43 >> An: FFmpeg development discussions and patches >> Betreff:

Re: [FFmpeg-devel] Nvidia nevec presets

2015-10-22 Thread Timo Rothenpieler
Thanks for all , if you add "*Streaming*" *Presets* To Nvidia Nevec Presets , like this : Automatic Low Latency Low Latency 2Pass HQ Low Latency 2Pass Lossless HP lossless Thanks Not sure what you mean with this. The ll/llhq/hp/... presets are all available. You can also enable 2pass mode

Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter

2015-10-22 Thread Sven Dueking
> -Ursprüngliche Nachricht- > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > von Carl Eugen Hoyos > Gesendet: Mittwoch, 14. Oktober 2015 12:43 > An: FFmpeg development discussions and patches > Betreff: Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter > > On

Re: [FFmpeg-devel] [PATCH] libavformat/electronicarts: also demux mpeg audio layer 2

2015-10-22 Thread Michael Niedermayer
On Thu, Oct 22, 2015 at 05:47:33AM +0200, Paul B Mahol wrote: > On 10/22/15, Peter Ross wrote: > > Signed-off-by: Peter Ross > > > > http://wiki.multimedia.cx/index.php?title=Electronic_Arts_SCxl > > --- > > libavformat/electronicarts.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --gi

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-22 Thread Ganesh Ajjanagadde
On Thu, Oct 22, 2015 at 5:49 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde gmail.com> writes: > >> > This broke fate with -ftrapv > >> > The problem seems to be in av_gcd() and not the De-Bruijn version of >> > ff_ctzll since the gcc builtin is being used. >> >> Don't have the time to investig

Re: [FFmpeg-devel] Encoding with QSV at an API level.

2015-10-22 Thread Paul Knopf
That was it! Thanks very much! On Wed, Oct 21, 2015 at 4:38 PM, Hendrik Leppkes wrote: > On Wed, Oct 21, 2015 at 9:42 PM, Paul Knopf > wrote: > > I just realized that I was also getting an error on the stdout. > > > > [h264_qsv @ 00c7e6d8e680] Specified pixel format yuv420p is > > invalid o

Re: [FFmpeg-devel] [PATCH] ffmpeg: fix ffmpeg.h trailing whitespace

2015-10-22 Thread Carl Eugen Hoyos
Marton Balint passwd.hu> writes: > How this passed through the commit hook? I'd also like to know but please push. Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] WIP: lavf/segment: provide a virtual AVIOContext representing all the segments

2015-10-22 Thread Rodger Combs
This allows the use of muxers like matroska, which attempt to seek even when an AVIOContext doesn't set `seekable`, without concern for a rouge seek leading the muxer to overwrite the wrong data in a later segment. --- doc/muxers.texi | 17 libavformat/segment.c | 262 +

Re: [FFmpeg-devel] [PATCH] ffmpeg: fix ffmpeg.h trailing whitespace

2015-10-22 Thread Michael Niedermayer
On Thu, Oct 22, 2015 at 05:30:27AM +0200, Marton Balint wrote: > How this passed through the commit hook? > > Signed-off-by: Marton Balint > --- > ffmpeg.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC78704

Re: [FFmpeg-devel] [PATCHv2] avutil/mathematics: speed up av_gcd by using Stein's binary GCD algorithm

2015-10-22 Thread Carl Eugen Hoyos
Ganesh Ajjanagadde gmail.com> writes: > > This broke fate with -ftrapv > > The problem seems to be in av_gcd() and not the De-Bruijn version of > > ff_ctzll since the gcc builtin is being used. > > Don't have the time to investigate right now - revert for now unless > someone can fix it quickly

Re: [FFmpeg-devel] [PATCH][RFC] avcodec: disallow hwaccel with frame threads

2015-10-22 Thread Hendrik Leppkes
On Thu, Oct 22, 2015 at 11:27 AM, Wang Bin wrote: > VLC is using frame threading with hwaccel Then they should fix their usage, its broken by design, as explained in detail my post earlier in this thread. - Hendrik ___ ffmpeg-devel mailing list ffmpeg-

Re: [FFmpeg-devel] [PATCH][RFC] avcodec: disallow hwaccel with frame threads

2015-10-22 Thread Wang Bin
VLC is using frame threading with hwaccel ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCHv2] mmaldec: Add mpeg2 decoding support

2015-10-22 Thread wm4
On Wed, 21 Oct 2015 17:10:48 +0200 Julian Scheel wrote: > Register mmaldec as mpeg2 decoder. Supporting mpeg2 in mmaldec is just a > matter of setting the correct MMAL_ENCODING on the input port. To ease the > addition of further supported mmal codecs a macro is introduced to generate > the decod

Re: [FFmpeg-devel] [PATCH] avformat/electronicarts: support ADPCM PSX

2015-10-22 Thread Paul B Mahol
Dana 22. 10. 2015. 06:30 osoba "Peter Ross" napisala je: > > On Wed, Oct 21, 2015 at 11:14:02PM +0200, Paul B Mahol wrote: > > Signed-off-by: Paul B Mahol > > --- > > libavformat/electronicarts.c | 19 +++ > > 1 file changed, 15 insertions(+), 4 deletions(-) > > > > diff --git a/