Re: [libav-devel] [PATCH 4/5] lavf: LucasArts SMUSH demuxer

On Mon, Mar 10, 2014 at 06:00:55PM +0100, Vittorio Giovara wrote: --- /dev/null +++ b/libavformat/smush.c @@ -0,0 +1,241 @@ + +#include libavutil/intreadwrite.h +#include avformat.h +#include internal.h +#include avio.h canonical ordering +typedef struct { +int version; +int

Re: [libav-devel] [PATCH 2/2] configure: Group and sort function/struct/type checks

On Mon, Mar 10, 2014 at 04:20:52PM +0100, Janne Grunau wrote: On 2014-03-06 14:55:06 +0100, Diego Biurrun wrote: --- a/configure +++ b/configure @@ -3805,19 +3805,19 @@ elif check_func dlopen -ldl; then if ! disabled network; then -check_type sys/types.h sys/socket.h socklen_t

Re: [libav-devel] [PATCH] fate: add a vp7 sample

On Sun, Mar 09, 2014 at 10:21:36AM +0100, Vittorio Giovara wrote: --- What's a decoder without a fate entry? I've chosen the smallest sample available and set to decoding 300 frames only. I can reduce/increase the number of frames and/or pick another sample if requested. Cheers,

Re: [libav-devel] [PATCH 2/2] configure: Group and sort function/struct/type checks

On Tue, Mar 11, 2014 at 12:21:37AM +0100, Janne Grunau wrote: On 2014-03-11 00:08:20 +0100, Diego Biurrun wrote: On Mon, Mar 10, 2014 at 04:20:52PM +0100, Janne Grunau wrote: On 2014-03-06 14:55:06 +0100, Diego Biurrun wrote: --- a/configure +++ b/configure @@ -3805,19 +3805,19

Re: [libav-devel] [PATCH] lavc: Silicon Graphics Motion Video Compressor 1 2 decoder

On Mon, Mar 10, 2014 at 10:33:27PM +0100, Vittorio Giovara wrote: --- /dev/null +++ b/libavcodec/mvcdec.c @@ -0,0 +1,287 @@ + +#include libavutil/intreadwrite.h +#include avcodec.h +#include bytestream.h +#include internal.h canonical spacing +#define PIX16(target, true, false) \ +

Re: [libav-devel] [PATCH 3/5] lavc: LucasArts SMUSH VIMA audio decoder

On Mon, Mar 10, 2014 at 06:00:54PM +0100, Vittorio Giovara wrote: From: Paul B Mahol one...@gmail.com Signed-off-by: Vittorio Giovara vittorio.giov...@gmail.com --- Which is an ADPCM decoder, so I had to move it around and place it in the right decoder list. Vittorio doc/general.texi

Re: [libav-devel] [PATCH 2/5] lavc: LucasArts SMUSH SANM video decoder

On Mon, Mar 10, 2014 at 06:00:53PM +0100, Vittorio Giovara wrote: From: Paul B Mahol one...@gmail.com Signed-off-by: Vittorio Giovara vittorio.giov...@gmail.com --- doc/general.texi |2 + libavcodec/Makefile|1 + libavcodec/allcodecs.c |1 + libavcodec/avcodec.h |

[libav-devel] [PATCH] configure: Group and sort function/struct/type checks

Group checks into logical groups and sort alphabetically within those groups unless checks have close coupling. Keep closely coupled checks together. --- Changed to address Janne's comments .. configure | 54 +++--- 1 file changed, 31

[libav-devel] THE GRAND DSPUTIL REFACTORING (act I)

Hi team! The patchset to end all patchsets is finally arriving. Or rather, the patchset to end all of dsputil. It weighs in at 132 pieces and reaches its goal slightly before the end. Here is the first installment, as a warmup exercise... Diego ___

[libav-devel] [PATCH 005/132] build: Drop stray LPC dependency on dsputil

--- configure | 1 - 1 file changed, 1 deletion(-) diff --git a/configure b/configure index 407f61c..d48e53e 100755 --- a/configure +++ b/configure @@ -1635,7 +1635,6 @@ threads_if_any=$THREADS_LIST # subsystems dct_select=rdft error_resilience_select=dsputil -lpc_select=dsputil

[libav-devel] [PATCH 004/132] build: Drop stray MJPEG encoder dependency on dsputil

--- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 5b9a955..407f61c 100755 --- a/configure +++ b/configure @@ -1725,7 +1725,7 @@ mdec_decoder_select=dsputil error_resilience mpegvideo metasound_decoder_select=lsp mdct sinewin

[libav-devel] [PATCH 001/132] build: Drop stray mpc8 decoder dependency on dsputil

--- configure | 2 +- libavcodec/mpc8.c | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/configure b/configure index 3d1fc29..55ba2ba 100755 --- a/configure +++ b/configure @@ -1740,7 +1740,7 @@ mp3float_decoder_select=mpegaudio mp3on4_decoder_select=mpegaudio

[libav-devel] [PATCH 006/132] ffv1: Drop bogus dependency on dsputil

ffv1 does not use any part of the dsputil framework. --- configure | 4 ++-- libavcodec/ffv1.c | 2 -- libavcodec/ffv1.h | 3 --- 3 files changed, 2 insertions(+), 7 deletions(-) diff --git a/configure b/configure index d48e53e..38bce0a 100755 --- a/configure +++ b/configure @@ -1683,8

[libav-devel] [PATCH 008/132] build: Record dependency of eatqi decoder on dsputil

--- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 3dd08b7..4635de7 100755 --- a/configure +++ b/configure @@ -1682,7 +1682,7 @@ eac3_decoder_select=ac3_decoder eac3_encoder_select=ac3_encoder eamad_decoder_select=aandcttables dsputil

[libav-devel] [PATCH 003/132] build: Drop stray MLP decoder dependency on dsputil

--- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index f5fe37d..5b9a955 100755 --- a/configure +++ b/configure @@ -1727,7 +1727,7 @@ mimic_decoder_select=dsputil hpeldsp mjpeg_decoder_select=dsputil hpeldsp mjpeg_encoder_select=aandcttables

[libav-devel] [PATCH 007/132] configure: Add explicit dsputil dependency to mpegvideoenc

Previously, it was an implicit dependency through mpegvideo. Change this, as implicit dependencies can cause all sorts of trouble. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 38bce0a..3dd08b7 100755 --- a/configure +++ b/configure

[libav-devel] [PATCH 002/132] build: Drop stray PNG decoder dependency on dsputil

--- configure | 1 - 1 file changed, 1 deletion(-) diff --git a/configure b/configure index 55ba2ba..f5fe37d 100755 --- a/configure +++ b/configure @@ -1760,7 +1760,6 @@ nellymoser_decoder_select=mdct sinewin nellymoser_encoder_select=audio_frame_queue mdct sinewin nuv_decoder_select=dsputil

Re: [libav-devel] [PATCH 004/132] build: Drop stray MJPEG encoder dependency on dsputil

On Tue, Mar 11, 2014 at 01:28:18PM +, Derek Buitenhuis wrote: On 3/11/2014 1:22 PM, Diego Biurrun wrote: --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) It really doesn't use dsputil? Not even for DCT? It does not. It does depend on mpegvideoenc, which in turn

[libav-devel] THE GRAND DSPUTIL REFACTORING (act II)

The first batch went through extra smooth, so here is the second: Remove some dead code and some optimizations for dead architectures. We spoke about this at VDD and FOSDEM. SH4 and SPARC are dead, no longer tested and generally not worth the trouble of maintaining anymore. [PATCH 009/132]

[libav-devel] [PATCH 013/132] dsputil: Drop ff_check_alignment() function

The function is supposed to confirm that the compiler provided enough alignment, but in practice it is only run in certain code paths and insufficient alignment problems are restricted to legacy compilers. --- libavcodec/dsputil.c | 24 libavcodec/dsputil.h |

[libav-devel] [PATCH 009/132] Remove all SH4 architecture optimizations

SH4 has been end-of-lifed and no more test machines are available. --- libavcodec/avcodec.h | 2 + libavcodec/dsputil.c | 2 - libavcodec/dsputil.h | 1 - libavcodec/options_table.h | 2 + libavcodec/sh4/Makefile | 2 - libavcodec/sh4/dsputil_sh4.c | 107

[libav-devel] [PATCH 012/132] dsputil: Remove prototypes for nonexisting optimization functions

--- libavcodec/arm/dsputil_arm.h | 1 - libavcodec/ppc/dsputil_altivec.h | 1 - 2 files changed, 2 deletions(-) diff --git a/libavcodec/arm/dsputil_arm.h b/libavcodec/arm/dsputil_arm.h index cd074df..4ac8019 100644 --- a/libavcodec/arm/dsputil_arm.h +++ b/libavcodec/arm/dsputil_arm.h @@

[libav-devel] [PATCH 011/132] x86: mpegvideoenc: Remove some remnants of the long-gone libmpeg2 IDCT

--- libavcodec/x86/mpegvideoenc_template.c | 44 -- 1 file changed, 44 deletions(-) diff --git a/libavcodec/x86/mpegvideoenc_template.c b/libavcodec/x86/mpegvideoenc_template.c index a8d2a2c..ed00e8e 100644 --- a/libavcodec/x86/mpegvideoenc_template.c +++

[libav-devel] THE GRAND DSPUTIL REFACTORING (act III)

.. the third round it is .. This time around I present miscellaneous small preparatory changes. [PATCH 014/132] dsputil: Move intreadwrite.h #include from header [PATCH 015/132] ppc: Add some missing headers [PATCH 016/132] x86: Add some more missing headers [PATCH 017/132] arm: dsputil: Add a

[libav-devel] [PATCH 017/132] arm: dsputil: Add a bunch of missing #includes

--- libavcodec/arm/dsputil_init_arm.c | 5 + libavcodec/arm/dsputil_init_armv5te.c | 5 + libavcodec/arm/dsputil_init_armv6.c | 2 ++ libavcodec/arm/dsputil_init_neon.c| 1 + 4 files changed, 13 insertions(+) diff --git a/libavcodec/arm/dsputil_init_arm.c

[libav-devel] [PATCH 015/132] ppc: Add some missing headers

