Re: [libav-devel] [PATCH] dsputil: x86: Fix compile error.

2013-02-07 Thread Martin Storsjö
On Wed, 6 Feb 2013, “Daniel wrote: From: Daniel Kang daniel.d.k...@gmail.com Accidentally prefixed ff_ with cextern --- Can someone test on MSVC? --- libavcodec/x86/dsputil.asm |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Confirmed that this fixes the msvc build. //

Re: [libav-devel] [PATCH] dsputil: x86: Fix compile error.

2013-02-07 Thread Luca Barbato
On 07/02/13 04:25, “Daniel wrote: From: Daniel Kang daniel.d.k...@gmail.com Accidentally prefixed ff_ with cextern I tested your patch on linux + amd64 and I assumed you tested on the other arches, now excelsior has mingw64 as well in the compile-test script. Please rework the commit message

[libav-devel] [PATCH] avplay: apply the stream aspect ratio to libavfilter'ed frames

2013-02-07 Thread Vladimir Pantelic
So far we took only the codec sample aspect ratio into account and ignored the one from the stream when using libavfilter. Signed-off-by: Vladimir Pantelic vlado...@gmail.com --- avplay.c |6 ++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/avplay.c b/avplay.c index

Re: [libav-devel] [PATCH] avplay: apply the stream aspect ratio to libavfilter'ed frames

2013-02-07 Thread Luca Barbato
On 07/02/13 14:09, Vladimir Pantelic wrote: So far we took only the codec sample aspect ratio into account and ignored the one from the stream when using libavfilter. Signed-off-by: Vladimir Pantelic vlado...@gmail.com --- avplay.c |6 ++ 1 files changed, 6 insertions(+), 0

[libav-devel] [PATCH] h264 sps cropping (right and bottom)

2013-02-07 Thread Vittorio Giovara
Thanks for the tip, I think i got it working. I cannot modify the values ff_get_buffer() as I go out of a H264Context and lose the sps information, so I applied the cropping directly on the AVFrame at the end of decode_frame(). This should be syntactically correct according to the spec anyways.

Re: [libav-devel] [PATCH] h264: deMpegEncContextize

2013-02-07 Thread Luca Barbato
On 06/02/13 19:40, Anton Khirnov wrote: Most of the changes are just trivial are just trivial replacements of fields from MpegEncContext with equivalent fields in H264Context. Everything in h264* other than h264.c are those trivial changes. The nontrivial parts are: 1) extracting a

Re: [libav-devel] [PATCH] h264 sps cropping (right and bottom)

2013-02-07 Thread Luca Barbato
On 07/02/13 16:08, Vittorio Giovara wrote: Thanks for the tip, I think i got it working. I cannot modify the values ff_get_buffer() as I go out of a H264Context and lose the sps information, so I applied the cropping directly on the AVFrame at the end of decode_frame(). This should be

Re: [libav-devel] [PATCH] avplay: apply the stream aspect ratio to libavfilter'ed frames

2013-02-07 Thread Vladimir Pantelic
On 02/07/2013 02:34 PM, Luca Barbato wrote: On 07/02/13 14:09, Vladimir Pantelic wrote: So far we took only the codec sample aspect ratio into account and ignored the one from the stream when using libavfilter. Signed-off-by: Vladimir Pantelicvlado...@gmail.com --- avplay.c |6 ++ 1

[libav-devel] [PATCH v2] avplay: apply the stream sample_aspect_ratio to decoded video frames

2013-02-07 Thread Vladimir Pantelic
If there is a sample_aspect_ratio in the stream, then apply it to every decoded frame in the same way as avconv does. This also makes sure that the avfilter chain has access to the aspect ratio. Signed-off-by: Vladimir Pantelic vlado...@gmail.com --- avplay.c |3 +++ 1 files changed, 3

Re: [libav-devel] [PATCH 057/124] vble: convert to refcounted frames.

2013-02-07 Thread Derek Buitenhuis
On 2013-01-08 9:37 AM, Anton Khirnov wrote: --- libavcodec/vble.c | 35 --- 1 file changed, 8 insertions(+), 27 deletions(-) OK. +AVFrame *pic = data; [...] -*(AVFrame *)data = *pic; Never thought this while song and dance made sense anyway.

Re: [libav-devel] [PATCH 050/124] cllc: convert to refcounted frames.

2013-02-07 Thread Derek Buitenhuis
On 2013-01-08 9:37 AM, Anton Khirnov wrote: --- libavcodec/cllc.c | 22 +++--- 1 file changed, 3 insertions(+), 19 deletions(-) OK. - Derek ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 017/124] zerocodec: convert to refcounted frames.

2013-02-07 Thread Derek Buitenhuis
On 2013-01-08 9:36 AM, Anton Khirnov wrote: --- libavcodec/zerocodec.c | 34 ++ 1 file changed, 6 insertions(+), 28 deletions(-) If I am reading/understanding this correctly, then it seems OK. - Derek ___

Re: [libav-devel] [PATCH] h264 sps cropping (right and bottom)

2013-02-07 Thread Vittorio Giovara
Yes, I ran 'make fate' on the FATE suite and no errors reported. Vittorio On Thu, Feb 7, 2013 at 4:59 PM, Luca Barbato lu_z...@gentoo.org wrote: On 07/02/13 16:08, Vittorio Giovara wrote: Thanks for the tip, I think i got it working. I cannot modify the values ff_get_buffer() as I go out of

[libav-devel] Build testing

2013-02-07 Thread Luca Barbato
Here an initial draft about cross compilers and build testing. I have a good number of compilers installed leveraging crossdev[1]. It works decently if you have Gentoo or Sabayon or you installed a Gentoo-Prefix or just Gentoo on a chroot. For those not having any of the above I'd suggest to

[libav-devel] dsputil.h slicing

2013-02-07 Thread Diego Biurrun
This patchset splits suitable pieces off of dsputil.h and moves them off to more suitable places. Some cruft is also removed. There is more that could be done, this just represent low-hanging fruit that I could pick quickly, but it already cuts down the size of dsputil.h by one third and

[libav-devel] [PATCH 07/14] dsputil: Move ff_h264_idct function declarations to a separate header

2013-02-07 Thread Diego Biurrun
--- libavcodec/dsputil.h | 18 -- libavcodec/h264dsp.c |1 + libavcodec/h264idct.c |2 ++ libavcodec/h264idct.h | 42 ++ 4 files changed, 45 insertions(+), 18 deletions(-) create mode 100644 libavcodec/h264idct.h diff --git

[libav-devel] [PATCH 11/14] dsputil: Remove commented-out, unused function declarations

2013-02-07 Thread Diego Biurrun
--- libavcodec/dsputil.h |8 1 file changed, 8 deletions(-) diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h index 8bfc74b..1f67733 100644 --- a/libavcodec/dsputil.h +++ b/libavcodec/dsputil.h @@ -86,14 +86,6 @@ could be reached easily ... !future video codecs might need

[libav-devel] [PATCH 01/14] dsputil: Move ff_block_permute to mpegvideo_enc

2013-02-07 Thread Diego Biurrun
--- libavcodec/dsputil.c | 29 - libavcodec/dsputil.h |6 -- libavcodec/mpegvideo.c | 29 + libavcodec/mpegvideo.h |6 ++ 4 files changed, 35 insertions(+), 35 deletions(-) diff --git a/libavcodec/dsputil.c

[libav-devel] [PATCH 09/14] dsputil: Move ff_shrink* function declarations to separate header

2013-02-07 Thread Diego Biurrun
--- libavcodec/dsputil.c|1 + libavcodec/dsputil.h|5 - libavcodec/imgconvert.c |1 + libavcodec/imgconvert.h | 29 + 4 files changed, 31 insertions(+), 5 deletions(-) create mode 100644 libavcodec/imgconvert.h diff --git

[libav-devel] [PATCH 12/14] dsputil: Move rnd_avg inline functions to a separate header

2013-02-07 Thread Diego Biurrun
--- libavcodec/bit_depth_template.c |1 + libavcodec/dsputil.h| 23 --- libavcodec/rnd_avg.h| 47 +++ libavcodec/sh4/dsputil_align.c |1 + libavcodec/sh4/qpel.c |1 + libavcodec/vp3dsp.c

[libav-devel] [PATCH 04/14] indeo3: replace use of copy_block4 with put_pixels

2013-02-07 Thread Diego Biurrun
From: Mans Rullgard m...@mansr.com The destination is sufficiently aligned for put_pixels here. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/indeo3.c | 21 - 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/libavcodec/indeo3.c

[libav-devel] [PATCH 08/14] dsputil: Move ff_svq3 function declarations to a separate header

2013-02-07 Thread Diego Biurrun
--- libavcodec/dsputil.h |3 --- libavcodec/h264.c|1 + libavcodec/svq3.c|1 + libavcodec/svq3.h| 27 +++ 4 files changed, 29 insertions(+), 3 deletions(-) create mode 100644 libavcodec/svq3.h diff --git a/libavcodec/dsputil.h

[libav-devel] [PATCH 13/14] dsputil: Move WRAPPER8_16_SQ macro to the only place it is used

2013-02-07 Thread Diego Biurrun
--- libavcodec/dsputil.c | 14 ++ libavcodec/dsputil.h | 14 -- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/libavcodec/dsputil.c b/libavcodec/dsputil.c index a48a371..dea7029 100644 --- a/libavcodec/dsputil.c +++ b/libavcodec/dsputil.c @@ -2316,6

[libav-devel] [PATCH 02/14] dsputil: Move get_penalty_factor() to the only place it is used.

2013-02-07 Thread Diego Biurrun
--- libavcodec/dsputil.h| 20 libavcodec/motion_est.c | 20 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h index 8898ece..675fe29 100644 --- a/libavcodec/dsputil.h +++

[libav-devel] [PATCH 10/14] dsputil: Move fdct function declarations to a separate header

2013-02-07 Thread Diego Biurrun
--- libavcodec/dct-test.c |2 +- libavcodec/dsputil.c|1 + libavcodec/dsputil.h| 14 -- libavcodec/fdct.h | 37 + libavcodec/jfdctfst.c |2 +- libavcodec/jfdctint_template.c

[libav-devel] [PATCH 06/14] dsputil: Move copy_block functions to a separate header

2013-02-07 Thread Diego Biurrun
--- libavcodec/copy_block.h | 61 +++ libavcodec/dsputil.c|1 + libavcodec/dsputil.h| 36 libavcodec/sh4/qpel.c |1 + 4 files changed, 63 insertions(+), 36 deletions(-) create mode 100644

[libav-devel] [PATCH 03/14] mjpegdec: use put_pixels instead of copy_block8

2013-02-07 Thread Diego Biurrun
From: Mans Rullgard m...@mansr.com Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/mjpegdec.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index d13c85e..e63287a 100644 --- a/libavcodec/mjpegdec.c +++

[libav-devel] [PATCH 14/14] dsputil: Move LOCAL_ALIGNED macros to a more suitable place

2013-02-07 Thread Diego Biurrun
--- libavcodec/aacps.c|2 +- libavcodec/aacsbr.c |1 + libavcodec/ac3enc.c |2 +- libavcodec/ac3enc_template.c |1 + libavcodec/bink.c |1 + libavcodec/dcadec.c |2 +-

[libav-devel] [PATCH 05/14] dsputil: Drop unused functions copy_block{2|4|16}

2013-02-07 Thread Diego Biurrun
--- libavcodec/dsputil.h | 33 - 1 file changed, 33 deletions(-) diff --git a/libavcodec/dsputil.h b/libavcodec/dsputil.h index 675fe29..d42518c 100644 --- a/libavcodec/dsputil.h +++ b/libavcodec/dsputil.h @@ -515,28 +515,6 @@ static int name16(void

Re: [libav-devel] [PATCH v2] avplay: apply the stream sample_aspect_ratio to decoded video frames

2013-02-07 Thread Anton Khirnov
On Thu, 7 Feb 2013 18:42:39 +0100, Vladimir Pantelic vlado...@gmail.com wrote: If there is a sample_aspect_ratio in the stream, then apply it to every decoded frame in the same way as avconv does. This also makes sure that the avfilter chain has access to the aspect ratio. Signed-off-by: