Re: [libav-devel] [PATCH] vaapi_encode: Add driver-specific behaviour for i965 param buffers

2016-06-13 Thread Gwenole Beauchesne
Hi, 2016-06-10 10:49 GMT+02:00 Mark Thompson : > On 10/06/16 08:06, Anton Khirnov wrote: >> Quoting Mark Thompson (2016-06-10 01:25:58) >>> Adds a general mechanism for setting driver quirks, and detects the >>> driver name to set them. >>> >>> When the i965 driver is being used,

Re: [libav-devel] [PATCH 102/115] dirac: Convert to the new bitstream reader

2016-06-13 Thread Vittorio Giovara
On Thu, Jun 9, 2016 at 11:13 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/dirac.c | 89 > +++--- > 1 file changed, 45 insertions(+), 44 deletions(-) seems ok -- Vittorio

Re: [libav-devel] [PATCH 034/115] exr: Convert to the new bitstream reader

2016-06-13 Thread Vittorio Giovara
On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/exr.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) ok -- Vittorio ___ libav-devel

Re: [libav-devel] [PATCH 041/115] hq_hqa: Convert to the new bitstream reader

2016-06-13 Thread Vittorio Giovara
On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/hq_hqa.c | 48 > 1 file changed, 24 insertions(+), 24 deletions(-) > i think it's ok -- Vittorio

Re: [libav-devel] [PATCH 042/115] hqx: Convert to the new bitstream header

2016-06-13 Thread Vittorio Giovara
On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/hqx.c | 64 > > libavcodec/hqx.h | 5 +++-- > 2 files changed, 35 insertions(+), 34

Re: [libav-devel] [PATCH 071/115] mov: Convert to the new bitstream reader

2016-06-13 Thread Vittorio Giovara
On Thu, Jun 9, 2016 at 11:12 AM, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavformat/mov.c| 10 ++ > libavformat/movenc.c | 41 + > 2 files changed, 27 insertions(+), 24 deletions(-) >

Re: [libav-devel] [PATCH 113/115] Remove legacy get_bits headers

2016-06-13 Thread Vittorio Giovara
On Thu, Jun 9, 2016 at 11:13 AM, Diego Biurrun wrote: > --- > libavcodec/get_bits.h | 531 > libavcodec/golomb_legacy.h | 589 > - > libavcodec/unary_legacy.h | 56 - > 3 files

Re: [libav-devel] [PATCH 016/115] 4xm: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:58) > From: Alexandra Hájková > > --- > libavcodec/4xm.c | 27 ++- > 1 file changed, 14 insertions(+), 13 deletions(-) > > diff --git a/libavcodec/4xm.c b/libavcodec/4xm.c > index b2d4db2..ee9d020 100644 >

Re: [libav-devel] [PATCH 015/115] webp: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:57) > From: Alexandra Hájková > > --- > libavcodec/webp.c | 106 > ++ > 1 file changed, 50 insertions(+), 56 deletions(-) > > diff --git a/libavcodec/webp.c b/libavcodec/webp.c >

Re: [libav-devel] [PATCH 014/115] proresdec: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:56) > From: Alexandra Hájková > > --- > libavcodec/proresdec.c | 69 > -- > 1 file changed, 33 insertions(+), 36 deletions(-) > > diff --git a/libavcodec/proresdec.c

Re: [libav-devel] [PATCH 013/115] on2avc: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:55) > From: Alexandra Hájková > > --- > libavcodec/on2avc.c | 64 > +++-- > 1 file changed, 33 insertions(+), 31 deletions(-) > > diff --git a/libavcodec/on2avc.c

Re: [libav-devel] [PATCH 012/115] h261dec: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:54) > From: Alexandra Hájková > > --- > libavcodec/h261dec.c | 92 > ++- > libavformat/h261dec.c | 11 +++--- > 2 files changed, 53 insertions(+), 50 deletions(-) > > diff --git

Re: [libav-devel] [PATCH 011/115] latmenc: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:53) > From: Alexandra Hájková > > --- > libavformat/latmenc.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > > diff --git a/libavformat/latmenc.c b/libavformat/latmenc.c > index a98a6ff..42fd4b9 100644 >

Re: [libav-devel] [PATCH 010/115] adtsenc: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:52) > From: Alexandra Hájková > > --- > libavformat/adtsenc.c | 16 > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/libavformat/adtsenc.c b/libavformat/adtsenc.c > index e7c72d8..639c39e 100644 >

Re: [libav-devel] [PATCH 2/2] checkasm: Add tests for h264 idct

2016-06-13 Thread Martin Storsjö
On Mon, 13 Jun 2016, Diego Biurrun wrote: On Mon, Jun 13, 2016 at 12:08:15AM +0300, Martin Storsjö wrote: On Sat, 11 Jun 2016, Martin Storsjö wrote: > --- /dev/null > +++ b/tests/checkasm/h264dsp.c > @@ -0,0 +1,268 @@ > +for (dc = 0; dc <= 1; dc++) { > +void

Re: [libav-devel] [PATCH 009/115] mpeg4audio: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:51) > From: Alexandra Hájková > > --- > libavcodec/mpeg4audio.c | 118 > > libavcodec/mpeg4audio.h | 5 +- > 2 files changed, 62 insertions(+), 61 deletions(-) > > diff --git

Re: [libav-devel] [PATCH] h264dsp: Change type of stride parameters to ptrdiff_t

2016-06-13 Thread Martin Storsjö
On Mon, 13 Jun 2016, Diego Biurrun wrote: This avoids SIMD-optimized functions having to sign-extend their line size argument manually to be able to do pointer arithmetic. --- Added missing changes for h264idct. libavcodec/aarch64/h264dsp_init_aarch64.c | 37 +++

Re: [libav-devel] [PATCH 008/115] spdif: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:50) > From: Alexandra Hájková > > --- > libavformat/spdifdec.c | 7 --- > libavformat/spdifenc.c | 7 --- > 2 files changed, 8 insertions(+), 6 deletions(-) > > diff --git a/libavformat/spdifdec.c b/libavformat/spdifdec.c >

Re: [libav-devel] [PATCH 007/115] ea: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:49) > From: Alexandra Hájková > > --- > libavcodec/eatgq.c | 34 +- > libavcodec/eatgv.c | 23 --- > 2 files changed, 29 insertions(+), 28 deletions(-) > > diff --git

Re: [libav-devel] [PATCH 006/115] mpeg12: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:48) > From: Alexandra Hájková > > --- > libavcodec/eatqi.c | 8 +- > libavcodec/mdec.c | 34 ++-- > libavcodec/mpeg12.c| 31 ++-- > libavcodec/mpeg12.h| 12 +- > libavcodec/mpeg12dec.c | 415 >

Re: [libav-devel] [PATCH] h264dsp: Change type of stride parameters to ptrdiff_t

2016-06-13 Thread Vittorio Giovara
On Mon, Jun 13, 2016 at 1:29 PM, Diego Biurrun wrote: > This avoids SIMD-optimized functions having to sign-extend their > line size argument manually to be able to do pointer arithmetic. > --- ok if oracle is happy -- Vittorio ___

[libav-devel] [PATCH] h264dsp: Change type of stride parameters to ptrdiff_t

2016-06-13 Thread Diego Biurrun
This avoids SIMD-optimized functions having to sign-extend their line size argument manually to be able to do pointer arithmetic. --- Added missing changes for h264idct. libavcodec/aarch64/h264dsp_init_aarch64.c | 37 +++ libavcodec/arm/h264dsp_init_arm.c | 37

Re: [libav-devel] [PATCH 005/115] dnxhd: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-09 17:11:47) > From: Alexandra Hájková > > --- > libavcodec/dnxhddec.c | 46 -- > 1 file changed, 20 insertions(+), 26 deletions(-) > > diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c >