--- libavcodec/ppc/dsputil_altivec.c | 1 + libavcodec/ppc/dsputil_ppc.c | 2 ++ libavcodec/ppc/vc1dsp_altivec.c | 1 + 3 files changed, 4 insertions(+) diff --git a/libavcodec/ppc/dsputil_altivec.c b/libavcodec/ppc/dsputil_altivec.c index 6f6c5f3..9342728 100644 ---

[libav-devel] [PATCH 016/132] x86: Add some more missing headers

--- libavcodec/x86/dsputil_init.c | 1 + libavcodec/x86/dsputil_x86.h | 1 + libavcodec/x86/hpeldsp_init.c | 1 + libavcodec/x86/simple_idct.c | 1 + 4 files changed, 4 insertions(+) diff --git a/libavcodec/x86/dsputil_init.c b/libavcodec/x86/dsputil_init.c index 82864e8..c62dcdd 100644 ---

[libav-devel] [PATCH 018/132] Update dsputil- and SIMD-related comments to match reality more closely

--- libavcodec/aacdec.c | 2 +- libavcodec/ac3dsp.c | 2 +- libavcodec/ac3dsp.h | 2 +- libavcodec/arm/hpeldsp_arm.S | 2 +- libavcodec/arm/hpeldsp_init_arm.c | 2 +- libavcodec/arm/videodsp_armv5te.S | 2 +-

[libav-devel] [PATCH 019/132] x86: h264_idct_10_bit: Use proper type in function prototype comments

--- libavcodec/x86/h264_idct_10bit.asm | 20 1 file changed, 12 insertions(+), 8 deletions(-) diff --git a/libavcodec/x86/h264_idct_10bit.asm b/libavcodec/x86/h264_idct_10bit.asm index 4e51d2b..66b684b 100644 --- a/libavcodec/x86/h264_idct_10bit.asm +++

[libav-devel] [PATCH 014/132] dsputil: Move intreadwrite.h #include from header file to .c files

--- libavcodec/dsputil.h | 1 - libavcodec/h264pred_template.c | 2 ++ libavcodec/h264qpel_template.c | 2 ++ libavcodec/nuv.c | 1 + libavcodec/vp3dsp.c| 1 + 5 files changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/dsputil.h

[libav-devel] [PATCH 022/132] dsputil.h: Clean up comments

--- libavcodec/dsputil.h | 85 1 file changed, 46 insertions(+), 39 deletions(-) diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h index 81efd42..56bf569 100644 --- a/libavcodec/dsputil.h +++ b/libavcodec/dsputil.h @@ -23,8 +23,8 @@

[libav-devel] [PATCH 021/132] dsputil: Remove disabled av_no_rnd_qpel leftovers

--- libavcodec/dsputil.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/libavcodec/dsputil.c b/libavcodec/dsputil.c index 12f4f5c..2b27a01 100644 --- a/libavcodec/dsputil.c +++ b/libavcodec/dsputil.c @@ -1258,9 +1258,8 @@ static void OPNAME ## qpel16_mc22_c(uint8_t *dst,

Re: [libav-devel] [PATCH 05/11] http: Add support reading ICY metadata

On Tue, Mar 11, 2014 at 03:25:26PM +0100, Luca Barbato wrote: --- a/doc/protocols.texi +++ b/doc/protocols.texi @@ -89,6 +89,26 @@ m3u8 files. + +@item icy_metadata_headers +If the server supports ICY metadata, this contains the ICY specific HTTP reply ICY-specific +headers, separated

[libav-devel] [PATCH 027/132] rnd_avg.h: KR formatting cosmetics

--- libavcodec/rnd_avg.h | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/libavcodec/rnd_avg.h b/libavcodec/rnd_avg.h index 7301578..8feac28 100644 --- a/libavcodec/rnd_avg.h +++ b/libavcodec/rnd_avg.h @@ -22,14 +22,16 @@ #include stddef.h #include stdint.h

[libav-devel] [PATCH 023/132] dsputil.h: KR formatting cosmetics

--- libavcodec/dsputil.h | 152 +-- 1 file changed, 99 insertions(+), 53 deletions(-) diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h index 56bf569..cdc2437 100644 --- a/libavcodec/dsputil.h +++ b/libavcodec/dsputil.h @@ -58,7 +58,8 @@

[libav-devel] [PATCH 026/132] dsputil: cosmetics: Lose camelCase on ff_cropTbl and ff_squareTbl names

--- libavcodec/bfin/vp3_bfin.c | 4 ++-- libavcodec/bit_depth_template.c| 2 +- libavcodec/cavs.c | 2 +- libavcodec/cavsdsp.c | 8 libavcodec/dsputil.c | 24 libavcodec/dsputil.h | 4

[libav-devel] [PATCH 025/132] dsputil: Merge variable declarations and initializations

--- libavcodec/dsputil.c | 105 ++ libavcodec/dsputil_template.c | 3 +- 2 files changed, 35 insertions(+), 73 deletions(-) diff --git a/libavcodec/dsputil.c b/libavcodec/dsputil.c index c17f35f..45efee4 100644 --- a/libavcodec/dsputil.c +++

[libav-devel] [PATCH 029/132] dsputilenc_mmx: Merge two assignment blocks with identical conditions

--- libavcodec/x86/dsputilenc_mmx.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/libavcodec/x86/dsputilenc_mmx.c b/libavcodec/x86/dsputilenc_mmx.c index a1f80af..e21ae24 100644 --- a/libavcodec/x86/dsputilenc_mmx.c +++ b/libavcodec/x86/dsputilenc_mmx.c @@ -981,9 +981,6 @@ av_cold void

[libav-devel] [PATCH 032/132] x86: motion_est: KR formatting cosmetics

--- libavcodec/x86/motion_est.c | 311 +++- 1 file changed, 161 insertions(+), 150 deletions(-) diff --git a/libavcodec/x86/motion_est.c b/libavcodec/x86/motion_est.c index 41b9c5c..d126012 100644 --- a/libavcodec/x86/motion_est.c +++

[libav-devel] [PATCH 028/132] dsputil_template: KR formatting cosmetics

--- libavcodec/dsputil_template.c | 421 ++ 1 file changed, 257 insertions(+), 164 deletions(-) diff --git a/libavcodec/dsputil_template.c b/libavcodec/dsputil_template.c index 98564eb..f6e03ec 100644 --- a/libavcodec/dsputil_template.c +++

[libav-devel] [PATCH 030/132] dsputil_mmx: KR formatting cosmetics

--- libavcodec/x86/dsputil_mmx.c | 104 --- 1 file changed, 49 insertions(+), 55 deletions(-) diff --git a/libavcodec/x86/dsputil_mmx.c b/libavcodec/x86/dsputil_mmx.c index 885c10a..d6136f6 100644 --- a/libavcodec/x86/dsputil_mmx.c +++

[libav-devel] [PATCH 034/132] x86: dsputil_init: KR formatting cosmetics

--- libavcodec/x86/dsputil_init.c | 123 +- 1 file changed, 62 insertions(+), 61 deletions(-) diff --git a/libavcodec/x86/dsputil_init.c b/libavcodec/x86/dsputil_init.c index c62dcdd..88692a0 100644 --- a/libavcodec/x86/dsputil_init.c +++

[libav-devel] [PATCH 031/132] dsputilenc_mmx: KR formatting cosmetics

--- libavcodec/x86/dsputilenc_mmx.c | 1495 --- 1 file changed, 769 insertions(+), 726 deletions(-) diff --git a/libavcodec/x86/dsputilenc_mmx.c b/libavcodec/x86/dsputilenc_mmx.c index e21ae24..2c32037 100644 --- a/libavcodec/x86/dsputilenc_mmx.c +++

[libav-devel] [PATCH 035/132] x86: dsputil_init: Drop some unnecessary parentheses

--- libavcodec/x86/dsputil_init.c | 60 +-- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/libavcodec/x86/dsputil_init.c b/libavcodec/x86/dsputil_init.c index 88692a0..00f89b2 100644 --- a/libavcodec/x86/dsputil_init.c +++

[libav-devel] [PATCH 033/132] x86: dsputil_x86.h: KR formatting cosmetics

--- libavcodec/x86/dsputil_x86.h | 17 ++--- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/libavcodec/x86/dsputil_x86.h b/libavcodec/x86/dsputil_x86.h index 990324a..8f1fc17 100644 --- a/libavcodec/x86/dsputil_x86.h +++ b/libavcodec/x86/dsputil_x86.h @@ -104,12

Re: [libav-devel] [PATCH 05/11] http: Add support reading ICY metadata

On Tue, Mar 11, 2014 at 05:47:39PM +0100, Luca Barbato wrote: On 11/03/14 17:38, Diego Biurrun wrote: On Tue, Mar 11, 2014 at 03:25:26PM +0100, Luca Barbato wrote: --- a/libavformat/http.c +++ b/libavformat/http.c @@ -68,6 +68,13 @@ typedef struct { +/* after how many bytes of read

Re: [libav-devel] [PATCH 022/132] dsputil.h: Clean up comments

On Tue, Mar 11, 2014 at 05:44:15PM +0100, Luca Barbato wrote: On 11/03/14 17:27, Diego Biurrun wrote: --- a/libavcodec/dsputil.h +++ b/libavcodec/dsputil.h @@ -23,8 +23,8 @@ /** * @file * DSP utils. - * note, many functions in here may use MMX which trashes the FPU state

[libav-devel] [PATCH] configure: Group toolchain options together in help output

--- configure | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/configure b/configure index 5395644..688da67 100755 --- a/configure +++ b/configure @@ -212,7 +212,10 @@ External library support: --enable-x11grab enable X11 grabbing [no]

Re: [libav-devel] [PATCH] configure: Group toolchain options together in help output

On Tue, Mar 11, 2014 at 06:57:31PM +0100, Luca Barbato wrote: On 11/03/14 18:53, Diego Biurrun wrote: --- configure | 14 -- 1 file changed, 8 insertions(+), 6 deletions(-) Probably it is fine but --enable-sramallow use of on-chip SRAM is unrelated

[libav-devel] [PATCH] configure: Group toolchain options together in help output

--- Now also moves --enable-sram. configure | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/configure b/configure index 5395644..add6b1b 100755 --- a/configure +++ b/configure @@ -212,7 +212,10 @@ External library support: --enable-x11grab

[libav-devel] [PATCH] configure: Document --pkg-config option

From: Luca Barbato lu_z...@gentoo.org --- configure |1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index add6b1b..e48c0fa 100755 --- a/configure +++ b/configure @@ -225,6 +225,7 @@ Toolchain options: --target-path=DIRpath to view of build directory on

Re: [libav-devel] [PATCH] configure: Provide --pkg-config-flags

On Tue, Mar 11, 2014 at 02:51:18PM +0100, Luca Barbato wrote: Should be used only to pass extra flags to pkgconf invocation, e.g. --static. Contextually add an help entry for --pkg-config Signed-off-by: Luca Barbato lu_z...@gentoo.org --- Hopefully with all the issues pointed fixed.

Re: [libav-devel] [PATCH] doc: Add section about AviSynth support

On Tue, Mar 11, 2014 at 02:38:09PM -0400, Stephen Hutchinson wrote: On Mon, Mar 10, 2014 at 7:48 PM, Timothy Gu timothyg...@gmail.com wrote: AviSynth+ doesn't have a 0.1 release: https://github.com/AviSynth/AviSynthPlus/releases While there aren't any binary packages/installers that

Re: [libav-devel] [PATCH] doc: Point to the correct, actually maintained gas-preprocessor repo

On Wed, Mar 12, 2014 at 01:47:32PM +0200, Martin Storsjö wrote: --- Just download the perl script isn't quite as easy with gitweb compared to github (or doesn't give as nice canonical urls at least) so just use the plain git url. --- doc/platform.texi | 4 ++-- 1 file changed, 2

Re: [libav-devel] [PATCH] configure: Group toolchain options together in help output

On Tue, Mar 11, 2014 at 07:08:05PM +0100, Diego Biurrun wrote: --- Now also moves --enable-sram. configure | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) OKed by Luca on IRC. Diego ___ libav-devel mailing list libav

Re: [libav-devel] [PATCH 009/132] Remove all SH4 architecture optimizations

On Wed, Mar 12, 2014 at 12:57:00PM +, Derek Buitenhuis wrote: On 3/11/2014 1:42 PM, Diego Biurrun wrote: SH4 has been end-of-lifed and no more test machines are available. --- Giant +1 from me. :) Diego ___ libav-devel mailing list libav

Re: [libav-devel] [PATCH 023/132] dsputil.h: KR formatting cosmetics

On Wed, Mar 12, 2014 at 07:35:55AM +0100, Vittorio Giovara wrote: On Tue, Mar 11, 2014 at 5:47 PM, Diego Biurrun di...@biurrun.de wrote: --- libavcodec/dsputil.h | 152 +-- 1 file changed, 99 insertions(+), 53 deletions(-) IMO 23 24

Re: [libav-devel] THE GRAND DSPUTIL REFACTORING (act I)

On Wed, Mar 12, 2014 at 12:32:24PM +, Derek Buitenhuis wrote: On 3/11/2014 1:22 PM, Diego Biurrun wrote: The patchset to end all patchsets is finally arriving. Or rather, the patchset to end all of dsputil. It weighs in at 132 pieces and reaches its goal slightly before the end. Here

Re: [libav-devel] THE GRAND DSPUTIL REFACTORING (act I)

On Wed, Mar 12, 2014 at 03:30:57PM +, Derek Buitenhuis wrote: On 3/12/2014 1:07 PM, Diego Biurrun wrote: These initial ones are rather simple, so they're getting OKed quickly. Later patches in the set are more complex and I generally prefer giving people at least 24 hours to comment

[libav-devel] [PATCH 043/132] bfin: hpeldsp: KR formatting cosmetics

--- libavcodec/bfin/hpeldsp_bfin.c | 75 ++ libavcodec/bfin/hpeldsp_bfin.h | 22 + 2 files changed, 62 insertions(+), 35 deletions(-) diff --git a/libavcodec/bfin/hpeldsp_bfin.c b/libavcodec/bfin/hpeldsp_bfin.c index c75e6bc..2dfc4c4 100644 ---

[libav-devel] [PATCH 040/132] bfin: dsputil: KR formatting cosmetics

Also add a few #endif comments. --- libavcodec/bfin/config_bfin.h | 41 +++--- libavcodec/bfin/dsputil_bfin.c | 120 + libavcodec/bfin/dsputil_bfin.h | 45 ++-- 3 files changed, 112 insertions(+), 94 deletions(-) diff --git

[libav-devel] [PATCH 036/132] bfin: dsputil: Remove unused global variable

--- libavcodec/bfin/dsputil_bfin.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/libavcodec/bfin/dsputil_bfin.c b/libavcodec/bfin/dsputil_bfin.c index 3ce2941..ba823f5 100644 --- a/libavcodec/bfin/dsputil_bfin.c +++ b/libavcodec/bfin/dsputil_bfin.c @@ -26,8 +26,6 @@ #include

[libav-devel] [PATCH 038/132] bfin: hpeldsp: Drop broken put_pixels_*_xy2_nornd bits

They were marked as broken and disabled in 2009 and will never get fixed. --- libavcodec/bfin/hpel_pixels_bfin.S | 109 - libavcodec/bfin/hpeldsp_bfin.c | 2 - libavcodec/bfin/hpeldsp_bfin.h | 3 - 3 files changed, 114 deletions(-) diff --git

[libav-devel] [PATCH 037/132] bfin: dsputil: Drop broken vsad bits

They were marked as broken and disabled in 2009 and will never get fixed. --- libavcodec/bfin/dsputil_bfin.c | 12 1 file changed, 12 deletions(-) diff --git a/libavcodec/bfin/dsputil_bfin.c b/libavcodec/bfin/dsputil_bfin.c index ba823f5..ed1 100644 ---

[libav-devel] THE GRAND DSPUTIL REFACTORING (act IV)

The next batch is about the Blackfin architecture optimizations. They're in a state of moderate disrepair and this set does some general refactoring. At FOSDEM Martin raised the issue of the general usefulness of the Blackfin architecture optimizations, arguing that they are not worth the

[libav-devel] [PATCH 041/132] bfin: Refactor duplicated l1_text/l1_data_B macros

--- libavcodec/bfin/dsputil_bfin.h | 9 + libavcodec/bfin/hpeldsp_bfin.h | 10 +- .../bfin/hpeldsp_bfin.h = libavutil/bfin/attributes.h | 18 -- libswscale/bfin/swscale_bfin.c | 13

[libav-devel] [PATCH 047/132] bfin: hpeldsp: Split no_rnd pixel operations off into a separate file

This saves some space in configurations that do not enable hpeldsp. --- libavcodec/bfin/Makefile | 1 + libavcodec/bfin/hpel_pixels_bfin.S | 64 libavcodec/bfin/hpel_pixels_no_rnd.S | 81 3 files changed, 82

[libav-devel] [PATCH 045/132] bfin: hpeldsp: nornd --- no_rnd function name suffix for consistency

--- libavcodec/bfin/hpel_pixels_bfin.S | 8 ++--- libavcodec/bfin/hpeldsp_bfin.c | 60 +++--- 2 files changed, 34 insertions(+), 34 deletions(-) diff --git a/libavcodec/bfin/hpel_pixels_bfin.S b/libavcodec/bfin/hpel_pixels_bfin.S index d311fdf..981b4e3

[libav-devel] [PATCH 039/132] bfin: dsputil: Drop broken and trivial profiling code

--- libavcodec/bfin/dsputil_bfin.h | 40 1 file changed, 40 deletions(-) diff --git a/libavcodec/bfin/dsputil_bfin.h b/libavcodec/bfin/dsputil_bfin.h index a082cd7..4916b56 100644 --- a/libavcodec/bfin/dsputil_bfin.h +++ b/libavcodec/bfin/dsputil_bfin.h

[libav-devel] [PATCH 044/132] bfin: Move all assembly declarations only used within one .c file there

This is done similarly for all other architectures. --- libavcodec/bfin/dsputil_bfin.c | 33 ++- libavcodec/bfin/dsputil_bfin.h | 59 -- libavcodec/bfin/hpeldsp_bfin.c | 8 ++ libavcodec/bfin/hpeldsp_bfin.h | 6 -

[libav-devel] [PATCH 048/132] bfin: Use more sensible file names

Drop non-informative _bfin suffix, rename some files for consistency with other architectures, rename others to reflect their content. --- libavcodec/bfin/Makefile | 12 ++-- libavcodec/bfin/{pixels_bfin.S = dsputil.S} | 0 libavcodec/bfin/{dsputil_bfin.c

[libav-devel] [PATCH 046/132] bfin: dsputil: Unconditionally compile hpel pixels code

The functions it contains are used from unconditionally compiled code. --- libavcodec/bfin/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/bfin/Makefile b/libavcodec/bfin/Makefile index fc5f506..62881c4 100644 --- a/libavcodec/bfin/Makefile +++

[libav-devel] [PATCH 042/132] bfin: Refactor duplicated assembly-related macros

--- libavcodec/bfin/fdct_bfin.S| 17 + libavcodec/bfin/hpel_pixels_bfin.S | 2 +- libavcodec/bfin/idct_bfin.S| 13 +++--- libavcodec/bfin/pixels_bfin.S | 3 ++- libavcodec/bfin/vp3_idct_bfin.S

Re: [libav-devel] [PATCH] avconv: Match stream id

On Wed, Mar 12, 2014 at 06:07:06PM +0100, Luca Barbato wrote: Quite useful to select by pid in mpegts It is quite useful to select by pid in MPEG-TS. --- a/doc/avtools-common-opts.texi +++ b/doc/avtools-common-opts.texi @@ -42,6 +42,8 @@ streams of this type. @item

Re: [libav-devel] THE GRAND DSPUTIL REFACTORING (act IV)

On Wed, Mar 12, 2014 at 11:39:24PM +0100, Luca Barbato wrote: On 12/03/14 22:55, Vittorio Giovara wrote: I think I remember the discussion ended up that if bfin sponsored a server for fate we could have kept them around. Has anyone tried contacting them? Not a server but some access to a

[libav-devel] [PATCH 2/2] avcodec: more correct printf specifiers

--- libavcodec/4xm.c|4 +++- libavcodec/aic.c|4 +++- libavcodec/alac.c |9 ++--- libavcodec/alsdec.c |9 +++-- libavcodec/apedec.c |5 - libavcodec/bmp.c| 21 ++--- libavcodec/cllc.c

[libav-devel] [PATCH 1/2] avfilter: more correct printf format specifiers

--- libavfilter/af_ashowinfo.c |4 ++-- libavfilter/vf_showinfo.c |4 +++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/libavfilter/af_ashowinfo.c b/libavfilter/af_ashowinfo.c index 2a2edcf..99d9dde 100644 --- a/libavfilter/af_ashowinfo.c +++

Re: [libav-devel] [PATCH 11/11] fate: add SGI tests

On Thu, Mar 13, 2014 at 04:27:29PM +0100, Vittorio Giovara wrote: --- a/tests/fate/image.mak +++ b/tests/fate/image.mak @@ -30,6 +30,35 @@ fate-pngparser: CMD = framecrc -f image2pipe -i $(TARGET_SAMPLES)/png1/libav_4x_ +FATE_SGI += fate-sgi-rgba +fate-sgi-rgba: CMD = framecrc -i

[libav-devel] [PATCH 051/132] hpeldsp_template: KR formatting cosmetics

--- libavcodec/hpeldsp_template.c | 510 -- 1 file changed, 297 insertions(+), 213 deletions(-) diff --git a/libavcodec/hpeldsp_template.c b/libavcodec/hpeldsp_template.c index 6518941..76c3fd3 100644 --- a/libavcodec/hpeldsp_template.c +++

[libav-devel] THE GRAND DSPUTIL REFACTORING (act V)

This part is the last bit of cleanup before reaching the meat of the set. [PATCH 049/132] hpel_template: KR formatting cosmetics [PATCH 050/132] hpel_template: Drop some unnecessary parentheses [PATCH 051/132] hpeldsp_template: KR formatting cosmetics [PATCH 052/132] hpeldsp_template: Merge some

[libav-devel] [PATCH 053/132] ppc: dsputil: comment formatting and wording/grammar improvements

--- libavcodec/ppc/dsputil_altivec.c | 233 +++ libavcodec/ppc/dsputil_ppc.c | 60 +- libavcodec/ppc/fdct_altivec.c| 7 +- libavcodec/ppc/fft_altivec.c | 12 +- libavcodec/ppc/gmc_altivec.c | 40 +++

[libav-devel] [PATCH 050/132] hpel_template: Drop some unnecessary parentheses

--- libavcodec/hpel_template.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/hpel_template.c b/libavcodec/hpel_template.c index 8f99950..0d90445 100644 --- a/libavcodec/hpel_template.c +++ b/libavcodec/hpel_template.c @@ -27,7 +27,7 @@ static inline void

[libav-devel] [PATCH 052/132] hpeldsp_template: Merge some declarations and initializations

--- libavcodec/hpeldsp_template.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/hpeldsp_template.c b/libavcodec/hpeldsp_template.c index 76c3fd3..9456490 100644 --- a/libavcodec/hpeldsp_template.c +++ b/libavcodec/hpeldsp_template.c @@ -133,14 +133,14 @@

[libav-devel] [PATCH 049/132] hpel_template: KR formatting cosmetics

--- libavcodec/hpel_template.c | 189 - 1 file changed, 117 insertions(+), 72 deletions(-) diff --git a/libavcodec/hpel_template.c b/libavcodec/hpel_template.c index 0c1f756..8f99950 100644 --- a/libavcodec/hpel_template.c +++

[libav-devel] [PATCH 057/132] ppc: dsputil: Simplify some ifdeffed function definitions

--- libavcodec/ppc/dsputil_ppc.c | 18 +- 1 file changed, 5 insertions(+), 13 deletions(-) diff --git a/libavcodec/ppc/dsputil_ppc.c b/libavcodec/ppc/dsputil_ppc.c index eac0d3d..1bd11b1 100644 --- a/libavcodec/ppc/dsputil_ppc.c +++ b/libavcodec/ppc/dsputil_ppc.c @@ -74,9 +74,9

[libav-devel] [PATCH 059/132] ppc: dsputil: Drop trailing semicolon from macros

This allows for a more natural macro usage. --- libavcodec/ppc/fdct_altivec.c | 10 - libavcodec/ppc/idct_altivec.c | 48 +-- 2 files changed, 29 insertions(+), 29 deletions(-) diff --git a/libavcodec/ppc/fdct_altivec.c

[libav-devel] [PATCH 056/132] ppc: dsputil: Drop some unnecessary parentheses

--- libavcodec/ppc/dsputil_altivec.c | 12 ++-- libavcodec/ppc/dsputil_ppc.c | 6 +++--- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/libavcodec/ppc/dsputil_altivec.c b/libavcodec/ppc/dsputil_altivec.c index c075c44..2091023 100644 ---

[libav-devel] [PATCH 055/132] ppc: dsputil: more KR formatting cosmetics

--- libavcodec/ppc/fdct_altivec.c | 384 +- libavcodec/ppc/gmc_altivec.c | 83 - libavcodec/ppc/idct_altivec.c | 295 libavcodec/ppc/int_altivec.c | 91 +- 4 files changed, 437 insertions(+), 416

[libav-devel] [PATCH 058/132] ppc: dsputil: Merge some declarations and initializations

--- libavcodec/ppc/dsputil_altivec.c | 403 ++- libavcodec/ppc/dsputil_ppc.c | 9 +- libavcodec/ppc/fdct_altivec.c| 3 +- libavcodec/ppc/gmc_altivec.c | 31 ++- libavcodec/ppc/idct_altivec.c| 37 ++-- libavcodec/ppc/int_altivec.c | 6

[libav-devel] [PATCH 060/132] arm: dsputil: KR formatting cosmetics

--- libavcodec/arm/dsputil_arm.h| 4 ++-- libavcodec/arm/dsputil_init_arm.c | 28 +--- libavcodec/arm/dsputil_init_armv6.c | 4 ++-- libavcodec/arm/dsputil_init_neon.c | 14 -- 4 files changed, 29 insertions(+), 21 deletions(-) diff --git

Re: [libav-devel] [PATCH 2/2] configure: Split some logical blocks off from HAVE_LIST

On Fri, Mar 07, 2014 at 02:04:32PM +0100, Diego Biurrun wrote: --- If this is too much of a hassle to review, I could split it further. configure | 193 +++-- 1 file changed, 110 insertions(+), 83 deletions(-) OKed by Luca on IRC

Re: [libav-devel] [PATCH 1/2] configure: Split some logical blocks off from CONFIG_LIST

On Fri, Mar 07, 2014 at 02:04:31PM +0100, Diego Biurrun wrote: --- configure | 54 +- 1 file changed, 33 insertions(+), 21 deletions(-) OKed by Luca on IRC. Diego ___ libav-devel mailing list

Re: [libav-devel] [PATCH] lavc: add OpenEXR decoder

On Mon, Mar 10, 2014 at 09:28:53PM +0100, Vittorio Giovara wrote: --- /dev/null +++ b/libavcodec/exr.c @@ -0,0 +1,1346 @@ +/** + * Get the size of the header variable. + * + * @param buf the current pointer location in the header where + * the variable data starts + * @param buf_end

Re: [libav-devel] [PATCH] [RFC] configure: set 'cc' as default compiler

On Thu, Mar 13, 2014 at 09:59:28PM -0400, Sean McGovern wrote: On Thu, Mar 13, 2014 at 8:51 PM, Vittorio Giovara vittorio.giov...@gmail.com wrote: --- cc seems to work on osx and linux, it would allow skipping --cc=clang at configure on modern osx, but I'm not sure of the overall impact.

Re: [libav-devel] [PATCH] Announce Releases 10_beta2, 9.12 and 0.8.11

On Thu, Mar 13, 2014 at 09:58:23PM -0400, Reinhard Tartler wrote: --- a/src/download +++ b/src/download @@ -201,50 +201,50 @@ and much faster bug fixes such as additional features and security patches. p -10 beta 1 has been released on 2014-02-17. It is the latest beta snapshot from

Re: [libav-devel] [PATCH 2/5] avformat: Rename internal function

On Thu, Mar 13, 2014 at 10:35:40PM +0100, Luca Barbato wrote: It is generic enough to be reused for other similar coma-separated fields. com_m_a Diego ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH] arm: dsputil: Add missing config.h #include for restrict keyword

--- libavcodec/arm/dsputil_init_armv6.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/arm/dsputil_init_armv6.c b/libavcodec/arm/dsputil_init_armv6.c index 357ae1e..ce64a85 100644 --- a/libavcodec/arm/dsputil_init_armv6.c +++ b/libavcodec/arm/dsputil_init_armv6.c @@ -20,6 +20,7

Re: [libav-devel] [PATCH] arm: dsputil: Add missing config.h #include for restrict keyword

On Fri, Mar 14, 2014 at 12:26:05PM +0200, Martin Storsjö wrote: On Fri, 14 Mar 2014, Diego Biurrun wrote: --- a/libavcodec/arm/dsputil_init_armv6.c +++ b/libavcodec/arm/dsputil_init_armv6.c @@ -20,6 +20,7 @@ +#include config.h #include libavutil/attributes.h #include libavcodec/avcodec.h

[libav-devel] [PATCH] arm: dsputil: Drop restrict keyword from add_pixels_clamped_armv6 prototype

The function is assigned to a function pointer that does not have the restrict keyword for that parameter. This fixes compilation for MSVC builds that don't recognize restrict. --- libavcodec/arm/dsputil_init_armv6.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[libav-devel] [PATCH 065/132] nuv: Reuse the DSPContext from RTJpegContext

There is no point in populating NuvContext with another DSPContext. --- libavcodec/nuv.c| 12 libavcodec/rtjpeg.c | 11 --- libavcodec/rtjpeg.h | 5 ++--- 3 files changed, 10 insertions(+), 18 deletions(-) diff --git a/libavcodec/nuv.c b/libavcodec/nuv.c index

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