[libav-devel] [PATCH 14/25] intrax8: Keep a reference to the GetBitContext reader

2016-02-22 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 35 --- libavcodec/intrax8.h | 4 +++- libavcodec/vc1_block.c | 2 +- libavcodec/wmv2dec.c | 2 +- 4 files changed, 21 insertions(+), 22 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index

[libav-devel] [PATCH 13/25] intrax8: Use a local buffer instead of a ScratchpadContext

2016-02-22 Thread Vittorio Giovara
Sustain possible size changes via av_fast_malloc(). --- libavcodec/intrax8.c | 11 --- libavcodec/intrax8.h | 2 ++ 2 files changed, 10 insertions(+), 3 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index d0e7e93..4955a2b 100644 --- a/libavcodec/intrax8.c +++

[libav-devel] [PATCH 06/25] intrax8: Check and propagate errors from ff_intrax8_common_init

2016-02-22 Thread Vittorio Giovara
This allows to drop the assert since it would be redundant. --- libavcodec/intrax8.c | 12 +--- libavcodec/intrax8.h | 3 ++- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 3699f3b..b432f96 100644 ---

[libav-devel] [PATCH 09/25] intrax8: Make x8_init_block_index not use mpegvideo fields

2016-02-22 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 23 +++ 1 file changed, 11 insertions(+), 12 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 64c92fe..7af5f65 100644 --- a/libavcodec/intrax8.c +++ b/libavcodec/intrax8.c @@ -719,22 +719,21 @@ block_placed: } // FIXME

[libav-devel] [PATCH 01/25] fate: Add test for WMV2 with jframes

2016-02-22 Thread Vittorio Giovara
--- The test sample is already on FATE strangely enough. Vittorio tests/fate/microsoft.mak| 3 + tests/ref/fate/wmv8-intrax8 | 475 2 files changed, 478 insertions(+) create mode 100644 tests/ref/fate/wmv8-intrax8 diff --git

[libav-devel] [PATCH 23/25] intrax8: Drop MB emulation code

2016-02-22 Thread Vittorio Giovara
This is already performed in init_context_frame(). --- libavcodec/intrax8.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index b306aa3..9c82ef9 100644 --- a/libavcodec/intrax8.c +++ b/libavcodec/intrax8.c @@ -828,8 +828,6 @@ int

[libav-devel] [PATCH 20/25] intrax8: Keep a reference to the decoder blocks

2016-02-22 Thread Vittorio Giovara
--- NOTE: both these elements are not actually used by the decoders: vc1 modifies them only for intra (non-j) or interframes, wmv2 does something similar. I'm not extremely confident in dropping the references, at least not right now, however these two buffers could and should be made local.

[libav-devel] [PATCH 08/25] intrax8: Use local destination buffers

2016-02-22 Thread Vittorio Giovara
This buffers are just a way to store frame pointers and be able to modify them without touching the original ones. The two dependent decoders (WMV2 and VC1) do not need special care for these fields: the former does not seem to use the dest buffers, while the latter reinits them every time to the

[libav-devel] [PATCH 11/25] intrax8: Carry over the loopfilter value in ff_intrax8_decode_picture

2016-02-22 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 5 +++-- libavcodec/intrax8.h | 7 +-- libavcodec/vc1_block.c | 2 +- libavcodec/wmv2dec.c | 3 ++- 4 files changed, 11 insertions(+), 6 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 5aa9c36..d9c156c 100644 ---

[libav-devel] [PATCH 05/25] intrax8: Let x8_vlc_init report errors

2016-02-22 Thread Vittorio Giovara
--- libavcodec/intrax8.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/libavcodec/intrax8.c b/libavcodec/intrax8.c index 9e9479b..3699f3b 100644 --- a/libavcodec/intrax8.c +++ b/libavcodec/intrax8.c @@ -46,7 +46,7 @@ static VLC j_ac_vlc[2][2][8]; // [quant < 13],

[libav-devel] [PATCH 16/25] intrax8: Add a local BlockDSPContext and initialize it

2016-02-22 Thread Vittorio Giovara
--- configure| 2 +- libavcodec/intrax8.c | 3 ++- libavcodec/intrax8.h | 2 ++ 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/configure b/configure index d11b128..51141db 100755 --- a/configure +++ b/configure @@ -1860,7 +1860,7 @@ error_resilience_select="me_cmp"

[libav-devel] [PATCH] fate: Add test for indeo2 with delta frames

2016-02-22 Thread Vittorio Giovara
--- tests/fate/indeo.mak| 3 ++ tests/ref/fate/indeo2-delta | 115 2 files changed, 118 insertions(+) create mode 100644 tests/ref/fate/indeo2-delta diff --git a/tests/fate/indeo.mak b/tests/fate/indeo.mak index e725a6b..1a47d79 100644 ---

[libav-devel] [PATCH 1/2] indeo2data: K formatting cosmetics

2016-02-22 Thread Vittorio Giovara
From: Luca Barbato <lu_z...@gentoo.org> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/indeo2data.h | 208 1 file changed, 104 insertions(+), 104 deletions(-) diff --git a/libavcodec/indeo2data.h

[libav-devel] [PATCH 2/2] indeo2: Fix banding artefacts

2016-02-22 Thread Vittorio Giovara
From: Luca Barbato <lu_z...@gentoo.org> Rename luma table to delta table and change how it is used. CC: libav-sta...@libav.org Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/indeo2.c | 15 +++-- libavcodec/indeo2

Re: [libav-devel] [PATCH 2/2] indeo2: Update the data tables

2016-02-22 Thread Vittorio Giovara
On Mon, Feb 22, 2016 at 5:13 AM, Luca Barbato wrote: > And use the delta table as it should be used. > Fix some banding artefacts. Imho the most important thing is fixing the banding in the video not updating the tables May I suggest "indeo2: Fix banding artefacts Update

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-22 Thread Vittorio Giovara
On Sun, Feb 21, 2016 at 5:51 AM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 21/02/16 02:49, Vittorio Giovara wrote: >> On Sat, Feb 20, 2016 at 8:22 PM, Luca Barbato <lu_z...@gentoo.org> wrote: >>> On 20/02/16 23:40, Diego Biurrun wrote: >>>> Are you on

Re: [libav-devel] [PATCH 274/275] examples/transcode_aac: convert to codecpar

2016-02-22 Thread Vittorio Giovara
On Sun, Feb 21, 2016 at 6:53 AM, Anton Khirnov wrote: > --- > doc/examples/transcode_aac.c | 66 > > 1 file changed, 49 insertions(+), 17 deletions(-) probably ok -- Vittorio ___

Re: [libav-devel] [PATCH 270/275] avplay: do not use AVStream.codec for decoding

2016-02-22 Thread Vittorio Giovara
On Sun, Feb 21, 2016 at 6:53 AM, Anton Khirnov wrote: > AVStream.codec is now deprecated. Allocate a separate codec context > instead. > --- > avplay.c | 53 +++-- > 1 file changed, 35 insertions(+), 18 deletions(-) probably ok

Re: [libav-devel] [PATCH 272/275] examples/output: convert to codecpar

2016-02-22 Thread Vittorio Giovara
On Sun, Feb 21, 2016 at 6:53 AM, Anton Khirnov wrote: > --- > doc/examples/output.c | 60 > --- > 1 file changed, 43 insertions(+), 17 deletions(-) ok i think -- Vittorio ___

Re: [libav-devel] [PATCH] configure: add --enable-rpath

2016-02-22 Thread Vittorio Giovara
On Mon, Feb 22, 2016 at 8:03 AM, Diego Biurrun wrote: > On Sat, Aug 23, 2014 at 10:59:53AM -0400, Reinhard Tartler wrote: >> This option facilitates testing shared libarary builds: for instance >> fate builders do no longer need to set LD_LIBRARY_PATH as the binaries will >> get

Re: [libav-devel] [PATCH 269/275] avprobe: convert to codecpar