Re: [libav-devel] [PATCH 004/115] eamad: Convert to the new bitstream reader

2016-06-13 Thread Anton Khirnov
Quoting Diego Biurrun (2016-06-13 18:08:29) > On Fri, Jun 10, 2016 at 10:25:38AM +0200, Anton Khirnov wrote: > > Quoting Anton Khirnov (2016-06-10 10:16:00) > > > Quoting Diego Biurrun (2016-06-09 17:11:46) > > > > From: Alexandra Hájková > > > > --- a/libavcodec/eamad.c >

Re: [libav-devel] [PATCH 031/115] dxva2: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 13/06/16 18:00, Diego Biurrun wrote: > Only the H.264 part requires it, the other is independent. I'll squash > the other part into the rest of the mpeg12 changes. > Good =) ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 2/2] checkasm: Add tests for h264 idct

2016-06-13 Thread Diego Biurrun
On Mon, Jun 13, 2016 at 12:08:15AM +0300, Martin Storsjö wrote: > On Sat, 11 Jun 2016, Martin Storsjö wrote: > > --- /dev/null > > +++ b/tests/checkasm/h264dsp.c > > @@ -0,0 +1,268 @@ > > +for (dc = 0; dc <= 1; dc++) { > > +void (*idct)(uint8_t *, int16_t *, int); > > +

Re: [libav-devel] [PATCH 004/115] eamad: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 13/06/16 18:08, Diego Biurrun wrote: > On Fri, Jun 10, 2016 at 10:25:38AM +0200, Anton Khirnov wrote: >> Quoting Anton Khirnov (2016-06-10 10:16:00) >>> Quoting Diego Biurrun (2016-06-09 17:11:46) From: Alexandra Hájková --- a/libavcodec/eamad.c +++

Re: [libav-devel] [PATCH] h264: Drop unused function check_opcodes()

2016-06-13 Thread Luca Barbato
On 13/06/16 18:15, Diego Biurrun wrote: > --- > libavcodec/h264_refs.c | 12 > 1 file changed, 12 deletions(-) > > diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c > index c4b33af..2e43a8f 100644 > --- a/libavcodec/h264_refs.c > +++ b/libavcodec/h264_refs.c > @@ -524,18

Re: [libav-devel] [PATCH] LICENSE: Fix silly typo

2016-06-13 Thread Luca Barbato
On 13/06/16 18:15, Diego Biurrun wrote: > --- > LICENSE | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/LICENSE b/LICENSE > index 701e6e1..1eae4e7 100644 > --- a/LICENSE > +++ b/LICENSE > @@ -62,5 +62,5 @@ incompatible libraries > The Fraunhofer AAC library, FAAC and

Re: [libav-devel] [PATCH] h264dsp: Change type of stride parameters to ptrdiff_t

2016-06-13 Thread Luca Barbato
On 13/06/16 18:23, Diego Biurrun wrote: > This avoids SIMD-optimized functions having to sign-extend their > line size argument manually to be able to do pointer arithmetic. > --- > libavcodec/aarch64/h264dsp_init_aarch64.c | 37 +++ > libavcodec/arm/h264dsp_init_arm.c

[libav-devel] [PATCH] h264dsp: Change type of stride parameters to ptrdiff_t

2016-06-13 Thread Diego Biurrun
This avoids SIMD-optimized functions having to sign-extend their line size argument manually to be able to do pointer arithmetic. --- libavcodec/aarch64/h264dsp_init_aarch64.c | 37 +++ libavcodec/arm/h264dsp_init_arm.c | 37 +++

[libav-devel] [PATCH] h264: Drop unused function check_opcodes()

2016-06-13 Thread Diego Biurrun
--- libavcodec/h264_refs.c | 12 1 file changed, 12 deletions(-) diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c index c4b33af..2e43a8f 100644 --- a/libavcodec/h264_refs.c +++ b/libavcodec/h264_refs.c @@ -524,18 +524,6 @@ void ff_h264_remove_all_refs(H264Context *h)

[libav-devel] [PATCH] LICENSE: Fix silly typo

2016-06-13 Thread Diego Biurrun
--- LICENSE | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/LICENSE b/LICENSE index 701e6e1..1eae4e7 100644 --- a/LICENSE +++ b/LICENSE @@ -62,5 +62,5 @@ incompatible libraries The Fraunhofer AAC library, FAAC and OpenSSL are under licenses incompatible with all (L)GPL

Re: [libav-devel] [PATCH 004/115] eamad: Convert to the new bitstream reader

2016-06-13 Thread Diego Biurrun
On Fri, Jun 10, 2016 at 10:25:38AM +0200, Anton Khirnov wrote: > Quoting Anton Khirnov (2016-06-10 10:16:00) > > Quoting Diego Biurrun (2016-06-09 17:11:46) > > > From: Alexandra Hájková > > > --- a/libavcodec/eamad.c > > > +++ b/libavcodec/eamad.c > > > @@ -51,7 +52,7 @@

Re: [libav-devel] [PATCH 031/115] dxva2: Convert to the new bitstream reader

2016-06-13 Thread Diego Biurrun
On Mon, Jun 13, 2016 at 10:16:42AM +0200, Luca Barbato wrote: > On 09/06/16 17:12, Diego Biurrun wrote: > > From: Alexandra Hájková > > --- a/libavcodec/dxva2_h264.c > > +++ b/libavcodec/dxva2_h264.c > > @@ -20,6 +20,7 @@ > > * Foundation, Inc., 51 Franklin Street, Fifth

Re: [libav-devel] [PATCH] avdevice: Ignore timefilter test program

2016-06-13 Thread Vittorio Giovara
On Mon, Jun 13, 2016 at 11:25 AM, Diego Biurrun wrote: > On Mon, Jun 13, 2016 at 10:50:40AM -0400, Vittorio Giovara wrote: >> On Mon, Jun 13, 2016 at 8:54 AM, Diego Biurrun wrote: >> > --- /dev/null >> > +++ b/libavdevice/tests/.gitignore >> > @@ -0,0 +1 @@ >>

Re: [libav-devel] [PATCH] avdevice: Ignore timefilter test program

2016-06-13 Thread Diego Biurrun
On Mon, Jun 13, 2016 at 10:50:40AM -0400, Vittorio Giovara wrote: > On Mon, Jun 13, 2016 at 8:54 AM, Diego Biurrun wrote: > > --- /dev/null > > +++ b/libavdevice/tests/.gitignore > > @@ -0,0 +1 @@ > > +/timefilter > > -- > > sure but why? In order to ignore the generated file

Re: [libav-devel] [PATCH 090/115] unary: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 13/06/16 15:54, Diego Biurrun wrote: > On Mon, Jun 13, 2016 at 10:22:24AM +0200, Luca Barbato wrote: >> On 09/06/16 17:13, Diego Biurrun wrote: >>> From: Alexandra Hájková >>> --- a/libavcodec/unary.h >>> +++ b/libavcodec/unary.h >>> @@ -21,36 +21,37 @@ >>> /** >>> *

Re: [libav-devel] [PATCH] avdevice: Ignore timefilter test program

2016-06-13 Thread Vittorio Giovara
On Mon, Jun 13, 2016 at 8:54 AM, Diego Biurrun wrote: > --- > libavdevice/tests/.gitignore | 1 + > 1 file changed, 1 insertion(+) > create mode 100644 libavdevice/tests/.gitignore > > diff --git a/libavdevice/tests/.gitignore b/libavdevice/tests/.gitignore > new file mode

Re: [libav-devel] [PATCH 090/115] unary: Convert to the new bitstream reader

2016-06-13 Thread Diego Biurrun
On Mon, Jun 13, 2016 at 10:22:24AM +0200, Luca Barbato wrote: > On 09/06/16 17:13, Diego Biurrun wrote: > > From: Alexandra Hájková > > --- a/libavcodec/unary.h > > +++ b/libavcodec/unary.h > > @@ -21,36 +21,37 @@ > > /** > > * Get unary code of limited length > > - *

Re: [libav-devel] [PATCH 049/115] qsvenc_hevc: Convert to the new bitstream reader

2016-06-13 Thread Diego Biurrun
On Mon, Jun 13, 2016 at 10:15:50AM +0200, Luca Barbato wrote: > On 09/06/16 17:12, Diego Biurrun wrote: > > From: Alexandra Hájková > > > > --- > > libavcodec/qsvenc_hevc.c | 14 +++--- > > 1 file changed, 7 insertions(+), 7 deletions(-) > > Should be folded with

Re: [libav-devel] [PATCH 001/115] attributes: Add av_unlikely macro

2016-06-13 Thread Anton Khirnov
Quoting Hendrik Leppkes (2016-06-12 12:46:12) > On Sun, Jun 12, 2016 at 12:28 PM, Anton Khirnov wrote: > > Ok, it seems the improvement is quite dependent on the compiler version. > > I did some more measurements and the results are quite ambiguous. > > The tests all decode a

[libav-devel] [PATCH] avdevice: Ignore timefilter test program

2016-06-13 Thread Diego Biurrun
--- libavdevice/tests/.gitignore | 1 + 1 file changed, 1 insertion(+) create mode 100644 libavdevice/tests/.gitignore diff --git a/libavdevice/tests/.gitignore b/libavdevice/tests/.gitignore new file mode 100644 index 000..b4a2281 --- /dev/null +++ b/libavdevice/tests/.gitignore @@ -0,0 +1

Re: [libav-devel] [PATCH 099/115] golomb: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 09/06/16 17:13, Diego Biurrun wrote: > From: Alexandra Hájková > OK. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 089/115] huffyuv: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 09/06/16 17:13, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/huffyuv.h| 4 +- > libavcodec/huffyuvdec.c | 104 > > 2 files changed, 54 insertions(+), 54 deletions(-) > 78-89 seem ok.

Re: [libav-devel] [PATCH 090/115] unary: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 09/06/16 17:13, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/aic.c | 2 +- > libavcodec/alac.c | 2 +- > libavcodec/alsdec.c| 3 ++- > libavcodec/apedec.c| 2

Re: [libav-devel] [PATCH 076/115] mp3: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 09/06/16 17:12, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/mpegaudiodec_template.c | 203 > +++-- > 1 file changed, 102 insertions(+), 101 deletions(-) 64-76 seem ok.

Re: [libav-devel] [PATCH 063/115] vaapi_mpeg2: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 09/06/16 17:12, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/vaapi_mpeg2.c | 19 ++- > 1 file changed, 10 insertions(+), 9 deletions(-) Looks fine. ___ libav-devel mailing list

Re: [libav-devel] [PATCH 062/115] utvideodec: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 09/06/16 17:12, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/utvideodec.c | 15 --- > 1 file changed, 8 insertions(+), 7 deletions(-) 50-64 look fine ___ libav-devel mailing list

Re: [libav-devel] [PATCH 049/115] qsvenc_hevc: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 09/06/16 17:12, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/qsvenc_hevc.c | 14 +++--- > 1 file changed, 7 insertions(+), 7 deletions(-) > Should be folded with the related hevc/h264 patch. lu

Re: [libav-devel] [PATCH 048/115] qdm2: Convert to the new bitstream reader

2016-06-13 Thread Luca Barbato
On 09/06/16 17:12, Diego Biurrun wrote: > From: Alexandra Hájková > > --- > libavcodec/qdm2.c | 200 > +++--- > 1 file changed, 100 insertions(+), 100 deletions(-) 32-48 seem fine as well. lu