Re: [FFmpeg-devel] lavf: Remove duplicated latm demuxer

Paul B Mahol gmail.com> writes: > > Attached patch reduces code duplication. > > (There was a API-relevant change in git right now.) > > > > Please comment, Carl Eugen > > Duplicate how? Replaced by what? The "latm" demuxer actually works on loas file and there already was a loas demuxer. I

Re: [FFmpeg-devel] [PATCH] avutil: install des.h, rc4.h and tree.h as public headers

Le duodi 22 vendémiaire, an CCXXIV, James Almer a écrit : > Since these two were never installed, we can remove the FF_API_CRYPTO_CONTEXT > wrapper from them before applying this patch, as this would be the first time > the API becomes public. > > Now, what i want to know is what will it be in

Re: [FFmpeg-devel] [PATCHv2 2/2] avfilter/formats: add av_warn_unused_result to function prototypes

On Mon, Oct 12, 2015 at 7:44 AM, Ganesh Ajjanagadde wrote: > On Mon, Oct 12, 2015 at 3:11 AM, Nicolas George wrote: >> Le nonidi 19 vendémiaire, an CCXXIV, Ganesh Ajjanagadde a écrit : >>> This uses the av_warn_unused_result attribute liberally to catch some

Re: [FFmpeg-devel] [PATCH] avutil/intmath: use de Bruijn based ff_ctz

On Wed, Oct 14, 2015 at 8:34 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Oct 12, 2015 at 7:14 AM, Ganesh Ajjanagadde > wrote: > >> On Mon, Oct 12, 2015 at 12:37 AM, James Almer wrote: >> > On 10/11/2015 10:55 PM, Ganesh Ajjanagadde

[FFmpeg-devel] [PATCHv2] avutil/intmath: use de Bruijn based ff_ctz

It has already been demonstrated that the de Bruijn method has benefits over the current implementation: commit 971d12b7f9d7be3ca8eb98e6c04ed521f83cbd3c. That commit implemented it for long long, this extends it to the int version. Tested with FATE. Signed-off-by: Ganesh Ajjanagadde

[FFmpeg-devel] avfilter_graph Question

Currently using: commit af5917698bd44f136fd0ff00a9e5f8b5f92f2d58 Author: Michael Niedermayer Date: Sun May 17 01:34:35 2015 +0200 I am trying to set up a small filter graph in order to use the YADIF implementation in libavfilter... I have set up an avfilter_graph with 3

Re: [FFmpeg-devel] [PATCHv3 1/2] avfilter/all: propagate errors of functions from avfilter/formats

On Wed, Oct 14, 2015 at 7:57 AM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 7:47 AM, Michael Niedermayer > wrote: >> On Tue, Oct 13, 2015 at 08:33:33PM -0400, Ganesh Ajjanagadde wrote: >>> Hi all, >>> >>> Attached is a new version reflecting all

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

Le primidi 11 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > >>Ah! So, in other words, it takes input frames, totally disregads their > >>pts/dts, and puts each one into a cfr output stream at the given fps? > >> > >>That solves a lot of problems and ridiculous math. :-) I will try it. > > Can't

[FFmpeg-devel] [PATCH] avfilter/af_tremolo: clean up extra newlines

Signed-off-by: Kyle Swanson --- libavfilter/af_tremolo.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavfilter/af_tremolo.c b/libavfilter/af_tremolo.c index 0ae7a7c..50df2e4 100644 --- a/libavfilter/af_tremolo.c +++ b/libavfilter/af_tremolo.c @@ -131,9 +131,7 @@ static

Re: [FFmpeg-devel] [PATCH]vc1 Autodetection

Michael Niedermayer niedermayer.cc> writes: > > Makefile |2 - > > rawdec.c |4 --- > > vc1dec.c | 81 +++ > > version.h |2 - > > 4 files changed, 83 insertions(+), 6 deletions(-) > > bdbacfecbe65093e8c50876d7ffa6eb9b8719a9d

Re: [FFmpeg-devel] Integration of RTMFP

Hi all, I would like to know what do I need to do if I want to share my "librtmfp" wrapper (something that should looks like this one ). Is there someone I should contact? Are there licensing prerequisites for the external

Re: [FFmpeg-devel] request for feedback on video codec idea

On Wed, Oct 14, 2015 at 8:08 PM, Roger Pack wrote: > Lacking a better place to debate this, I would like to ask some > questions on a video codec idea... > > The goal is basically to create a very fast lossless screen capture > codec (i.e. in the input there will be lots of

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

Dana 14. 10. 2015. 16:20 osoba "Nicolas George" napisala je: > > Le primidi 11 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > > >>Ah! So, in other words, it takes input frames, totally disregads their > > >>pts/dts, and puts each one into a cfr output stream at the given fps? >

[FFmpeg-devel] [PATCH] dnxhd: interleave AC levels and flags

This allows more efficient access to the array as the level and flags are contiguous. Around 4% faster coefficient decoding. --- Not the cleanest, and may complicate adding new profiles. But DNxHR just got added, so it should be ok for some years. --- libavcodec/dnxhddata.c | 492

[FFmpeg-devel] [PATCH] aac: Make codec init run under ff_thread_once

This makes AAC init threadsafe. Signed-off-by: Derek Buitenhuis --- This relies on the latest merge from Libav, which hasn't happened yet. --- libavcodec/aacdec.c | 2 ++ libavcodec/aacdec_fixed.c| 1 + libavcodec/aacdec_template.c | 80

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: rename loop option to loop2

On Wed, Oct 14, 2015 at 2:23 PM, Michael Niedermayer wrote: > From: Michael Niedermayer > > The "loop" option is used in several demuxers (like img2dec) and muxers, > using the same name in ffmpeg_opt > breaks them. Feel free to revert this and replace

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

Le tridi 23 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > This changes pts but framerate remains same. Care to ellaborate? Regards, -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] avfilter_graph Question

Dana 14. 10. 2015. 17:36 osoba "Grady Player" < grady.pla...@verizondigitalmedia.com> napisala je: > > Currently using: > > commit af5917698bd44f136fd0ff00a9e5f8b5f92f2d58 > Author: Michael Niedermayer > Date: Sun May 17 01:34:35 2015 +0200 > > I am trying to set up a small

Re: [FFmpeg-devel] [PATCHv2] avutil/intmath: use de Bruijn based ff_ctz

Hi, On Wed, Oct 14, 2015 at 10:26 AM, Ganesh Ajjanagadde wrote: > It has already been demonstrated that the de Bruijn method has benefits > over the current implementation: commit > 971d12b7f9d7be3ca8eb98e6c04ed521f83cbd3c. > That commit implemented it for long long,

Re: [FFmpeg-devel] avfilter_graph Question

Yes, sorry this was omitted - after I am done I call: av_frame_free(); it seems to be something that is internally referenced by the graph... On Oct 14, 2015, at 10:50 AM, Paul B Mahol wrote: > Dana 14. 10. 2015. 17:36 osoba "Grady Player" < >

[FFmpeg-devel] request for feedback on video codec idea

Lacking a better place to debate this, I would like to ask some questions on a video codec idea... The goal is basically to create a very fast lossless screen capture codec (i.e. in the input there will be lots of repeated "colors" of neighboring pixels, not a lot of dynamic content between

Re: [FFmpeg-devel] [PATCH] avutil: install des.h, rc4.h and tree.h as public headers

On 10/14/2015 2:16 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Oct 14, 2015 at 10:50 AM, Nicolas George wrote: > >> Le duodi 22 vendémiaire, an CCXXIV, James Almer a écrit : >>> Since these two were never installed, we can remove the >> FF_API_CRYPTO_CONTEXT >>> wrapper from

Re: [FFmpeg-devel] [PATCH] winrt: multithreading support

0001-winrt-multithreading-support.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] ffmpeg_opt: rename loop option to loop2

From: Michael Niedermayer The "loop" option is used in several demuxers (like img2dec) and muxers, using the same name in ffmpeg_opt breaks them. Feel free to revert this and replace by any other solution or rename both as preferred This is just as a quick fix to avoid

[FFmpeg-devel] [PATCH] fate: use PROGSSUF

May require exporting in the shell var PROGSUF when invoking a shell script. --- tests/Makefile | 14 +- tests/fate-run.sh| 38 +++--- tests/fate/ffprobe.mak | 2 +- tests/fate/filter-video.mak | 4 ++--

Re: [FFmpeg-devel] [PATCHv2] avutil/intmath: use de Bruijn based ff_ctz

On Wed, Oct 14, 2015 at 1:13 PM, Ronald S. Bultje wrote: > Hi, > > On Wed, Oct 14, 2015 at 10:26 AM, Ganesh Ajjanagadde > wrote: >> >> It has already been demonstrated that the de Bruijn method has benefits >> over the current implementation: commit >>

Re: [FFmpeg-devel] request for feedback on video codec idea

Le tridi 23 vendémiaire, an CCXXIV, Roger Pack a écrit : > For instance, given 3 frames of video ("one after another" from the > incoming video stream), "combine them" into one stream like: > pixel 1 frame 1, pixel 1 frame 2, pixel 1 frame 3, pixel 2 frame 2, > pixel 2 frame 2, pixel 2 frame 3 ...

Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration

2015-10-14 14:31 GMT+08:00 Timo Rothenpieler : > Hi >> >> I have check the error, for some card, yes, just like what I said above, >> some cards does not support double type, so we must check the opencl >> extension: cl_khr_fp64 before the kernel compile. If the device

Re: [FFmpeg-devel] [PATCH 3/3][RFC] avfilter/vf_chromakey: Add OpenCL acceleration

Hi I have check the error, for some card, yes, just like what I said above, some cards does not support double type, so we must check the opencl extension: cl_khr_fp64 before the kernel compile. If the device support, we use GPU, else, we use CPU or use float type, Is it OK? Thanks Best regards

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

Ganesh Ajjanagadde mit.edu> writes: > >> I think the general case, it'd be nice to figure out > >> why Carl's results are slightly different from yours > > > > Why do you think they are different at all? > > Did you look at the tables? > > They are different, and our conclusions are different

[FFmpeg-devel] [PATCH 1/2] avutil/fifo: add function av_fifo_generic_peek_at()

--- libavutil/fifo.c| 55 + libavutil/fifo.h| 11 +++ tests/ref/fate/fifo | 27 ++ 3 files changed, 93 insertions(+) diff --git a/libavutil/fifo.c b/libavutil/fifo.c index 07fb4ec..a7da591 100644 ---

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

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 thing. These > w> have special APIs to set

Re: [FFmpeg-devel] [PATCH 2/2] avformat/async: cache some data for fast seek backward

On Tue, Oct 13, 2015 at 06:30:47PM +0800, Zhang Rui wrote: > --- > libavformat/async.c | 123 > > 1 file changed, 104 insertions(+), 19 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] request for feedback on video codec idea

2015-10-14 20:08 GMT+02:00 Roger Pack : > I have become aware of some "fast" compression tools like LZO, LZ4, > density, etc. It seems like they all basically compress "the first > 64KB then the next 64KB" or something like that [1]. It's generally the size of a window or

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: rename loop option to loop2

On Wed, Oct 14, 2015 at 3:08 PM, Michael Niedermayer wrote: > On Wed, Oct 14, 2015 at 02:33:30PM -0400, Ganesh Ajjanagadde wrote: >> On Wed, Oct 14, 2015 at 2:23 PM, Michael Niedermayer >> wrote: >> > From: Michael Niedermayer

Re: [FFmpeg-devel] request for feedback on video codec idea

On Wed, Oct 14, 2015 at 9:04 PM, Paul B Mahol wrote: > Dana 14. 10. 2015. 20:42 osoba "Hendrik Leppkes" > napisala je: >> >> On Wed, Oct 14, 2015 at 8:08 PM, Roger Pack wrote: >> > Lacking a better place to debate this, I would like

Re: [FFmpeg-devel] request for feedback on video codec idea

On Wed, Oct 14, 2015 at 12:08:58 -0600, Roger Pack wrote: > The goal is basically to create a very fast lossless screen capture > codec (i.e. in the input there will be lots of repeated "colors" of > neighboring pixels, not a lot of dynamic content between frames). Have you benchmarked existing

Re: [FFmpeg-devel] avfilter_graph Question

On Wed, Oct 14, 2015 at 4:13 PM, Grady Player wrote: > Figured it out, sharing (if anyone were to care)… > > I was using > av_buffersrc_add_frame > rather than: > av_buffersrc_write_frame > > which informs the buffer to retain an internal reference

Re: [FFmpeg-devel] request for feedback on video codec idea

Dana 14. 10. 2015. 20:42 osoba "Hendrik Leppkes" napisala je: > > On Wed, Oct 14, 2015 at 8:08 PM, Roger Pack wrote: > > Lacking a better place to debate this, I would like to ask some > > questions on a video codec idea... > > > > The goal is

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: rename loop option to loop2

On Wed, Oct 14, 2015 at 02:33:30PM -0400, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 2:23 PM, Michael Niedermayer wrote: > > From: Michael Niedermayer > > > > The "loop" option is used in several demuxers (like img2dec) and muxers, > > using

Re: [FFmpeg-devel] avfilter_graph Question

Figured it out, sharing (if anyone were to care)… I was using av_buffersrc_add_frame rather than: av_buffersrc_write_frame which informs the buffer to retain an internal reference to the original picture/frame.. -Grady On Oct 14, 2015, at 11:50 AM, Grady Player

Re: [FFmpeg-devel] [PATCH] winrt: multithreading support

On Thu, Oct 15, 2015 at 02:46:21AM +0800, Wang Bin wrote: > > compat/w32pthreads.h |5 + > configure|7 +++ > libavutil/cpu.c |8 +++- > 3 files changed, 19 insertions(+), 1 deletion(-) > 252925c34c8d2b6f09ce0fd40c2923494b005a7a >

Re: [FFmpeg-devel] avfilter_graph Question

the doxygen, docs dont really point to any difference… (the prototype for av_buffersrc_add_frame doesn’t match my header, not sure if that is a version issue) int av_buffersrc_write_frame( AVFilterContext * s, const AVFrame * frame) Add a

[FFmpeg-devel] [PATCH] w32pthreads: fix mingw build on x86 with -msse2 or higher

When SSE2 or higher compiler optimizations are used, mingw uses the _mm_mfence intrinsic for MemoryBarrier, however it doesn't include the appropriate headers automatically. --- compat/w32pthreads.h | 5 + 1 file changed, 5 insertions(+) diff --git a/compat/w32pthreads.h

Re: [FFmpeg-devel] [PATCH 1/4] opencl: Print error string when compilation fails

On Tue, Oct 13, 2015 at 09:48:35PM -0700, Timothy Gu wrote: > --- > libavutil/opencl.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) LGTM thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Rewriting code that is poorly written but fully

Re: [FFmpeg-devel] [PATCH 1/2] vp9: split header into separate struct and expose in vp9.h

On Wed, Oct 14, 2015 at 10:16 AM, Hendrik Leppkes wrote: > From: "Ronald S. Bultje" > > This allows hwaccels to access the bitstream header information. > --- > libavcodec/vp9.c | 717 > +++ >

[FFmpeg-devel] [PATCH 3/4] avformat: add vag demuxer

Signed-off-by: Paul B Mahol --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/vag.c| 83 3 files changed, 85 insertions(+) create mode 100644 libavformat/vag.c diff --git

Re: [FFmpeg-devel] [PATCH] ffmpeg_opt: rename loop option to loop2

Michael Niedermayer niedermayer.cc> writes: > > quick alternative (based on help text) - stream_loop in ffmpeg_opt? > > ok, renamed and applied Thank you both! Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] request for feedback on video codec idea

On Wed, 14 Oct 2015 12:08:58 -0600 Roger Pack wrote: > Lacking a better place to debate this, I would like to ask some > questions on a video codec idea... > > The goal is basically to create a very fast lossless screen capture > codec (i.e. in the input there will be

[FFmpeg-devel] [PATCH 2/4] avformat: add genh demuxer

Signed-off-by: Paul B Mahol --- libavformat/Makefile | 1 + libavformat/allformats.c | 1 + libavformat/genh.c | 176 +++ 3 files changed, 178 insertions(+) create mode 100644 libavformat/genh.c diff --git

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

Dana 14. 10. 2015. 18:50 osoba "Nicolas George" napisala je: > > Le tridi 23 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > > This changes pts but framerate remains same. > > Care to ellaborate? Timebase and stream framerate are same. > > Regards, > > -- > Nicolas George >

[FFmpeg-devel] [PATCH 1/4] avcodec: add ADPCM PSX decoder

Signed-off-by: Paul B Mahol --- libavcodec/Makefile | 1 + libavcodec/adpcm.c | 40 libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 +++ libavcodec/utils.c | 1 + 6 files

Re: [FFmpeg-devel] [PATCH 2/4] avformat: add genh demuxer

Dana 14. 10. 2015. 23:57 osoba "Carl Eugen Hoyos" napisala je: > > Hi! > > Paul B Mahol gmail.com> writes: > > > + case 1: > > + case 11: st->codec->bits_per_coded_sample = 4; > > +st->codec->codec_id = AV_CODEC_ID_ADPCM_IMA_WAV; > > > + case 3:

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

Le quartidi 24 vendémiaire, an CCXXIV, Paul B Mahol a écrit : > Timebase and stream framerate are same. Not at all. For Matroska, timebase is usually 1/1000; for MPEG-[PT]S, timebase is 1/9; timebase can also be set using settb. Regards, -- Nicolas George signature.asc Description:

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

On Wed, Oct 14, 2015 at 2:40 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: > >> >> I think the general case, it'd be nice to figure out >> >> why Carl's results are slightly different from yours >> > >> > Why do you think they are different at all? >> > Did

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

On Wednesday 14 October 2015 10:07:04 am Sven Dueking wrote: > Hi all, > > The attached patches adds the VPP as filter module to FFMpeg. > Looking forward to get feedback. Just a few comments, I expect you will get a comment explaining what you can't do;-) (Sorry for the partly broken

Re: [FFmpeg-devel] configure: Require libkvazaar < 0.7

On 2015-10-14 13:19, Carl Eugen Hoyos wrote: > Hi! > > Attached patch for release/2.8 is supposed to fix ticket #4925, completely > untested. > > Please comment, Carl Eugen > +enabled libkvazaar&& require_pkg_config "kvazaar < 0.7.0" kvazaar > kvazaar.h kvz_api_get There is an extra

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

On Wed, Oct 14, 2015 at 5:01 AM, Matt Oliver wrote: > On 14 October 2015 at 09:46, Ganesh Ajjanagadde wrote: > >> On Tue, Oct 13, 2015 at 9:12 AM, Ganesh Ajjanagadde >> wrote: >> > On Tue, Oct 13, 2015 at 4:02 AM, Clément Bœsch

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

On Wed, Oct 14, 2015 at 12:37 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 5:01 AM, Matt Oliver wrote: >> On 14 October 2015 at 09:46, Ganesh Ajjanagadde wrote: >> >>> On Tue, Oct 13, 2015 at 9:12 AM, Ganesh Ajjanagadde

[FFmpeg-devel] configure: Require libkvazaar < 0.7

Hi! Attached patch for release/2.8 is supposed to fix ticket #4925, completely untested. Please comment, Carl Eugen diff --git a/configure b/configure index b661157..c2b58e5 100755 --- a/configure +++ b/configure @@ -5240,7 +5240,7 @@ enabled libgsm&& { for gsm_hdr in "gsm.h"

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

Ganesh Ajjanagadde mit.edu> writes: > What? My numbers actually show that the new code may be faster - No, you are misunderstanding the numbers you posted. (Or I misunderstand them but nobody said so yet.) Highest runs are most relevant, skips have to be avoided (afaik). [...] > If you

Re: [FFmpeg-devel] [PATCH] fate: use PROGSSUF

On Wed, Oct 14, 2015 at 07:18:27PM +0200, Christophe Gisquet wrote: > May require exporting in the shell var PROGSUF when invoking a > shell script. > --- > tests/Makefile | 14 +- > tests/fate-run.sh| 38 +++--- >

Re: [FFmpeg-devel] [PATCH] dnxhd: interleave AC levels and flags

On Wed, Oct 14, 2015 at 07:20:49PM +0200, Christophe Gisquet wrote: > This allows more efficient access to the array as the level and flags > are contiguous. Around 4% faster coefficient decoding. > --- > Not the cleanest, and may complicate adding new profiles. But DNxHR > just got added, so it

Re: [FFmpeg-devel] avfilter_graph Question

On Wed, Oct 14, 2015 at 4:48 PM, Grady Player wrote: > the doxygen, docs dont really point to any difference… (the prototype for > av_buffersrc_add_frame doesn’t match my header, not sure if that is a version > issue) Must have been the

Re: [FFmpeg-devel] lavf: Remove duplicated latm demuxer

Carl Eugen Hoyos ag.or.at> writes: > I will commit tonight if there are no objections, Carl Eugen Patch applied. Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] fate: use PROGSSUF

On Thu, Oct 15, 2015 at 01:31:11AM +0200, Michael Niedermayer wrote: > On Wed, Oct 14, 2015 at 07:18:27PM +0200, Christophe Gisquet wrote: > > May require exporting in the shell var PROGSUF when invoking a > > shell script. > > --- > > tests/Makefile | 14 +- > >

Re: [FFmpeg-devel] [PATCH] avfilter/af_tremolo: clean up extra newlines

On Wed, Oct 14, 2015 at 09:56:48AM -0500, Kyle Swanson wrote: > Signed-off-by: Kyle Swanson > --- > libavfilter/af_tremolo.c | 2 -- > 1 file changed, 2 deletions(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Into a blind

Re: [FFmpeg-devel] [PATCH 2/3] simple_idct12: align C and x86

On Wed, Oct 14, 2015 at 08:46:22AM +0200, Christophe Gisquet wrote: > Hi, > > 2015-10-14 0:04 GMT+02:00 Michael Niedermayer : > > the ome and syserr values worsen by this > > I have mixed feelings about this, too. > > On the one hand, omse in any case says anyway none of

Re: [FFmpeg-devel] [PATCHv3 1/2] avfilter/all: propagate errors of functions from avfilter/formats

On Tue, Oct 13, 2015 at 08:33:33PM -0400, Ganesh Ajjanagadde wrote: > Hi all, > > Attached is a new version reflecting all freedback received so far. builds and passes fate here also small nitpick libavfilter/vsrc_life.c: In function ‘query_formats’: libavfilter/vsrc_life.c:420:9: warning:

Re: [FFmpeg-devel] [PATCHv3 1/2] avfilter/all: propagate errors of functions from avfilter/formats

On Wed, Oct 14, 2015 at 7:47 AM, Michael Niedermayer wrote: > On Tue, Oct 13, 2015 at 08:33:33PM -0400, Ganesh Ajjanagadde wrote: >> Hi all, >> >> Attached is a new version reflecting all freedback received so far. > > builds and passes fate here > also small nitpick >

Re: [FFmpeg-devel] [PATCH] avutil/intmath: use de Bruijn based ff_ctz

Hi, On Mon, Oct 12, 2015 at 7:14 AM, Ganesh Ajjanagadde wrote: > On Mon, Oct 12, 2015 at 12:37 AM, James Almer wrote: > > On 10/11/2015 10:55 PM, Ganesh Ajjanagadde wrote: > >> It has already been demonstrated that the de Bruijn method has benefits > >>

Re: [FFmpeg-devel] [PATCH] avutil: install des.h, rc4.h and tree.h as public headers

On Wed, Oct 14, 2015 at 12:37:31AM +0200, Andreas Cadhalpun wrote: > These headers contain functions supposed to be public. > > libavutil/des.h: > av_des_alloc > av_des_crypt > av_des_init > av_des_mac > libavutil/rc4.h: > av_rc4_alloc > av_rc4_crypt > av_rc4_init > libavutil/tree.h >

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

> -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 > >

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

Carl Eugen Hoyos ag.or.at> writes: > > +AV_PIX_FMT_RGB32 , > > Trailing whitespace that cannot be committed to our repo, There is no trailing whitespace, just an (unintended?) whitespace, sorry. > please use tools/patcheck. This is still a good idea! Carl Eugen

Re: [FFmpeg-devel] [PATCH] avutil/intmath: use de Bruijn based ff_ctz

On Mon, Oct 12, 2015 at 7:14 AM, Ganesh Ajjanagadde wrote: > On Mon, Oct 12, 2015 at 12:37 AM, James Almer wrote: >> On 10/11/2015 10:55 PM, Ganesh Ajjanagadde wrote: >>> It has already been demonstrated that the de Bruijn method has benefits >>> over the

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

On Wed, Oct 14, 2015 at 6:49 AM, Carl Eugen Hoyos wrote: > Ganesh Ajjanagadde mit.edu> writes: > >> What? My numbers actually show that the new code may be faster - > > No, you are misunderstanding the numbers you posted. > (Or I misunderstand them but nobody said so yet.) > >

Re: [FFmpeg-devel] [PATCH] avfilter: add setfps filter

On Tue, Sep 29, 2015 at 23:39:10 +0200, Paul B Mahol wrote: > +static const AVOption setfps_options[] = { > +{ "fps", "A string describing desired output framerate", > OFFSET(framerate), AV_OPT_TYPE_VIDEO_RATE, { .str = "25" }, .flags = V|F }, I know this is taken 1:1 from the fps filter,

Re: [FFmpeg-devel] configure: Require libkvazaar < 0.7

Arttu Ylä-Outinen tut.fi> writes: > > Attached patch for release/2.8 is supposed to fix > > ticket #4925, completely untested. > > +enabled libkvazaar&& > > require_pkg_config "kvazaar < 0.7.0" kvazaar kvazaar.h kvz_api_get > > There is an extra "kvazaar" between 0.7.0 and kvazaar.h.

[FFmpeg-devel] lavf: Remove duplicated latm demuxer

Hi! Attached patch reduces code duplication. (There was a API-relevant change in git right now.) Please comment, Carl Eugen diff --git a/libavformat/Makefile b/libavformat/Makefile index 2971912..953b55d 100644 --- a/libavformat/Makefile +++ b/libavformat/Makefile @@ -227,7 +227,6 @@

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

> -Ursprüngliche Nachricht- > Von: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] Im Auftrag > von Carl Eugen Hoyos > Gesendet: Mittwoch, 14. Oktober 2015 14:41 > An: ffmpeg-devel@ffmpeg.org > Betreff: Re: [FFmpeg-devel] [PATCH] Added QSV based VPP filter > > Carl Eugen Hoyos

Re: [FFmpeg-devel] [PATCH] avutil: install des.h, rc4.h and tree.h as public headers

Hi, On Wed, Oct 14, 2015 at 10:50 AM, Nicolas George wrote: > Le duodi 22 vendémiaire, an CCXXIV, James Almer a écrit : > > Since these two were never installed, we can remove the > FF_API_CRYPTO_CONTEXT > > wrapper from them before applying this patch, as this would be the

Re: [FFmpeg-devel] [PATCHv2] avutil/intmath: use de Bruijn based ff_ctz

Hi, On Wed, Oct 14, 2015 at 1:26 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 1:13 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Wed, Oct 14, 2015 at 10:26 AM, Ganesh Ajjanagadde > > wrote: > >> > >> It has

Re: [FFmpeg-devel] lavf: Remove duplicated latm demuxer

On 10/14/15, Carl Eugen Hoyos wrote: > Hi! > > Attached patch reduces code duplication. > (There was a API-relevant change in git right now.) > > Please comment, Carl Eugen > Duplicate how? Replaced by what? ___ ffmpeg-devel mailing

Re: [FFmpeg-devel] [PATCH 1/2] avutil/fifo: add function av_fifo_generic_peek_at()

On Wed, Oct 14, 2015 at 02:20:07PM +0800, Zhang Rui wrote: > --- > libavutil/fifo.c| 55 > + > libavutil/fifo.h| 11 +++ > tests/ref/fate/fifo | 27 ++ > 3 files changed, 93 insertions(+) > > diff --git

[FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

This patch results in identical behavior of movenc, and suppresses -Wstrict-overflow warnings observed in GCC 5.2: http://fate.ffmpeg.org/log.cgi?time=20150926231053=compile=x86_64-archlinux-gcc-threads-misc, "warning: assuming signed overflow does not occur when assuming that (X - c) > X is

Re: [FFmpeg-devel] [PATCH] avformat/movenc: suppress -Wstrict-overflow warnings

On Mon, Oct 12, 2015 at 12:00 PM, Ganesh Ajjanagadde wrote: > On Mon, Oct 12, 2015 at 11:47 AM, Michael Niedermayer > wrote: >> On Mon, Oct 12, 2015 at 11:16:00AM -0400, Ganesh Ajjanagadde wrote: >>> On Mon, Oct 12, 2015 at 10:54 AM, Ronald S. Bultje

[FFmpeg-devel] [PATCH] doc/ffmpeg: use stream_loop instead of loop

Commit dbb03b8e47f905f0f7b10c3eb5ac11f9bfc793ff renamed loop to stream_loop. This documents the change. Signed-off-by: Ganesh Ajjanagadde --- doc/ffmpeg.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/ffmpeg.texi b/doc/ffmpeg.texi index

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg: use stream_loop instead of loop

On Wed, Oct 14, 2015 at 9:54 PM, Ganesh Ajjanagadde wrote: > Commit dbb03b8e47f905f0f7b10c3eb5ac11f9bfc793ff renamed loop to > stream_loop. This documents the change. > > Signed-off-by: Ganesh Ajjanagadde > --- > doc/ffmpeg.texi | 2 +- > 1 file

[FFmpeg-devel] [PATCH] avfilter/avfiltergraph: fix -Wunused-result warnings

Commit bf0d2d6030c239f91e0368a20fb2dc0705bfec99 introduced av_warn_unused_result to avfilter/formats, whose associated warnings were mostly fixed in 6aaac24d72a7da631173209841a3944fcb4a3309. This fixes the issues in avfilter/avfiltergraph. Tested with FATE. Signed-off-by: Ganesh Ajjanagadde

[FFmpeg-devel] [PATCH 1/2] avfilter/internal: Doxygen for ff_fmt_is_in

This clarifies and Doxygen's the comment for ff_fmt_is_in. Signed-off-by: Ganesh Ajjanagadde --- libavfilter/internal.h | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/libavfilter/internal.h b/libavfilter/internal.h index bb94707..c07d306

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg: use stream_loop instead of loop

On Wed, Oct 14, 2015 at 9:55 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 9:54 PM, Ganesh Ajjanagadde > wrote: >> Commit dbb03b8e47f905f0f7b10c3eb5ac11f9bfc793ff renamed loop to >> stream_loop. This documents the change. >> >>

[FFmpeg-devel] [PATCH] mips: disable all features in configure if no cpu is matched

Signed-off-by: Vicente Olivert Riera --- configure |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 7e55e92..cd27ad7 100755 --- a/configure +++ b/configure @@ -4130,6 +4130,15 @@ elif enabled mips; then

Re: [FFmpeg-devel] [PATCH 08/10] tools/crypto_bench: add AES-192 and AES-256

Le primidi 21 vendémiaire, an CCXXIV, Rodger Combs a écrit : > --- > tools/crypto_bench.c | 82 > > 1 file changed, 82 insertions(+) LGTM Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH 10/10] tools/crypto_bench: add support for multiple lavu versions by cpuflag

Le primidi 21 vendémiaire, an CCXXIV, Rodger Combs a écrit : > --- > tools/crypto_bench.c | 51 +++ > 1 file changed, 43 insertions(+), 8 deletions(-) > > diff --git a/tools/crypto_bench.c b/tools/crypto_bench.c > index 0b1bfc8..f84e2a7 100644 >

Re: [FFmpeg-devel] [PATCH] doc/ffmpeg: use stream_loop instead of loop

On Wed, Oct 14, 2015 at 8:08 PM Ganesh Ajjanagadde wrote: > pushed. Please let me know if such things are ok or not in the future > - technically I guess doc maintainers need to respond before I can > push, but as this change was trivial I thought I would do it.\ > As a

Re: [FFmpeg-devel] [PATCH 1/2] avfilter/internal: Doxygen for ff_fmt_is_in

On Wed, Oct 14, 2015 at 8:05 PM Ganesh Ajjanagadde wrote: > This clarifies and Doxygen's the comment for ff_fmt_is_in. ^ gajjanag.c:1:23: error: unexpected token ‘adjective’ > > Signed-off-by: Ganesh Ajjanagadde > --- >

[FFmpeg-devel] [libavformat rtsp] RTSP protocol not sending the ending TEARDOWN packet

When a client close a rtsp connexion, it is supposed to send the TEARDOWN packet according to ​https://en.wikipedia.org/wiki/Real_Time_Streaming_Protocol to tell to the server to stop the streaming. ffmpeg doesn't (controlled by wireshark, works with some other soft), while i guess it worked at a