[libav-devel] [PATCH 4/4] avcodec/hap: Support chunked Hap frames

2015-07-17 Thread Tom Butterworth
--- libavcodec/Makefile | 4 +- libavcodec/hap.c | 51 + libavcodec/hap.h | 68 libavcodec/hapdec.c | 278 --- libavcodec/hapenc.c | 190 ++-- tests/fate/video.mak |

[libav-devel] [PATCH 3/4] avcodec/snappy: refactor so ff_snappy_uncompress uses an existing buffer

2015-07-17 Thread Tom Butterworth
Some uses of Snappy require uncompressing to positions within an existing buffer. Also adds a function to get the uncompressed length of Snappy data. --- libavcodec/hapdec.c | 7 ++- libavcodec/snappy.c | 24 +--- libavcodec/snappy.h | 19 ++- 3 files

[libav-devel] [PATCH 2/4] avcodec/hap: move some per-stream setup into decoder init rather than per-frame

2015-07-17 Thread Tom Butterworth
This change will reject frames with a texture type which doesn't match the stream description. --- libavcodec/hapdec.c | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) diff --git a/libavcodec/hapdec.c b/libavcodec/hapdec.c index ca1c158..324a43a 100644 ---

[libav-devel] [PATCH 1/4] lavc/hapenc: Silence a warning for multithreaded encoding.

2015-07-17 Thread Tom Butterworth
From: Carl Eugen Hoyos ceho...@ag.or.at --- libavcodec/hapdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/hapdec.c b/libavcodec/hapdec.c index 72db9f4..ca1c158 100644 --- a/libavcodec/hapdec.c +++ b/libavcodec/hapdec.c @@ -177,7 +177,8 @@ static int

Re: [libav-devel] [PATCH 1/2] hap: Decode using optimal slices sizes

2015-07-17 Thread Luca Barbato
On 17/07/15 11:32, Tom Butterworth wrote: see also https://ffmpeg.org/pipermail/ffmpeg-devel/2015-July/175567.html Interesting approach, but I'd rather avoid the overhead to begin with. You are welcome to send patches here btw. lu ___ libav-devel

[libav-devel] [PATCH 8/9] asfdec: avoid crash in the case when chunk_len is 0