2016-02-22 Thread Vittorio Giovara
On Mon, Feb 22, 2016 at 5:23 AM, Anton Khirnov wrote: > Quoting Diego Biurrun (2016-02-21 15:07:24) >> On Sun, Feb 21, 2016 at 12:53:25PM +0100, Anton Khirnov wrote: >> > --- a/avprobe.c >> > +++ b/avprobe.c >> > @@ -619,31 +619,25 @@ static void show_stream(AVFormatContext

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
On Sat, Feb 20, 2016 at 4:12 PM, Derek Buitenhuis <derek.buitenh...@gmail.com> wrote: > On 2/20/2016 7:36 PM, Vittorio Giovara wrote: >> I am able to reproduce with that exact same line, on OSX, with clang and gcc > > A real (ports) GCC or the llvm-gcc that apple ships? >

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
On Sat, Feb 20, 2016 at 8:22 PM, Luca Barbato wrote: > On 20/02/16 23:40, Diego Biurrun wrote: >> Are you on Git HEAD and did you distclean? The wmv2 decoder depends on > > ar -t libavcodec/libavcodec.a | grep idctdsp.o > idctdsp.o > > Looks strange, the object is there... >

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
On Sat, Feb 20, 2016 at 12:57 PM, Diego Biurrun <di...@biurrun.de> wrote: > On Sat, Feb 20, 2016 at 11:22:18AM -0500, Vittorio Giovara wrote: >> On Sat, Feb 20, 2016 at 6:46 AM, Diego Biurrun <di...@biurrun.de> wrote: >> > On Sat, Feb 20, 2016 at 04:22:37AM

Re: [libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
On Sat, Feb 20, 2016 at 6:46 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Sat, Feb 20, 2016 at 04:22:37AM -0500, Vittorio Giovara wrote: >> The test program fails to build with `configure --disable-decoders`. > > I cannot reproduce. my full config line is ./configure --

[libav-devel] [PATCH] build: Add idct dependency to dct test

2016-02-20 Thread Vittorio Giovara
The test program fails to build with `configure --disable-decoders`. --- libavcodec/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 256dee3..3c57539 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -797,7

Re: [libav-devel] [PATCH 05/13] smoothstreamingenc: eliminate ffurl_* usage

2016-02-19 Thread Vittorio Giovara
On Fri, Feb 19, 2016 at 2:13 PM, Anton Khirnov wrote: > Now all IO should go through the IO callbacks and be interceptable by > the caller. > --- > libavformat/smoothstreamingenc.c | 44 > ++-- > 1 file changed, 20 insertions(+), 24

Re: [libav-devel] [PATCH 01/13] avprobe: do not call avio_close() on a custom context

2016-02-19 Thread Vittorio Giovara
On Fri, Feb 19, 2016 at 2:13 PM, Anton Khirnov wrote: > avio_close() can only be called on AVIOContexts created by avio_open(2). > --- > avprobe.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/avprobe.c b/avprobe.c > index 2ab8ce0..63e6c2a 100644 >

Re: [libav-devel] [PATCH] configure: Remove a typo

2016-02-17 Thread Vittorio Giovara
On Tue, Feb 16, 2016 at 4:00 PM, Luca Barbato wrote: > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index 270a6ec..fee6534 100755 > --- a/configure > +++ b/configure > @@ -2155,7 +2155,7 @@

Re: [libav-devel] [PATCH] configure: Use set_all to force the dependency refresh

2016-02-16 Thread Vittorio Giovara
On Tue, Feb 16, 2016 at 9:34 AM, Luca Barbato wrote: > Allow to issue twice --enable options. > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index 270a6ec..7837ce4 100755 > --- a/configure > +++ b/configure >

Re: [libav-devel] [PATCH] mov: Trim dref absolute path

2016-02-16 Thread Vittorio Giovara
On Tue, Feb 16, 2016 at 2:19 AM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 16/02/16 03:44, Vittorio Giovara wrote: >> Samples produced by Omneon (Harmonic) store external references with >> paths ending with 0s. Such movs cannot be loaded properly since every >> 0 i

[libav-devel] [PATCH] libx264: Fix noise_reduction assignment

2016-02-16 Thread Vittorio Giovara
First check the context, then check internal option. Drop the ! typo. Introduced in 60f0fde3092d18d4d36555962c192af8691a099c. --- libavcodec/libx264.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/libx264.c b/libavcodec/libx264.c index 7798ef3..8d72f1c 100644

Re: [libav-devel] [PATCH] mov: Fix the format specifier type for size

2016-02-16 Thread Vittorio Giovara
On Tue, Feb 16, 2016 at 4:38 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Mon, Feb 15, 2016 at 09:46:16PM -0500, Vittorio Giovara wrote: >> --- a/libavformat/mov.c >> +++ b/libavformat/mov.c >> @@ -1796,7 +1796,8 @@ int ff_mov_read_stsd_entries(MOVContext *c, >

Re: [libav-devel] [PATCH] configure: Relax the implication of --enable for components

2016-02-15 Thread Vittorio Giovara
On Sun, Feb 14, 2016 at 7:51 PM, Luca Barbato wrote: > Do not error out if some subcomponents cannot be enabled. > > Reported-By: RT|AO Who? -- Vittorio ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH] mov: Fix the format specifier type for size

2016-02-15 Thread Vittorio Giovara
--- libavformat/mov.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 7081ead..d0a30c2 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -1796,7 +1796,8 @@ int ff_mov_read_stsd_entries(MOVContext *c, AVIOContext *pb,

[libav-devel] [PATCH] mov: Trim dref absolute path

2016-02-15 Thread Vittorio Giovara
than to the actual referenced file. Add a brief trimming loop that drops all ending 0s before trying to parse the external reference path. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Note: all samples I was given only have one zero appended, so it might be enough to j

Re: [libav-devel] [PATCH 11/14] lavfi: add a filter for uploading normal frames to CUDA

2016-02-12 Thread Vittorio Giovara
On Fri, Feb 12, 2016 at 4:09 AM, Anton Khirnov wrote: > --- > configure | 1 + > doc/filters.texi | 11 ++ > libavfilter/Makefile | 1 + > libavfilter/allfilters.c | 1 + > libavfilter/vf_hwupload_cuda.c | 237 >

[libav-devel] [PATCH] build: Add missing celp_math dependency for G723_1 encoder and decoder

2016-02-12 Thread Vittorio Giovara
--- libavcodec/Makefile | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 4800779..eae62c8 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -229,9 +229,11 @@ OBJS-$(CONFIG_FRAPS_DECODER) += fraps.o

Re: [libav-devel] [PATCH] buffersrc: do not discard the error from ff_filter_frame()

2016-02-11 Thread Vittorio Giovara
On Thu, Feb 11, 2016 at 5:27 AM, Anton Khirnov <an...@khirnov.net> wrote: > Quoting Vittorio Giovara (2016-02-09 22:43:13) >> On Tue, Feb 9, 2016 at 4:02 PM, Anton Khirnov <an...@khirnov.net> wrote: >> > --- >> > libavfilter/buffersrc.c | 2 +- >>

Re: [libav-devel] [PATCH 07/13] pixfmt: add a CUDA hwaccelled format

2016-02-09 Thread Vittorio Giovara
On Tue, Feb 9, 2016 at 1:51 PM, Anton Khirnov wrote: > Quoting Hendrik Leppkes (2016-02-09 19:19:32) >> Why a different comment style to all the other formats? > > Because I don't like the ultralong lines. +1 -- Vittorio ___

Re: [libav-devel] [PATCH] tiny_psnr: Support large files

2016-02-09 Thread Vittorio Giovara
On Tue, Feb 9, 2016 at 6:12 AM, Luca Barbato wrote: > From: Thomas Lee no real email available? > If an input file is bigger than 2GB (assume sizeof(int) == 4)), > size0/size1 will overflow, making stddev and PSNR invalid. > >

[libav-devel] [PATCH] h264: Add an AVClass pointer to H264Context

2016-02-09 Thread Vittorio Giovara
From: Michael Niedermayer Fixes: asan_heap-uaf_3660f67_757_cov_1257014655_Hi422FR1_SONY_A.jsv Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer --- Not sure about the security fix, but makes sense to have, since h264

[libav-devel] [PATCH] h264: Add an AVClass pointer to H264Context

2016-02-09 Thread Vittorio Giovara
From: Michael Niedermayer <michae...@gmx.at> Fixes: asan_heap-uaf_3660f67_757_cov_1257014655_Hi422FR1_SONY_A.jsv Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer <michae...@gmx.at> Signed-off-by: Vittorio Giovara <v

Re: [libav-devel] [PATCH] buffersrc: do not discard the error from ff_filter_frame()

2016-02-09 Thread Vittorio Giovara
On Tue, Feb 9, 2016 at 4:02 PM, Anton Khirnov wrote: > --- > libavfilter/buffersrc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavfilter/buffersrc.c b/libavfilter/buffersrc.c > index f5b852f..0079f51 100644 > --- a/libavfilter/buffersrc.c > +++

[libav-devel] [PATCH] mov: check atom nesting depth and limit it to 10

2016-02-09 Thread Vittorio Giovara
From: Michael Niedermayer Fixes call stack overflow Fixes: case1_call_stack_overflow.mp4 Found-by: Michal Zalewski Signed-off-by: Michael Niedermayer --- libavformat/isom.h | 1 + libavformat/mov.c | 13 - 2 files changed,

Re: [libav-devel] [PATCH] x265: Add the same frame-type-matching options added to x264

2016-02-08 Thread Vittorio Giovara
On Mon, Aug 31, 2015 at 11:07 AM, Luca Barbato wrote: > Same rationale and same code > --- > libavcodec/libx265.c | 31 +++ > 1 file changed, 27 insertions(+), 4 deletions(-) > > diff --git a/libavcodec/libx265.c b/libavcodec/libx265.c > index

Re: [libav-devel] [PATCH 1/2] mpeg12enc: always write closed gops for intra only outputs

2016-02-08 Thread Vittorio Giovara
On Mon, Jan 5, 2015 at 9:38 AM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 05/01/15 15:15, Vittorio Giovara wrote: >> >> I am not extremely sure, probably not to interfere with the flag >> parameter. >> Do you think it would be a more appropriate to do that? &g

Re: [libav-devel] [PATCH 2/4] mpeg12dec: Refactor mpeg1_decode_block_intra()

2016-02-08 Thread Vittorio Giovara
On Mon, Feb 8, 2016 at 2:05 PM, Diego Biurrun wrote: > Reduce variable scope, improve variable name, drop pointless ff_dlog(), > merge variable declaration and initialization, whitespace cosmetics. > --- > libavcodec/mpeg12dec.c | 27 +-- > 1 file

Re: [libav-devel] [PATCH] asfdec: remove unused parameters from the function arguments

2016-02-08 Thread Vittorio Giovara
On Tue, Jan 5, 2016 at 4:06 AM, Alexandra Hájková wrote: > --- > libavformat/asfdec.c | 25 - > 1 file changed, 12 insertions(+), 13 deletions(-) > > diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c > index ca7712f..62c7775 100644 >

Re: [libav-devel] [PATCH 2/2] mpeg12enc: always set closed gop flag on the first gop

2016-02-08 Thread Vittorio Giovara
On Mon, Dec 15, 2014 at 7:51 AM, Luca Barbato <lu_z...@gentoo.org> wrote: > On 08/12/14 15:27, Vittorio Giovara wrote: >> >> From: Marton Balint <c...@passwd.hu> >> >> Improves compatibility with XDCAM HD formats. It has been set for >> a long time in

Re: [libav-devel] [PATCH] asfdec: hadle invalid Object size properly

2016-02-08 Thread Vittorio Giovara
On Tue, Jun 30, 2015 at 5:19 AM, Luca Barbato wrote: > On 30/06/15 11:35, Alexandra Hájková wrote: >> --- >> libavformat/asfdec.c | 12 ++-- >> 1 file changed, 10 insertions(+), 2 deletions(-) >> >> diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c >> index

Re: [libav-devel] [PATCH 1/3] configure: Use `require` for the non-component options

2016-02-04 Thread Vittorio Giovara
On Tue, Jan 26, 2016 at 7:16 PM, Luca Barbato wrote: > On 23/01/16 19:35, Luca Barbato wrote: >> And make sure to enable all the components needed >> --- >> configure | 11 --- >> 1 file changed, 8 insertions(+), 3 deletions(-) >> > > Ping. if nobody is against, then

Re: [libav-devel] [PATCH] build: Disable VDA if an unsupported compiler is used

2016-02-04 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 9:22 AM, Martin Storsjö <mar...@martin.st> wrote: > On Thu, 21 Jan 2016, Diego Biurrun wrote: > >> On Wed, Jan 20, 2016 at 11:11:31AM -0500, Vittorio Giovara wrote: >>> >>> On Wed, Jan 20, 2016 at 3:19 AM, Martin Storsjö <mar...@mart

Re: [libav-devel] [PATCH] rtsp: Factor out the rtp stream opening

2016-02-04 Thread Vittorio Giovara
On Thu, Dec 17, 2015 at 1:58 PM, Luca Barbato wrote: > --- > libavformat/rtsp.c | 74 > +- > 1 file changed, 40 insertions(+), 34 deletions(-) > > diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c > index

Re: [libav-devel] [PATCH] configure: Support MSYS2 mingw-w64 64bit

2016-02-04 Thread Vittorio Giovara
On Wed, Dec 2, 2015 at 7:55 PM, Hendrik Leppkes wrote: > On Wed, Dec 2, 2015 at 5:48 PM, Luca Barbato wrote: >> --- >> configure | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/configure b/configure >> index

Re: [libav-devel] [PATCH 3/3] eatqi: Remove MpegEncContext dependency

2016-02-04 Thread Vittorio Giovara
On Thu, Jan 28, 2016 at 6:19 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > On Thu, Jan 28, 2016 at 5:00 AM, Diego Biurrun <di...@biurrun.de> wrote: >> On Mon, Jan 25, 2016 at 12:56:12PM -0500, Vittorio Giovara wrote: >>> --- a/libavcodec/eatqi.

Re: [libav-devel] [PATCH 1/3] x264: Optionally forward the input frame type

2016-02-04 Thread Vittorio Giovara
On Tue, Sep 1, 2015 at 12:50 AM, Luca Barbato wrote: > On 31/08/15 20:55, Anton Khirnov wrote: >> Looks like a hack to me. If you don't want to force the frame type, >> just don't set it. > > So I should move that on an option that does selectively forward only > some kind of

Re: [libav-devel] [PATCH] avconv: Move the reporting in a separate thread

2016-02-04 Thread Vittorio Giovara
On Wed, Jan 6, 2016 at 9:22 PM, Luca Barbato wrote: > Now the report is printed even if poll_filters() does not > return often. > --- > avconv.c | 106 > -- > avconv.h | 4 +++ > configure | 3 +- > 3 files

Re: [libav-devel] [PATCH] avf: Dump the cpb side data information

2016-02-04 Thread Vittorio Giovara
On Sat, Jan 2, 2016 at 6:25 PM, Anton Khirnov wrote: > Quoting Luca Barbato (2016-01-02 15:34:25) >> Subject: Re: [libav-devel] [PATCH] avf: Dump the cpb side data information > ^ > missing l > >> --- >> libavformat/dump.c | 20

Re: [libav-devel] [PATCH] h264: Parse only the x264 info unregisterd sei

2016-02-04 Thread Vittorio Giovara
On Thu, Jan 28, 2016 at 9:49 PM, Luca Barbato wrote: > On 29/07/15 22:51, Luca Barbato wrote: >> And restrict the string to ascii text. >> >> CC: libav-sta...@libav.org >> >> Signed-off-by: Luca Barbato >> --- >> libavcodec/h264_sei.c | 57 >>

Re: [libav-devel] [RFC] avcodec: Add native DCA decoder based on libdcadec.

2016-02-04 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 10:18 AM, Luca Barbato wrote: > On 07/01/16 13:18, foo86 wrote: >> Yes, I'm willing to do that. I'm going to change the patch to replace >> existing dca decoder and resubmit it for review. > > Had been 2 weeks, any news about it? If needed I can help,

[libav-devel] [PATCH] flacenc: Load default prediction_order parameters if none is selected

2016-02-03 Thread Vittorio Giovara
From: Michael Niedermayer <mich...@niedermayer.cc> Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/flacenc.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/liba

Re: [libav-devel] [PATCH] avplay: drop support for building without lavfi

2016-01-31 Thread Vittorio Giovara
On Sun, Jan 31, 2016 at 2:44 PM, Anton Khirnov wrote: > lavfi has been considered to be stable for a while now, so it is enabled > in most configurations. Supporting avplay without lavfi requires a lot > of nontrivial ifdef mess for no good reason. > --- > avplay.c | 88 >

[libav-devel] [PATCH 2/2] swscale: Add input support for 12 bit formats

2016-01-31 Thread Vittorio Giovara
From: Michael Niedermayer <michae...@gmx.at> Implemented for AV_PIX_FMT_GBRP12. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libswscale/input.c | 34 ++ libswscale/ppc/swscale_altivec.c | 2 +- libswsca

[libav-devel] [PATCH 1/2] pixfmt: Add AV_PIX_FMT_GBRP12 format

2016-01-31 Thread Vittorio Giovara
From: Michael Niedermayer <michae...@gmx.at> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavutil/pixdesc.c | 25 + libavutil/pixfmt.h | 4 libavutil/version.h | 2 +- 3 files changed, 30 insertions(+), 1 deletion(-) diff --git

Re: [libav-devel] [PATCH 3/3] eatqi: Remove MpegEncContext dependency

2016-01-28 Thread Vittorio Giovara
On Thu, Jan 28, 2016 at 5:00 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Mon, Jan 25, 2016 at 12:56:12PM -0500, Vittorio Giovara wrote: >> --- a/libavcodec/eatqi.c >> +++ b/libavcodec/eatqi.c >> @@ -35,42 +35,51 @@ >> >> -static int tqi_decode_mb

Re: [libav-devel] [PATCHv3] avcodec: Cineform HD Decoder

2016-01-28 Thread Vittorio Giovara
On Mon, Jan 25, 2016 at 6:15 PM, Kieran Kunhya wrote: >>> +{ >>> +CFHDContext *s = avctx->priv_data; >>> + >>> +avctx->pix_fmt = AV_PIX_FMT_YUV422P10; >> >> if the decoder supports multiple pixel formats it's better not to >> initialize the pixel

[libav-devel] [PATCH] libx264: Make sure to preserve default option values

2016-01-28 Thread Vittorio Giovara
The private options chromaoffset, sc_threshold, and noise_reduction were set to 0 rather than -1, and were always initializing values in libx264 rather than letting the library use its default. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- Thanks to Derek for sp

Re: [libav-devel] [PATCH 2/3] mpeg1: Make intra-block decoding independent of MpegEncContext

2016-01-28 Thread Vittorio Giovara
On Thu, Jan 28, 2016 at 5:14 AM, Diego Biurrun wrote: > > Why is block_last_index suddenly only set in one place? Looks suspicious > and unrelated to me. You mean why is it set only in mpeg1 and not on eatqi? Well it's one of the fields that are used in mpegvideo, but are

Re: [libav-devel] [FFmpeg-devel] [PATCHv3] avcodec: Cineform HD Decoder

2016-01-28 Thread Vittorio Giovara
On Thu, Jan 28, 2016 at 1:15 PM, Kieran Kunhya <kieran...@googlemail.com> wrote: > On Thu, 28 Jan 2016 at 16:26 Vittorio Giovara <vittorio.giov...@gmail.com> > wrote: > >> On Mon, Jan 25, 2016 at 6:15 PM, Kieran Kunhya <kieran...@googlemail.com> >> wrot

Re: [libav-devel] [PATCH 4/4] avcodec/v210: document the requirement for sample_factor

2016-01-27 Thread Vittorio Giovara
On Wed, Jan 27, 2016 at 2:59 PM, James Darnley wrote: > On 21/01/16 14:00, Anton Khirnov wrote: >>> Does it make sense for both 8bit and 10bit functions to be called on one >>> context? IIUC, it only works this way for the checkasm tests. So perhaps >>> it'd be better and

Re: [libav-devel] [PATCH 2/2] eatqi: Remove mpegvideo dependency

2016-01-26 Thread Vittorio Giovara
On Tue, Jan 26, 2016 at 1:11 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Mon, Jan 25, 2016 at 12:55:05PM -0500, Vittorio Giovara wrote: >> On Fri, Jan 22, 2016 at 1:36 AM, Diego Biurrun <di...@biurrun.de> wrote: >> > On Thu, Jan 21, 2016 at 04:53:56PM

Re: [libav-devel] [PATCH] v210: x86: Add the correct guards around the asm code

2016-01-26 Thread Vittorio Giovara
On Sun, Jan 24, 2016 at 11:58 AM, Luca Barbato wrote: > On 24/01/16 16:58, Luca Barbato wrote: >> --- >> >> As done in ac3 >> > > And with this patch the v210 set passes the tests on travis as well. possibly ok -- Vittorio ___

[libav-devel] [PATCH] mpeg12dec: Always close reader on error

2016-01-25 Thread Vittorio Giovara
A reader left open may lead to hangs. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/mpeg12dec.c | 63 ++ 1 file changed, 48 insertions(+), 15 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg1

Re: [libav-devel] [PATCHv3] avcodec: Cineform HD Decoder

2016-01-25 Thread Vittorio Giovara
On Sun, Jan 24, 2016 at 7:34 PM, Kieran Kunhya wrote: > Decodes YUV 4:2:2 10-bit and RGB 12-bit files. > Older files with more subbands, skips, Bayer, alpha not supported. > Alpha requires addition of GBRAP12 pixel format. Actually you could set AV_PIX_FMT_GBRAP16 and tweak

Re: [libav-devel] [PATCH 3/3] pixdesc: Use AV_CEIL_RSHIFT in documentation

2016-01-25 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 6:00 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > --- > libavutil/pixdesc.h | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/libavutil/pixdesc.h b/libavutil/pixdesc.h > index fe06d94..f52c434 100644 >

Re: [libav-devel] [PATCH 2/2] eatqi: Remove mpegvideo dependency

2016-01-25 Thread Vittorio Giovara
On Fri, Jan 22, 2016 at 1:36 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Thu, Jan 21, 2016 at 04:53:56PM -0500, Vittorio Giovara wrote: >> --- a/libavcodec/eatqi.c >> +++ b/libavcodec/eatqi.c >> @@ -137,15 +149,15 @@ static int tqi_decode_frame(AVCodecContext *avctx

[libav-devel] [PATCH 3/3] eatqi: Remove MpegEncContext dependency

2016-01-25 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- configure | 2 +- libavcodec/Makefile | 2 +- libavcodec/eatqi.c | 90 ++--- 3 files changed, 53 insertions(+), 41 deletions(-) diff --git a/configure b/configure

[libav-devel] [PATCH 2/3] mpeg1: Make intra-block decoding independent of MpegEncContext

2016-01-25 Thread Vittorio Giovara
This allows untangling the eatqi decoder from the MPEG-1 decoder. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/eatqi.c | 9 - libavcodec/mpeg12.c| 89 +++ libavcodec/mpeg12.h| 6 ++- liba

[libav-devel] [PATCH 1/3] mpeg12dec: Always close reader on error

2016-01-25 Thread Vittorio Giovara
A reader left open may lead to hangs. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/mpeg12dec.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index 711b310..1e1e4a5

Re: [libav-devel] [PATCH 1/2] mpeg1: Make intra-block decoding independent of mpegvideoenc

2016-01-22 Thread Vittorio Giovara
On Fri, Jan 22, 2016 at 1:47 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Thu, Jan 21, 2016 at 04:53:55PM -0500, Vittorio Giovara wrote: >> Move it to a more generic file. Needed to untangle eatqi. >> >> Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com&

Re: [libav-devel] [PATCH] hpeldsp: Shrink put_no_rnd_pixels_tab array to the required minimal size

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 3:03 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Wed, Jan 20, 2016 at 11:06:29AM -0500, Vittorio Giovara wrote: >> On Wed, Jan 20, 2016 at 5:54 AM, Diego Biurrun <di...@biurrun.de> wrote: >> > --- >> > >> > I d

Re: [libav-devel] [PATCH 1/3] imgconvert: Move the shrink functions only where needed

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 3:05 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Wed, Jan 20, 2016 at 10:53:38AM -0500, Vittorio Giovara wrote: >> On Wed, Jan 20, 2016 at 4:59 AM, Diego Biurrun <di...@biurrun.de> wrote: >> >> +src += 2 * src_wra

Re: [libav-devel] [PATCH 1/3] imgconvert: Move the shrink functions only where needed

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 12:19 PM, Diego Biurrun <di...@biurrun.de> wrote: > On Thu, Jan 21, 2016 at 11:59:22AM -0500, Vittorio Giovara wrote: >> On Thu, Jan 21, 2016 at 3:05 AM, Diego Biurrun <di...@biurrun.de> wrote: >> > On Wed, Jan 20, 2016 at 10:53:38AM

Re: [libav-devel] [PATCH/RFC] lavc: configure: Factor out celp module

2016-01-21 Thread Vittorio Giovara
On Wed, Jan 20, 2016 at 3:44 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Tue, Jan 19, 2016 at 02:27:13PM -0500, Vittorio Giovara wrote: >> On Tue, Jan 19, 2016 at 1:36 PM, Diego Biurrun <di...@biurrun.de> wrote: >> > Leave these out or create proper fine-g

Re: [libav-devel] [PATCH] build: Disable VDA if an unsupported compiler is used

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 3:07 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Wed, Jan 20, 2016 at 11:11:31AM -0500, Vittorio Giovara wrote: >> On Wed, Jan 20, 2016 at 3:19 AM, Martin Storsjö <mar...@martin.st> wrote: >> > On Wed, 20 Jan 2016, Vittorio Giovara wro

Re: [libav-devel] [PATCH] configure: Improve requesting specific features

2016-01-21 Thread Vittorio Giovara
On Wed, Jan 20, 2016 at 7:05 PM, Luca Barbato wrote: > Before this patch trying to enable only https by issuing > > --disable-protocols --enable-protocol=https --enable-gnutls > > does not enable https, and > > --disable-all --enable-protocols > > does not enable

Re: [libav-devel] [PATCH] eatqi: Remove mpegvideo dependency

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 1:48 PM, Diego Biurrun <di...@biurrun.de> wrote: > On Thu, Jan 21, 2016 at 12:47:29PM -0500, Vittorio Giovara wrote: >> On Thu, Jan 21, 2016 at 12:23 PM, Diego Biurrun <di...@biurrun.de> wrote: >> > On Thu, Jan 21, 2016 at 11:38:20AM

Re: [libav-devel] [PATCH] eatqi: Remove mpegvideo dependency

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 12:23 PM, Diego Biurrun <di...@biurrun.de> wrote: > On Thu, Jan 21, 2016 at 11:38:20AM -0500, Vittorio Giovara wrote: >> On Thu, Jan 21, 2016 at 3:17 AM, Diego Biurrun <di...@biurrun.de> wrote: >> > On Wed, Jan 20, 2016 at 10:48:12AM

Re: [libav-devel] [PATCH] configure: Force-enable select_any dependencies only on --enable

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 2:37 PM, Luca Barbato wrote: > Unbreak --enable-gnutls enabling tls_openssl_protocol > --- > > Thanks for spotting Vittorio > > configure | 19 +-- > 1 file changed, 17 insertions(+), 2 deletions(-) works for me, thanks -- Vittorio

Re: [libav-devel] [PATCH] eatqi: Remove mpegvideo dependency

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 2:41 PM, Diego Biurrun <di...@biurrun.de> wrote: > On Tue, Jan 19, 2016 at 07:45:25PM -0500, Vittorio Giovara wrote: >> --- a/libavcodec/eatqi.c >> +++ b/libavcodec/eatqi.c >> @@ -35,69 +35,177 @@ >> >> +/* Based off mpeg1_decode_blo

Re: [libav-devel] [PATCH] eatqi: Remove mpegvideo dependency

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 3:17 AM, Diego Biurrun <di...@biurrun.de> wrote: > On Wed, Jan 20, 2016 at 10:48:12AM -0500, Vittorio Giovara wrote: >> On Wed, Jan 20, 2016 at 5:10 AM, Diego Biurrun <di...@biurrun.de> wrote: >> > On Tue, Jan 19, 2016 at 07:45:25PM

[libav-devel] [PATCH 1/3] vf_gradfun: Use AV_CEIL_RSHIFT

2016-01-21 Thread Vittorio Giovara
--- libavfilter/vf_gradfun.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_gradfun.c b/libavfilter/vf_gradfun.c index f7c4372..ce79329 100644 --- a/libavfilter/vf_gradfun.c +++ b/libavfilter/vf_gradfun.c @@ -172,8 +172,8 @@ static int

[libav-devel] [PATCH 2/3] vf_unsharp: Use AV_CEIL_RSHIFT instead of private macro

2016-01-21 Thread Vittorio Giovara
--- libavfilter/vf_unsharp.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/libavfilter/vf_unsharp.c b/libavfilter/vf_unsharp.c index d0d59e2..dbe3874 100644 --- a/libavfilter/vf_unsharp.c +++ b/libavfilter/vf_unsharp.c @@ -48,9 +48,6 @@ #define MIN_SIZE 3 #define

[libav-devel] [PATCH 3/3] pixdesc: Use AV_CEIL_RSHIFT in documentation

2016-01-21 Thread Vittorio Giovara
--- libavutil/pixdesc.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavutil/pixdesc.h b/libavutil/pixdesc.h index fe06d94..f52c434 100644 --- a/libavutil/pixdesc.h +++ b/libavutil/pixdesc.h @@ -84,16 +84,16 @@ typedef struct AVPixFmtDescriptor { /** *

Re: [libav-devel] [PATCH 1/3] vf_gradfun: Use AV_CEIL_RSHIFT

2016-01-21 Thread Vittorio Giovara
On Thu, Jan 21, 2016 at 6:00 PM, Vittorio Giovara <vittorio.giov...@gmail.com> wrote: > --- > libavfilter/vf_gradfun.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libavfilter/vf_gradfun.c b/libavfilter/vf_gradfun.c > index

[libav-devel] [PATCH 2/2] eatqi: Remove mpegvideo dependency

2016-01-21 Thread Vittorio Giovara
Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- configure | 2 +- libavcodec/Makefile | 2 +- libavcodec/eatqi.c | 90 ++--- 3 files changed, 53 insertions(+), 41 deletions(-) diff --git a/configure b/configure

[libav-devel] [PATCH 1/2] mpeg1: Make intra-block decoding independent of mpegvideoenc

2016-01-21 Thread Vittorio Giovara
Move it to a more generic file. Needed to untangle eatqi. Signed-off-by: Vittorio Giovara <vittorio.giov...@gmail.com> --- libavcodec/eatqi.c | 9 +++-- libavcodec/mpeg12.c| 89 ++ libavcodec/mpeg12.h| 6 +++- libavcodec/mpeg12dec.

[libav-devel] [PATCH] pixdesc: Drop unneeded deprecation warning guards

2016-01-21 Thread Vittorio Giovara
--- libavutil/pixdesc.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index 9e7d8bd..059bd62 100644 --- a/libavutil/pixdesc.c +++ b/libavutil/pixdesc.c @@ -1610,7 +1610,6 @@ static const char *chroma_location_names[AVCHROMA_LOC_NB] = { "top",

[libav-devel] [PATCH 1/2] yuv4mpegenc: Use AV_CEIL_RSHIFT

2016-01-21 Thread Vittorio Giovara
--- libavformat/yuv4mpegenc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/yuv4mpegenc.c b/libavformat/yuv4mpegenc.c index ef1e775..83f79d2 100644 --- a/libavformat/yuv4mpegenc.c +++ b/libavformat/yuv4mpegenc.c @@ -129,8 +129,8 @@ static int

<    8   9   10   11   12   13   14   15   16   17   >