Re: [libav-devel] [PATCH 3/8] file: Add an option for following a file that is being written

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:37) > Using this requires setting the rw_timeout option to make it > terminate, alternatively using the interrupt callback (if used via > the API). > --- > libavformat/file.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/file.c b

Re: [libav-devel] [PATCH 4/8] opt: Add const to av_opt_next in the next major bump

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:38) > From: Lukasz Marek > > Also add const to pointers in static functions within opt.c where > possible. > --- > Or can we freely add const here without waiting for a major bump? > I don't see any case where adding const actually would break third > part

Re: [libav-devel] [PATCH 2/8] avio: Add an option 'rw_timeout'

2016-03-19 Thread Anton Khirnov
Quoting Martin Storsjö (2016-03-18 13:01:36) > From: Andrey Utkin > > If set non-zero, this limits duration of the retry_transfer_wrapper() > loop, thus affecting ffurl_read*(), ffurl_write(). As soon as > one single byte is successfully received/transmitted, the timer > restarts. > > This has f

Re: [libav-devel] [PATCH 6/9] lavf: VAAPI scale filter

2016-03-19 Thread Mark Thompson
On 19/03/16 08:47, Anton Khirnov wrote: > Quoting Mark Thompson (2016-03-16 00:00:09) >> + >> +static int scale_vaapi_pipeline_uninit(ScaleVAAPIContext *ctx) >> +{ >> +if (ctx->va_context != VA_INVALID_ID) { > > Some tests reveal that these checks are not good, because this function > can be c

[libav-devel] [PATCH] Mark tables used only within their files as static

2016-03-19 Thread Diego Biurrun
--- libavcodec/dxtory.c| 6 +++--- libavcodec/escape130.c | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/dxtory.c b/libavcodec/dxtory.c index 0a6f331..01726b9 100644 --- a/libavcodec/dxtory.c +++ b/libavcodec/dxtory.c @@ -171,9 +171,9 @@ static int dxtory_dec

Re: [libav-devel] [PATCH] h264: Add missing ff_ prefix to public h264_init_dequant_tables()

2016-03-19 Thread Anton Khirnov
Quoting Diego Biurrun (2016-03-17 12:10:08) > On Wed, Mar 09, 2016 at 05:15:44PM +0100, Anton Khirnov wrote: > > > > Not sure if you noticed, but there are other symbols with the same > > problem: > > def_lru > > def_lru_555 > > def_lru_565 > > chroma_vals > > No, I didn't, how did you? nm -g --

Re: [libav-devel] [PATCH 7/9] lavc: VAAPI encode common infrastructure

2016-03-19 Thread Anton Khirnov
Quoting Mark Thompson (2016-03-16 00:00:39) > --- > configure | 2 + > libavcodec/Makefile | 1 + > libavcodec/vaapi_encode.c | 951 > ++ > libavcodec/vaapi_encode.h | 203 ++ > 4 files changed, 1157 insertions(+) > cr

Re: [libav-devel] [PATCH 7/9] lavc: VAAPI encode common infrastructure

2016-03-19 Thread Mark Thompson
On 17/03/16 10:47, Diego Biurrun wrote: > On Tue, Mar 15, 2016 at 11:00:39PM +, Mark Thompson wrote: >> --- a/configure >> +++ b/configure >> @@ -1874,6 +1875,7 @@ nvenc_deps_any="dlopen LoadLibrary" >> nvenc_extralibs='$ldl' >> qsvdec_select="qsv" >> qsvenc_select="qsv" >> +vaapi_encode_dep

[libav-devel] [PATCH 3/8] file: Add an option for following a file that is being written

2016-03-19 Thread Martin Storsjö
Using this requires setting the rw_timeout option to make it terminate, alternatively using the interrupt callback (if used via the API). --- libavformat/file.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/file.c b/libavformat/file.c index 4f581cf..8683c1b 100644 --- a/libav

Re: [libav-devel] [PATCH] Mark tables used only within their files as static

2016-03-19 Thread Luca Barbato
On 17/03/16 12:09, Diego Biurrun wrote: > --- > libavcodec/dxtory.c| 6 +++--- > libavcodec/escape130.c | 2 +- > 2 files changed, 4 insertions(+), 4 deletions(-) > Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mai

Re: [libav-devel] [PATCH 6/9] lavf: VAAPI scale filter

2016-03-19 Thread Anton Khirnov
Quoting Mark Thompson (2016-03-16 00:00:09) > + > +static int scale_vaapi_pipeline_uninit(ScaleVAAPIContext *ctx) > +{ > +if (ctx->va_context != VA_INVALID_ID) { Some tests reveal that these checks are not good, because this function can be called without the context/config ids ever being init

Re: [libav-devel] [PATCH 6/8] RFC: avio: Copy URLContext generic options into child URLContexts

2016-03-19 Thread Luca Barbato
On 18/03/16 13:01, Martin Storsjö wrote: > Suggestions on how to do this better are welcome. We could add flags to the options and have a consume mode that drops the options as they are found and a preserve mode that does not as alternative. lu ___ liba

Re: [libav-devel] [PATCH 6/9] lavf: VAAPI scale filter

2016-03-19 Thread Diego Biurrun
On Tue, Mar 15, 2016 at 11:00:09PM +, Mark Thompson wrote: > --- a/libavfilter/allfilters.c > +++ b/libavfilter/allfilters.c > @@ -108,6 +108,7 @@ void avfilter_register_all(void) > REGISTER_FILTER(TRANSPOSE, transpose, vf); > REGISTER_FILTER(TRIM, trim,

[libav-devel] [PATCH 5/8] opt: Add av_opt_copy()

2016-03-19 Thread Martin Storsjö
From: Michael Niedermayer This includes documentation and other modifications by Lukasz Marek. --- doc/APIchanges | 3 +++ libavutil/opt.c | 58 + libavutil/opt.h | 13 libavutil/version.h | 2 +- 4 files changed, 75

[libav-devel] [PATCH 1/8] avio: Apply avoptions on the URLContext itself as well

2016-03-19 Thread Martin Storsjö
Currently the list of avoptions for URLContext is empty though, but such options will be added. --- libavformat/avio.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/avio.c b/libavformat/avio.c index 668e5ea..4da6b74 100644 --- a/libavformat/avio.c +++ b/libavformat/avio.c @@ -

<    1   2