2015-07-17 Thread Alexandra Hájková
--- libavformat/asfdec.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index c522bd2..8e85749 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -780,6 +780,8 @@ static int asf_read_stream_properties(AVFormatContext *s, const

[libav-devel] [PATCH 9/9] asfdec: ommit the wrong condition

2015-07-17 Thread Alexandra Hájková
--- libavformat/asfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 8e85749..44f2438 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -909,7 +909,7 @@ static int asf_read_data(AVFormatContext *s, const

[libav-devel] [PATCH 7/9] asfdec: make nb_sub to be unsigned int

2015-07-17 Thread Alexandra Hájková
number of subpayloads should be always positive --- libavformat/asfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 1fe8bf4..c522bd2 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -134,7 +134,7 @@ typedef

[libav-devel] [PATCH 6/9] asfdec: read the replicated data in a separate function

2015-07-17 Thread Alexandra Hájková
--- libavformat/asfdec.c | 37 - 1 file changed, 24 insertions(+), 13 deletions(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 4afa0f4..1fe8bf4 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -1109,6 +1109,27 @@ static void

[libav-devel] [PATCH 4/9] asfdec: do not align an object when broadcast flag is set

2015-07-17 Thread Alexandra Hájková
the object size is invalid in this case --- libavformat/asfdec.c | 35 --- 1 file changed, 20 insertions(+), 15 deletions(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 7146dcc..ad08649 100644 --- a/libavformat/asfdec.c +++

[libav-devel] [PATCH 5/9] asfdec: convert condition for the replicated data reading to be safer

2015-07-17 Thread Alexandra Hájková
--- libavformat/asfdec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index ad08649..4afa0f4 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -1137,7 +1137,7 @@ static int

[libav-devel] [PATCH 3/9] asfdec: do not read replicated data when their length is 0

2015-07-17 Thread Alexandra Hájková
--- libavformat/asfdec.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 7458e19..7146dcc 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -1123,17 +1123,19 @@ static int

[libav-devel] [PATCH 2/9] asfdec: close the demuxer properly when read_header is failing

2015-07-17 Thread Alexandra Hájková
--- libavformat/asfdec.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 7d28f18..7458e19 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -1646,14 +1646,14 @@ static int

[libav-devel] [PATCH 1/9] asfdec: factor out seeking to the Data Object outside while

2015-07-17 Thread Alexandra Hájková
return INVALIDDATA if Data Object wasn't found --- libavformat/asfdec.c | 29 - 1 file changed, 16 insertions(+), 13 deletions(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 7af3728..7d28f18 100644 --- a/libavformat/asfdec.c +++

Re: [libav-devel] [PATCH 9/9] asfdec: ommit the wrong condition

2015-07-17 Thread Luca Barbato
On 17/07/15 14:17, Alexandra Hájková wrote: --- libavformat/asfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 8e85749..44f2438 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -909,7 +909,7 @@

Re: [libav-devel] [PATCH 2/4] checkasm: test all architectures with optimisations

2015-07-17 Thread Henrik Gramner
lgtm. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] parseutil: Add more resolution aliases

2015-07-17 Thread Vittorio Giovara
On Fri, Jul 17, 2015 at 2:27 AM, Luca Barbato lu_z...@gentoo.org wrote: On 17/07/15 01:44, Vittorio Giovara wrote: On Thu, Jul 16, 2015 at 11:33 PM, Luca Barbato lu_z...@gentoo.org wrote: Add DCI 4k and 2160p. --- Yes, I'm testing which encoders are able to do something with 4k now =)

Re: [libav-devel] [PATCH 1/4] lavc/hapenc: Silence a warning for multithreaded encoding.

2015-07-17 Thread Luca Barbato
On 17/07/15 14:11, Tom Butterworth wrote: From: Carl Eugen Hoyos ceho...@ag.or.at --- libavcodec/hapdec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/hapdec.c b/libavcodec/hapdec.c index 72db9f4..ca1c158 100644 --- a/libavcodec/hapdec.c +++

Re: [libav-devel] [PATCH 1/4] lavc/hapenc: Silence a warning for multithreaded encoding.

2015-07-17 Thread Tom Butterworth
okay in that case I'll submit a new patch set which omits this, but will await any comments on the other patches in this one first On 17 July 2015 at 14:04, Luca Barbato lu_z...@gentoo.org wrote: On 17/07/15 14:11, Tom Butterworth wrote: From: Carl Eugen Hoyos ceho...@ag.or.at ---

Re: [libav-devel] [PATCH 2/4] avcodec/hap: move some per-stream setup into decoder init rather than per-frame

2015-07-17 Thread Luca Barbato
On 17/07/15 14:11, Tom Butterworth wrote: This change will reject frames with a texture type which doesn't match the stream description. --- libavcodec/hapdec.c | 27 +-- 1 file changed, 21 insertions(+), 6 deletions(-) I doubt somebody would generate a stream

Re: [libav-devel] [PATCH 2/4] avcodec/hap: move some per-stream setup into decoder init rather than per-frame

2015-07-17 Thread Vittorio Giovara
Hi Tom, On Fri, Jul 17, 2015 at 1:11 PM, Tom Butterworth bangno...@gmail.com wrote: This change will reject frames with a texture type which doesn't match the stream description. Is this change required by the format specification or just as a precaution for malformed streams? ---

Re: [libav-devel] [PATCH 3/4] avcodec/snappy: refactor so ff_snappy_uncompress uses an existing buffer

2015-07-17 Thread Vittorio Giovara
On Fri, Jul 17, 2015 at 1:11 PM, Tom Butterworth bangno...@gmail.com wrote: Some uses of Snappy require uncompressing to positions within an existing buffer. Also adds a function to get the uncompressed length of Snappy data. --- libavcodec/hapdec.c | 7 ++- libavcodec/snappy.c | 24

Re: [libav-devel] [PATCH] asfdec: do not read replicated data when their length is 0

2015-07-17 Thread Luca Barbato
On 16/07/15 12:14, Alexandra Hájková wrote: --- libavformat/asfdec.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) Will you send an update with that block spun out as function? lu ___ libav-devel mailing list

Re: [libav-devel] [PATCH] OS/2:Cleanup slib_create_def_cmd

2015-07-17 Thread Luca Barbato
On 17/07/15 07:21, Dave Yeo wrote: From 7aa9c4c5035f777c9c08b49c3af256edf36f4961 Mon Sep 17 00:00:00 2001 From: Dave Yeo dave.r@gmail.com Date: Thu, 16 Jul 2015 21:13:49 -0700 Subject: [PATCH] OS/2:Cleanup slib_create_def_cmd Export symbols by name rather then ordinal. Remove PROTMODE

Re: [libav-devel] [PATCH 4/4] avcodec/hap: Support chunked Hap frames

2015-07-17 Thread Vittorio Giovara
On Fri, Jul 17, 2015 at 1:11 PM, Tom Butterworth bangno...@gmail.com wrote: --- libavcodec/Makefile | 4 +- libavcodec/hap.c | 51 + libavcodec/hap.h | 68 libavcodec/hapdec.c | 278 ---

[libav-devel] [PATCH 2/2] tests/checkasm/checkasm: Give macro a body to avoid potential unexpected syntax issues

2015-07-17 Thread Henrik Gramner
From: Michael Niedermayer mich...@niedermayer.cc Signed-off-by: Michael Niedermayer mich...@niedermayer.cc --- tests/checkasm/checkasm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/checkasm/checkasm.h b/tests/checkasm/checkasm.h index 1a46e9b..b54be16 100644 ---

Re: [libav-devel] [PATCH 4/4] avcodec/hap: Support chunked Hap frames

2015-07-17 Thread Vittorio Giovara
On Fri, Jul 17, 2015 at 1:11 PM, Tom Butterworth bangno...@gmail.com wrote: --- libavcodec/Makefile | 4 +- libavcodec/hap.c | 51 + libavcodec/hap.h | 68 libavcodec/hapdec.c | 278 ---

[libav-devel] [PATCH 1/2] checkasm: exit with status 0 instead of 1 if there are no tests to perform

2015-07-17 Thread Henrik Gramner
--- tests/checkasm/checkasm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c index 7b1ea8f..0aa3d1c 100644 --- a/tests/checkasm/checkasm.c +++ b/tests/checkasm/checkasm.c @@ -317,7 +317,7 @@ int main(int argc, char

Re: [libav-devel] [PATCH 1/2] hap: Decode using optimal slices sizes

2015-07-17 Thread Tom Butterworth
see also https://ffmpeg.org/pipermail/ffmpeg-devel/2015-July/175567.html On 17 July 2015 at 02:07, Luca Barbato lu_z...@gentoo.org wrote: Enjoy some cache locality and use less threads. About 5x speedup (from 60ms to 12ms to decode a 4k frame). --- libavcodec/hap.h| 2 ++

Re: [libav-devel] [PATCH 1/2] checkasm: exit with status 0 instead of 1 if there are no tests to perform

2015-07-17 Thread Janne Grunau
On 2015-07-17 18:07:57 +0200, Henrik Gramner wrote: --- tests/checkasm/checkasm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c index 7b1ea8f..0aa3d1c 100644 --- a/tests/checkasm/checkasm.c +++

Re: [libav-devel] [PATCH 2/2] tests/checkasm/checkasm: Give macro a body to avoid potential unexpected syntax issues

2015-07-17 Thread Janne Grunau
n 2015-07-17 18:07:58 +0200, Henrik Gramner wrote: From: Michael Niedermayer mich...@niedermayer.cc Signed-off-by: Michael Niedermayer mich...@niedermayer.cc --- tests/checkasm/checkasm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/checkasm/checkasm.h

[libav-devel] [PATCH 1/1] arm: videodsp: add 'it' for thumb before conditional branch

2015-07-17 Thread Janne Grunau
From: Csaba Osztrogonác o...@webkit.org The Tizen 3.0 toolchain seems not to be able to calculate the offset over the implicit it instruction resulting in an out of range relocation. tested-by: Ludovic Fauvet e...@videolan.org Bug-Id: https://bugs.webkit.org/show_bug.cgi?id=137022 CC:

Re: [libav-devel] [PATCH] cosmetics: Reformat checkasm tests

2015-07-17 Thread Henrik Gramner
On Fri, Jul 17, 2015 at 8:08 PM, Luca Barbato lu_z...@gentoo.org wrote: -qpel_mc_func (*tab)[16] = op ? h.avg_h264_qpel_pixels_tab : h.put_h264_qpel_pixels_tab; +qpel_mc_func(*tab)[16] = op ? h.avg_h264_qpel_pixels_tab : h.put_h264_qpel_pixels_tab; No space between type and

Re: [libav-devel] [PATCH 1/1] arm: videodsp: add 'it' for thumb before conditional branch

2015-07-17 Thread Luca Barbato
On 17/07/15 19:18, Janne Grunau wrote: From: Csaba Osztrogonác o...@webkit.org The Tizen 3.0 toolchain seems not to be able to calculate the offset over the implicit it instruction resulting in an out of range relocation. tested-by: Ludovic Fauvet e...@videolan.org Bug-Id:

[libav-devel] [PATCH] cosmetics: Reformat checkasm tests

2015-07-17 Thread Luca Barbato
--- Since I forget to do that before pushing the changes and I noticed that the set of macros is being more or less copy-pasted. tests/checkasm/bswapdsp.c | 46 +++--- tests/checkasm/h264pred.c | 36 ++--

Re: [libav-devel] [PATCH] cosmetics: Reformat checkasm tests

2015-07-17 Thread Luca Barbato
On 17/07/15 21:21, Henrik Gramner wrote: On Fri, Jul 17, 2015 at 8:08 PM, Luca Barbato lu_z...@gentoo.org wrote: -qpel_mc_func (*tab)[16] = op ? h.avg_h264_qpel_pixels_tab : h.put_h264_qpel_pixels_tab; +qpel_mc_func(*tab)[16] = op ? h.avg_h264_qpel_pixels_tab :

Re: [libav-devel] [PATCH] parseutil: Add more resolution aliases

2015-07-17 Thread Vittorio Giovara
On Fri, Jul 17, 2015 at 9:22 PM, Luca Barbato lu_z...@gentoo.org wrote: On 17/07/15 15:18, Vittorio Giovara wrote: On Fri, Jul 17, 2015 at 2:27 AM, Luca Barbato lu_z...@gentoo.org wrote: On 17/07/15 01:44, Vittorio Giovara wrote: On Thu, Jul 16, 2015 at 11:33 PM, Luca Barbato

Re: [libav-devel] [PATCH] parseutil: Add more resolution aliases

2015-07-17 Thread Luca Barbato
On 17/07/15 15:18, Vittorio Giovara wrote: On Fri, Jul 17, 2015 at 2:27 AM, Luca Barbato lu_z...@gentoo.org wrote: On 17/07/15 01:44, Vittorio Giovara wrote: On Thu, Jul 16, 2015 at 11:33 PM, Luca Barbato lu_z...@gentoo.org wrote: Add DCI 4k and 2160p. --- Yes, I'm testing which encoders

Re: [libav-devel] [PATCH] parseutil: Add more resolution aliases

2015-07-17 Thread Luca Barbato
On 18/07/15 00:00, Vittorio Giovara wrote: On Fri, Jul 17, 2015 at 9:22 PM, Luca Barbato lu_z...@gentoo.org wrote: On 17/07/15 15:18, Vittorio Giovara wrote: On Fri, Jul 17, 2015 at 2:27 AM, Luca Barbato lu_z...@gentoo.org wrote: On 17/07/15 01:44, Vittorio Giovara wrote: On Thu, Jul 16,

Re: [libav-devel] [PATCH 1/1] arm: videodsp: add 'it' for thumb before conditional branch

2015-07-17 Thread Janne Grunau
On 2015-07-17 19:18:50 +0200, Janne Grunau wrote: From: Csaba Osztrogonác o...@webkit.org The Tizen 3.0 toolchain seems not to be able to calculate the offset over the implicit it instruction resulting in an out of range relocation. tested-by: Ludovic Fauvet e...@videolan.org Bug-Id: