Re: [libav-devel] [PATCH 2/5] lavc: schedule unused FF_QSCALE_TYPE_* for removal on next bump

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 8:56 AM, Anton Khirnov an...@khirnov.net wrote: --- libavcodec/avcodec.h |2 ++ libavcodec/version.h |3 +++ 2 files changed, 5 insertions(+) OK Vittorio ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 4/5] lavc: deprecate FF_MAX_B_FRAMES

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 8:56 AM, Anton Khirnov an...@khirnov.net wrote: We should not arbitrarily decide the maximum B-frame number for all encoders supported by Libav, each encoder should be able to set its own limits. --- libavcodec/avcodec.h |5 + libavcodec/mpegvideo.h

Re: [libav-devel] [PATCH 3/5] lavc: move MB_TYPE_* macros from avcodec.h to mpegvideo.h

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 8:56 AM, Anton Khirnov an...@khirnov.net wrote: They are mpegvideo-specific and not supposed to be used by callers. --- libavcodec/avcodec.h |2 ++ libavcodec/mpegvideo.h | 24 libavcodec/version.h |3 +++ 3 files changed, 29

Re: [libav-devel] [PATCH] lavc: Add colorimetry values for BT.2020, other non-included ones

2013-11-04 Thread Vittorio Giovara
On Sun, Nov 3, 2013 at 7:32 PM, Jan Ekström jee...@gmail.com wrote: --- doc/APIchanges |4 libavcodec/avcodec.h | 42 +++--- libavcodec/version.h |2 +- 3 files changed, 32 insertions(+), 16 deletions(-) I think it looks OK. Vittorio

Re: [libav-devel] [PATCH] hevc_ps: Use AV_PIX_FMT_YUVJ420P if YUV 4:2:0 8-bit full scale.

2013-11-04 Thread Vittorio Giovara
On Sun, Nov 3, 2013 at 8:47 PM, Yusuke Nakamura muken.the.vfrman...@gmail.com wrote: --- libavcodec/hevc_ps.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c index 15f73be..a51784d 100644 --- a/libavcodec/hevc_ps.c +++

[libav-devel] [PATCH] Deprecate XvMC decoding.

2013-11-04 Thread Anton Khirnov
It has long ago been superseded by newer acceleration APIs, such as VDPAU. Most of our major downstreams do not use it. It is not implemented within the hwaccel framework, but requires its own specific code in the decoders, which is a maintenance burden. --- doc/APIchanges|3

Re: [libav-devel] [PATCH] Deprecate XvMC decoding.

2013-11-04 Thread Rémi Denis-Courmont
On Mon, 4 Nov 2013 11:58:43 +0100, Anton Khirnov an...@khirnov.net wrote: It has long ago been superseded by newer acceleration APIs, such as VDPAU. Most of our major downstreams do not use it. Well, XvMC is only supported by one or two rather old generations of graphic cards, and even then it

Re: [libav-devel] [PATCH] Deprecate XvMC decoding.

2013-11-04 Thread Kostya Shishkov
On 4 November 2013 11:58, Anton Khirnov an...@khirnov.net wrote: It has long ago been superseded by newer acceleration APIs, such as VDPAU. Most of our major downstreams do not use it. It is not implemented within the hwaccel framework, but requires its own specific code in the decoders, which

Re: [libav-devel] [PATCH 4/6] vdpau: add a constructor for AVVDPAUContext.

2013-11-04 Thread Rémi Denis-Courmont
On Sun, 3 Nov 2013 23:27:48 +0100, Anton Khirnov an...@khirnov.net wrote: We will likely want to add new fields to it in the future, so this is needed to avoid breaking ABI. Should this not take a pix_fmt parameter and support all hwaccels? Or should there be one of these for each hwaccel? Or

[libav-devel] [PATCH v2] lavc: Add colorimetry values for BT.2020, other non-included ones

2013-11-04 Thread Jan Ekström
--- doc/APIchanges |4 libavcodec/avcodec.h | 42 +++--- libavcodec/version.h |2 +- 3 files changed, 32 insertions(+), 16 deletions(-) diff --git a/doc/APIchanges b/doc/APIchanges index 561c828..519709e 100644 --- a/doc/APIchanges +++

Re: [libav-devel] [PATCH 2/2] h264_ps: Set AVCHROMA_LOC_UNSPECIFIED if chroma location info is absent.

2013-11-04 Thread Vittorio Giovara
On Sun, Nov 3, 2013 at 8:15 PM, Yusuke Nakamura muken.the.vfrman...@gmail.com wrote: --- libavcodec/h264_ps.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) LGTM Vittorio ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH v2] lavc: Add colorimetry values for BT.2020, other non-included ones

2013-11-04 Thread Jan Ekstrom
On Mon, Nov 4, 2013 at 1:28 PM, Jan Ekström jee...@gmail.com wrote: --- doc/APIchanges |4 libavcodec/avcodec.h | 42 +++--- libavcodec/version.h |2 +- 3 files changed, 32 insertions(+), 16 deletions(-) The only difference to v1 is

Re: [libav-devel] [PATCH 2/2] h264_ps: Set AVCHROMA_LOC_UNSPECIFIED if chroma location info is absent.

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 12:47 PM, Vittorio Giovara vittorio.giov...@gmail.com wrote: On Sun, Nov 3, 2013 at 8:15 PM, Yusuke Nakamura muken.the.vfrman...@gmail.com wrote: --- libavcodec/h264_ps.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) LGTM Vittorio Sorry I was too hasty,

Re: [libav-devel] [PATCH 3/6] vdpau: add a convenience function for getting a decoder profile.

2013-11-04 Thread Rémi Denis-Courmont
On Sun, 3 Nov 2013 23:27:47 +0100, Anton Khirnov an...@khirnov.net wrote: Based on the code by Rémi Denis-Courmont r...@remlab.net from VLC. --- doc/APIchanges |3 +++ libavcodec/vdpau.c | 37 + libavcodec/vdpau.h | 10 ++

Re: [libav-devel] [PATCH 4/6] vdpau: add a constructor for AVVDPAUContext.

2013-11-04 Thread Rémi Denis-Courmont
On Mon, 4 Nov 2013 01:03:22 +0100, wm4 nfx...@googlemail.com wrote: Completely arbitrary API changes out of the blue? How nice. At this point, it remains backward compatible. In any case, the VDPAU hwaccel has not been in any libav release, hasn't it? So I don't mind breaking the ABI as yet. In

Re: [libav-devel] [PATCH 6/6] avconv: add support for VDPAU decoding

2013-11-04 Thread Rémi Denis-Courmont
On Sun, 3 Nov 2013 23:27:50 +0100, Anton Khirnov an...@khirnov.net wrote: --- Changelog |1 + Makefile|5 +- avconv.c|3 + avconv.h|5 + avconv_opt.c|2 + avconv_vdpau.c | 339 +++

Re: [libav-devel] [PATCH 1/6] lavc: use buf[0] instead of data[0] in checks whether a frame is allocated

2013-11-04 Thread Rémi Denis-Courmont
On Sun, 3 Nov 2013 23:27:45 +0100, Anton Khirnov an...@khirnov.net wrote: data[0] may be NULL for valid frames with hwaccel pixel formats. This looks OK (and VLC already supplies a dummy buf[0] for hwaccel surfaces) but I would not expect downstreams to adapt anytime soon, for the sake of

[libav-devel] [PATCH 1/2] h264_ps: use specifications' name for chroma location variables

2013-11-04 Thread Vittorio Giovara
--- libavcodec/h264_ps.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c index 54b735d..819b7ff 100644 --- a/libavcodec/h264_ps.c +++ b/libavcodec/h264_ps.c @@ -131,6 +131,9 @@ static inline int

[libav-devel] [PATCH 2/2] h264_ps: check validity of parsed chroma location values

2013-11-04 Thread Vittorio Giovara
--- libavcodec/h264_ps.c |5 + 1 file changed, 5 insertions(+) diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c index 819b7ff..95e0b9a 100644 --- a/libavcodec/h264_ps.c +++ b/libavcodec/h264_ps.c @@ -181,6 +181,11 @@ static inline int decode_vui_parameters(H264Context *h, SPS

Re: [libav-devel] [PATCH 1/6] lavc: use buf[0] instead of data[0] in checks whether a frame is allocated

2013-11-04 Thread Hendrik Leppkes
On Mon, Nov 4, 2013 at 1:05 PM, Rémi Denis-Courmont r...@remlab.net wrote: On Sun, 3 Nov 2013 23:27:45 +0100, Anton Khirnov an...@khirnov.net wrote: data[0] may be NULL for valid frames with hwaccel pixel formats. This looks OK (and VLC already supplies a dummy buf[0] for hwaccel surfaces)

[libav-devel] [PATCH] avfilter/buffer: Suppress deprecation warning for avfilter_unref_buffer

2013-11-04 Thread Diego Biurrun
The function is used in a block of code that is deprecated anyway. --- libavfilter/buffer.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/libavfilter/buffer.c b/libavfilter/buffer.c index 88b5f22..fd0b18f 100644 --- a/libavfilter/buffer.c +++ b/libavfilter/buffer.c @@ -18,6 +18,7 @@

[libav-devel] [PATCH 1/2] hevc: Set chroma_sample_location of AVCodecContext.

2013-11-04 Thread Yusuke Nakamura
--- libavcodec/hevc.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index ed980e3..e3d306b 100644 --- a/libavcodec/hevc.c +++ b/libavcodec/hevc.c @@ -401,6 +401,13 @@ static int set_sps(HEVCContext *s, const HEVCSPS *sps)

[libav-devel] [PATCH 2/2] h264_ps: Set chroma_sample_location appropriately if chroma location info is absent.

2013-11-04 Thread Yusuke Nakamura
--- libavcodec/h264_ps.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c index 54b735d..422f522 100644 --- a/libavcodec/h264_ps.c +++ b/libavcodec/h264_ps.c @@ -178,7 +178,10 @@ static inline int decode_vui_parameters(H264Context

Re: [libav-devel] [PATCH 1/6] lavc: use buf[0] instead of data[0] in checks whether a frame is allocated

2013-11-04 Thread Rémi Denis-Courmont
On Mon, 4 Nov 2013 13:12:36 +0100, Hendrik Leppkes h.lepp...@gmail.com wrote: If downstream uses get_buffer2, they absolutely need to provide a buf[0], and if downstream uses get_buffer, the compat layer should provide a buf[0], IIRC. Or am I missing something? I don't expect downstreams to

Re: [libav-devel] [PATCH] avfilter/buffer: Suppress deprecation warning for avfilter_unref_buffer

2013-11-04 Thread Luca Barbato
On 04/11/13 13:17, Diego Biurrun wrote: The function is used in a block of code that is deprecated anyway. --- libavfilter/buffer.c |3 +++ 1 file changed, 3 insertions(+) Ok. ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 1/2] hevc: Set chroma_sample_location of AVCodecContext.

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 1:26 PM, Yusuke Nakamura muken.the.vfrman...@gmail.com wrote: --- libavcodec/hevc.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavcodec/hevc.c b/libavcodec/hevc.c index ed980e3..e3d306b 100644 --- a/libavcodec/hevc.c +++ b/libavcodec/hevc.c @@

Re: [libav-devel] [PATCH 2/2] h264_ps: Set chroma_sample_location appropriately if chroma location info is absent.

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 2:27 PM, Vittorio Giovara vittorio.giov...@gmail.com wrote: On Mon, Nov 4, 2013 at 1:28 PM, Yusuke Nakamura muken.the.vfrman...@gmail.com wrote: --- libavcodec/h264_ps.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/libavcodec/h264_ps.c

[libav-devel] [PATCH] configure: Check whether MSVC supports C99

2013-11-04 Thread Martin Storsjö
MSVC does support most of C99 since the 2013 version. Try to detect whether the compiler in the path needs to run the C99 converter or not. When the converter is omitted, the compilation time is reduced quite drastically. Prior to this, users could still use --cc=c99conv -noconv cl to run MSVC

Re: [libav-devel] [PATCH] Remove all Alpha architecture optimizations

2013-11-04 Thread Luca Barbato
On 04/11/13 14:53, Diego Biurrun wrote: Alpha has been end-of-lifed and no more test machines are available. --- ... as discussed at VDD in Paris ... Ok, I guess, I doubt we'll see many alpha systems still functional nowadays. lu ___ libav-devel

Re: [libav-devel] [PATCH] Remove all Alpha architecture optimizations

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 14:53:10 +0100, Diego Biurrun di...@biurrun.de wrote: Alpha has been end-of-lifed and no more test machines are available. --- ... as discussed at VDD in Paris ... libavcodec/alpha/Makefile |9 - libavcodec/alpha/asm.h| 186

Re: [libav-devel] [PATCH] configure: Check whether MSVC supports C99

2013-11-04 Thread Luca Barbato
On 04/11/13 15:00, Martin Storsjö wrote: MSVC does support most of C99 since the 2013 version. Try to detect whether the compiler in the path needs to run the C99 converter or not. When the converter is omitted, the compilation time is reduced quite drastically. Ok.

Re: [libav-devel] [PATCH] Remove all Alpha architecture optimizations

2013-11-04 Thread Diego Biurrun
On Mon, Nov 04, 2013 at 03:01:56PM +0100, Anton Khirnov wrote: On Mon, 4 Nov 2013 14:53:10 +0100, Diego Biurrun di...@biurrun.de wrote: Alpha has been end-of-lifed and no more test machines are available. --- ... as discussed at VDD in Paris ... --- a/libavcodec/avcodec.h +++

Re: [libav-devel] [PATCH] Remove all Alpha architecture optimizations

2013-11-04 Thread Derek Buitenhuis
On 11/4/2013 2:20 PM, Diego Biurrun wrote: I'm removing the IDCT, so what use is the option? I'd have to schedule everything for removal on the next bump then. That never did anything on nonalpha anyway. Just leave the IDCT thing in there, and remove it later with the rest of the IDCT stuff.

Re: [libav-devel] [PATCH v2] lavc: Add colorimetry values for BT.2020, other non-included ones

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 12:28 PM, Jan Ekström jee...@gmail.com wrote: --- doc/APIchanges |4 libavcodec/avcodec.h | 42 +++--- libavcodec/version.h |2 +- 3 files changed, 32 insertions(+), 16 deletions(-) Queued, thank you. Vittorio

[libav-devel] [PATCH] filtfmts-test: Fix use of deprecated API

2013-11-04 Thread Diego Biurrun
--- libavfilter/filtfmts.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavfilter/filtfmts.c b/libavfilter/filtfmts.c index 69288af..f68287a 100644 --- a/libavfilter/filtfmts.c +++ b/libavfilter/filtfmts.c @@ -71,12 +71,12 @@ int main(int argc, char **argv)

Re: [libav-devel] [PATCH] filtfmts-test: Fix use of deprecated API

2013-11-04 Thread Luca Barbato
On 04/11/13 15:56, Diego Biurrun wrote: --- libavfilter/filtfmts.c |8 1 file changed, 4 insertions(+), 4 deletions(-) Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] configure: Check whether MSVC supports C99

2013-11-04 Thread Derek Buitenhuis
On 11/4/2013 2:00 PM, Martin Storsjö wrote: MSVC does support most of C99 since the 2013 version. Try to detect whether the compiler in the path needs to run the C99 converter or not. When the converter is omitted, the compilation time is reduced quite drastically. Prior to this, users

Re: [libav-devel] [PATCH] hevc: Use parsed VUI colorimetry in avcodec

2013-11-04 Thread Vittorio Giovara
On Sun, Nov 3, 2013 at 9:45 PM, Jan Ekström jee...@gmail.com wrote: Also limit the parsed VUI values to known valid ones. --- libavcodec/hevc.c| 16 libavcodec/hevc_ps.c |8 2 files changed, 24 insertions(+), 0 deletions(-) LGTM, queued. Vittorio

Re: [libav-devel] [PATCH] filtfmts-test: Fix use of deprecated API

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 3:56 PM, Diego Biurrun di...@biurrun.de wrote: --- libavfilter/filtfmts.c |8 1 file changed, 4 insertions(+), 4 deletions(-) Queued, thank you. Vittorio ___ libav-devel mailing list libav-devel@libav.org

[libav-devel] [PATCH] png: add a standalone parser

2013-11-04 Thread Vittorio Giovara
From: Peter Holik pe...@holik.at --- Addressed Kostya's comments and added some minor comments. Vittorio libavcodec/Makefile |1 + libavcodec/allcodecs.c |1 + libavcodec/png_parser.c | 124 +++ 3 files changed, 126 insertions(+) create

Re: [libav-devel] [PATCH] hevc_ps: Use AV_PIX_FMT_YUVJ420P if YUV 4:2:0 8-bit full scale.

2013-11-04 Thread Vittorio Giovara
On Mon, Nov 4, 2013 at 11:55 AM, Vittorio Giovara vittorio.giov...@gmail.com wrote: On Sun, Nov 3, 2013 at 8:47 PM, Yusuke Nakamura muken.the.vfrman...@gmail.com wrote: --- libavcodec/hevc_ps.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c

Re: [libav-devel] [PATCH] lavc: Add a VP9 decoder

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 17:27:36 +0100, Luca Barbato lu_z...@gentoo.org wrote: From: Ronald S. Bultje rsbul...@gmail.com Originally written by Ronald S. Bultje rsbul...@gmail.com with the help of Clément Bœsch ubi...@gmail.com. Further contributions by: Anton Khirnov an...@khirnov.net Luca

[libav-devel] [PATCH] graph2dot: Fix use of deprecated API

2013-11-04 Thread Diego Biurrun
--- tools/graph2dot.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/graph2dot.c b/tools/graph2dot.c index 692ada2..86d3d14 100644 --- a/tools/graph2dot.c +++ b/tools/graph2dot.c @@ -58,7 +58,7 @@ static void print_digraph(FILE *outfile, AVFilterGraph *graph)

Re: [libav-devel] [PATCH] graph2dot: Fix use of deprecated API

2013-11-04 Thread Luca Barbato
On 04/11/13 17:36, Diego Biurrun wrote: --- tools/graph2dot.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) Ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH] lavc: Add a VP9 decoder

2013-11-04 Thread Luca Barbato
On 04/11/13 17:34, Anton Khirnov wrote: Other than the missing entry in the decoder list i think it's ok. Forgot to amend for it. Pushing soon. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

[libav-devel] [PATCH 1/3] mxfdec: update current edit unit when seeking in mxf files without index tables

2013-11-04 Thread Anton Khirnov
From: Marton Balint c...@passwd.hu Fixes audio packet pts values in some files generated by AVID TRMG 3.01. Signed-off-by: Marton Balint c...@passwd.hu Reviewed-by: Tomas Härdin tomas.har...@codemill.se Signed-off-by: Anton Khirnov an...@khirnov.net --- libavformat/mxfdec.c |1 + 1 file

[libav-devel] [PATCH 2/3] lavf: improve support for AVC-Intra files.

2013-11-04 Thread Anton Khirnov
From: Reimar Döffinger reimar.doeffin...@gmx.de Generate extradata with SPS/PPS based on container dimensions. Authors of this commit are: Reimar and Thomas Mundt Signed-off-by: Anton Khirnov an...@khirnov.net --- libavformat/internal.h |6 +++ libavformat/isom.c |1 +

[libav-devel] [PATCH 3/3] h264: free the tables and uninitialize the context on flush

2013-11-04 Thread Anton Khirnov
Prevents referencing empty frames. --- libavcodec/h264.c |3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/h264.c b/libavcodec/h264.c index f0ef2eb..5d8406f 100644 --- a/libavcodec/h264.c +++ b/libavcodec/h264.c @@ -2768,6 +2768,9 @@ static void flush_dpb(AVCodecContext *avctx)

[libav-devel] [PATCH] vdpau: Add missing #includes to fix standalone header compilation

2013-11-04 Thread Diego Biurrun
--- libavcodec/vdpau.h |2 ++ libavcodec/vdpau_internal.h |5 - 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/vdpau.h b/libavcodec/vdpau.h index 8bedea5..bf8f666 100644 --- a/libavcodec/vdpau.h +++ b/libavcodec/vdpau.h @@ -52,6 +52,8 @@ #include

Re: [libav-devel] Small OMA fixes

2013-11-04 Thread Anton Khirnov
On Thu, 24 Oct 2013 15:24:23 +0200, David Goldwich david.goldw...@gmail.com wrote: First two patches are security fixes that landed in FFmpeg a while ago (4f5d146, e74fa25, 91e72e3; first one was partially addressed in libav in 934cd18). Third patch enables format detection for files with

Re: [libav-devel] [PATCH 1/6] lavc: use buf[0] instead of data[0] in checks whether a frame is allocated

2013-11-04 Thread Anton Khirnov
On Mon, 04 Nov 2013 13:37:55 +0100, Rémi Denis-Courmont r...@remlab.net wrote: On Mon, 4 Nov 2013 13:12:36 +0100, Hendrik Leppkes h.lepp...@gmail.com wrote: If downstream uses get_buffer2, they absolutely need to provide a buf[0], and if downstream uses get_buffer, the compat layer should

Re: [libav-devel] [PATCH 3/6] vdpau: add a convenience function for getting a decoder profile.

2013-11-04 Thread Anton Khirnov
On Mon, 04 Nov 2013 12:54:08 +0100, Rémi Denis-Courmont r...@remlab.net wrote: On Sun, 3 Nov 2013 23:27:47 +0100, Anton Khirnov an...@khirnov.net wrote: Based on the code by Rémi Denis-Courmont r...@remlab.net from VLC. --- doc/APIchanges |3 +++ libavcodec/vdpau.c | 37

Re: [libav-devel] [PATCH 4/6] vdpau: add a constructor for AVVDPAUContext.

2013-11-04 Thread Anton Khirnov
On Mon, 04 Nov 2013 12:08:06 +0100, Rémi Denis-Courmont r...@remlab.net wrote: On Sun, 3 Nov 2013 23:27:48 +0100, Anton Khirnov an...@khirnov.net wrote: We will likely want to add new fields to it in the future, so this is needed to avoid breaking ABI. Should this not take a pix_fmt

Re: [libav-devel] [PATCH 6/6] avconv: add support for VDPAU decoding

2013-11-04 Thread Anton Khirnov
On Mon, 04 Nov 2013 13:02:37 +0100, Rémi Denis-Courmont r...@remlab.net wrote: On Sun, 3 Nov 2013 23:27:50 +0100, Anton Khirnov an...@khirnov.net wrote: --- Changelog |1 + Makefile|5 +- avconv.c|3 + avconv.h|5 + avconv_opt.c|

Re: [libav-devel] [PATCH] vdpau: Add missing #includes to fix standalone header compilation

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 18:16:46 +0100, Diego Biurrun di...@biurrun.de wrote: --- libavcodec/vdpau.h |2 ++ libavcodec/vdpau_internal.h |5 - 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/vdpau.h b/libavcodec/vdpau.h index 8bedea5..bf8f666

Re: [libav-devel] [PATCH] vdpau: Add missing #includes to fix standalone header compilation

2013-11-04 Thread Diego Biurrun
On Mon, Nov 04, 2013 at 07:07:34PM +0100, Anton Khirnov wrote: On Mon, 4 Nov 2013 18:16:46 +0100, Diego Biurrun di...@biurrun.de wrote: --- a/libavcodec/vdpau_internal.h +++ b/libavcodec/vdpau_internal.h @@ -26,8 +26,11 @@ #include stdint.h #include vdpau/vdpau.h -#include

[libav-devel] [PATCH] aacps: Adjust some const qualifiers to suppress warnings

2013-11-04 Thread Diego Biurrun
--- While clearly not a thing of supreme beauty, this silences all warnings on a previously very noisy file, both with hardcoded and generated tables. libavcodec/aacps.c | 20 libavcodec/aacps_tablegen.h |2 ++ libavcodec/aacpsdsp.h |2 +- 3 files

Re: [libav-devel] [PATCH] aacps: Adjust some const qualifiers to suppress warnings

2013-11-04 Thread Luca Barbato
On 04/11/13 19:14, Diego Biurrun wrote: --- While clearly not a thing of supreme beauty, this silences all warnings on a previously very noisy file, both with hardcoded and generated tables. It is compact enough all in all. lu ___ libav-devel

[libav-devel] [PATCH 1/2] dsputil: Split off H.263 bits into their own H263DSPContext

2013-11-04 Thread Diego Biurrun
--- configure |5 +- libavcodec/Makefile |1 + libavcodec/dsputil.c | 79 -- libavcodec/dsputil.h |3 -- libavcodec/h263.c | 33 +++-- libavcodec/h263data.h |5 --

[libav-devel] [PATCH 2/2] h263dsp: KR formatting cosmetics

2013-11-04 Thread Diego Biurrun
--- libavcodec/h263dsp.c | 110 +- 1 file changed, 63 insertions(+), 47 deletions(-) diff --git a/libavcodec/h263dsp.c b/libavcodec/h263dsp.c index 1166b93..63d0972 100644 --- a/libavcodec/h263dsp.c +++ b/libavcodec/h263dsp.c @@ -23,78 +23,94 @@

Re: [libav-devel] [PATCH] configure: Check whether MSVC supports C99

2013-11-04 Thread Diego Biurrun
On Mon, Nov 04, 2013 at 04:00:48PM +0200, Martin Storsjö wrote: MSVC does support most of C99 since the 2013 version. Try to detect whether the compiler in the path needs to run the C99 converter or not. When the converter is omitted, the compilation s/the compilation/compilation/ Prior to

Re: [libav-devel] [PATCH] hevc_ps: Use AV_PIX_FMT_YUVJ420P if YUV 4:2:0 8-bit full scale.

2013-11-04 Thread wm4
On Mon, 4 Nov 2013 11:55:02 +0100 Vittorio Giovara vittorio.giov...@gmail.com wrote: On Sun, Nov 3, 2013 at 8:47 PM, Yusuke Nakamura muken.the.vfrman...@gmail.com wrote: --- libavcodec/hevc_ps.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/hevc_ps.c

Re: [libav-devel] [PATCH 4/6] vdpau: add a constructor for AVVDPAUContext.

2013-11-04 Thread wm4
On Sun, 3 Nov 2013 23:27:48 +0100 Anton Khirnov an...@khirnov.net wrote: We will likely want to add new fields to it in the future, so this is needed to avoid breaking ABI. --- doc/APIchanges |2 ++ libavcodec/vdpau.c |5 + libavcodec/vdpau.h | 11 +++ 3 files

Re: [libav-devel] [PATCH] configure: Check whether MSVC supports C99

2013-11-04 Thread Martin Storsjö
On Mon, 4 Nov 2013, Diego Biurrun wrote: On Mon, Nov 04, 2013 at 04:00:48PM +0200, Martin Storsjö wrote: MSVC does support most of C99 since the 2013 version. Try to detect whether the compiler in the path needs to run the C99 converter or not. When the converter is omitted, the compilation

[libav-devel] [PATCH] mpegaudioenc: Move some static tables to the only place they are used

2013-11-04 Thread Diego Biurrun
--- libavcodec/mpegaudioenc.c | 14 ++ libavcodec/mpegaudiotab.h | 14 -- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/libavcodec/mpegaudioenc.c b/libavcodec/mpegaudioenc.c index 739eb64..7e0fda5 100644 --- a/libavcodec/mpegaudioenc.c +++

[libav-devel] [PATCH] configure: Move -Wno-maybe-uninitialized check into gcc section

2013-11-04 Thread Diego Biurrun
As another example of bizarre compiler behavior clang groks the -Wmaybe-uninitialized option, but not -Wno-maybe-uninitialized and spews a warning for every file that gets compiled. --- This is very similar to what Vittorio sent, but I had created this patch a few days ago already, just got

Re: [libav-devel] [PATCH 4/6] vdpau: add a constructor for AVVDPAUContext.

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 21:02:02 +0100, wm4 nfx...@googlemail.com wrote: On Sun, 3 Nov 2013 23:27:48 +0100 Anton Khirnov an...@khirnov.net wrote: We will likely want to add new fields to it in the future, so this is needed to avoid breaking ABI. --- doc/APIchanges |2 ++

Re: [libav-devel] [PATCH 4/6] vdpau: add a constructor for AVVDPAUContext.

2013-11-04 Thread wm4
On Mon, 04 Nov 2013 21:35:45 +0100 Anton Khirnov an...@khirnov.net wrote: On Mon, 4 Nov 2013 21:02:02 +0100, wm4 nfx...@googlemail.com wrote: On Sun, 3 Nov 2013 23:27:48 +0100 Anton Khirnov an...@khirnov.net wrote: We will likely want to add new fields to it in the future, so this

Re: [libav-devel] [PATCH] lavc: Add a VP9 decoder

2013-11-04 Thread Ronald S. Bultje
Hi, On Mon, Nov 4, 2013 at 11:27 AM, Luca Barbato lu_z...@gentoo.org wrote: From: Ronald S. Bultje rsbul...@gmail.com Originally written by Ronald S. Bultje rsbul...@gmail.com with the help of Clément Bœsch ubi...@gmail.com. No. The decoder was written by A and B, not A with the help of B.

Re: [libav-devel] [PATCH 4/6] vdpau: add a constructor for AVVDPAUContext.

2013-11-04 Thread Luca Barbato
On 04/11/13 21:52, wm4 wrote: I don't think having public fields in structs ever worked out well in libav*. It's an endless cycle of deprecations, ABI hacks, and working around bad ABI properties of structs (like adding a constructor function). That's why we are trying to seal when makes

Re: [libav-devel] [PATCH] mpegaudioenc: Move some static tables to the only place they are used

2013-11-04 Thread Luca Barbato
On 04/11/13 21:24, Diego Biurrun wrote: --- libavcodec/mpegaudioenc.c | 14 ++ libavcodec/mpegaudiotab.h | 14 -- 2 files changed, 14 insertions(+), 14 deletions(-) Probably ok. ___ libav-devel mailing list

Re: [libav-devel] [PATCH] vdpau: Add missing #includes to fix standalone header compilation

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 19:09:38 +0100, Diego Biurrun di...@biurrun.de wrote: On Mon, Nov 04, 2013 at 07:07:34PM +0100, Anton Khirnov wrote: On Mon, 4 Nov 2013 18:16:46 +0100, Diego Biurrun di...@biurrun.de wrote: --- a/libavcodec/vdpau_internal.h +++ b/libavcodec/vdpau_internal.h @@

[libav-devel] [PATCH] graph2dot: Add missing #include for av_get_channel_layout_string()

2013-11-04 Thread Diego Biurrun
--- tools/graph2dot.c |1 + 1 file changed, 1 insertion(+) diff --git a/tools/graph2dot.c b/tools/graph2dot.c index 86d3d14..12f1066 100644 --- a/tools/graph2dot.c +++ b/tools/graph2dot.c @@ -25,6 +25,7 @@ #include stdio.h #include string.h +#include libavutil/channel_layout.h #include

Re: [libav-devel] [PATCH] graph2dot: Add missing #include for av_get_channel_layout_string()

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 22:34:18 +0100, Diego Biurrun di...@biurrun.de wrote: --- tools/graph2dot.c |1 + 1 file changed, 1 insertion(+) ok -- Anton Khirnov ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH] vdpau: Add missing #includes to fix standalone header compilation

2013-11-04 Thread Diego Biurrun
On Mon, Nov 04, 2013 at 10:23:00PM +0100, Anton Khirnov wrote: On Mon, 4 Nov 2013 19:09:38 +0100, Diego Biurrun di...@biurrun.de wrote: On Mon, Nov 04, 2013 at 07:07:34PM +0100, Anton Khirnov wrote: On Mon, 4 Nov 2013 18:16:46 +0100, Diego Biurrun di...@biurrun.de wrote: ---

Re: [libav-devel] [PATCH] mpegaudioenc: Move some static tables to the only place they are used

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 21:24:11 +0100, Diego Biurrun di...@biurrun.de wrote: --- libavcodec/mpegaudioenc.c | 14 ++ libavcodec/mpegaudiotab.h | 14 -- 2 files changed, 14 insertions(+), 14 deletions(-) diff --git a/libavcodec/mpegaudioenc.c

Re: [libav-devel] [PATCH 4/6] vdpau: add a constructor for AVVDPAUContext.

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 21:52:17 +0100, wm4 nfx...@googlemail.com wrote: On Mon, 04 Nov 2013 21:35:45 +0100 Anton Khirnov an...@khirnov.net wrote: On Mon, 4 Nov 2013 21:02:02 +0100, wm4 nfx...@googlemail.com wrote: On Sun, 3 Nov 2013 23:27:48 +0100 Anton Khirnov an...@khirnov.net

Re: [libav-devel] [PATCH] configure: clang filter for maybe-unintialized

2013-11-04 Thread Vittorio Giovara
On Sun, Nov 3, 2013 at 11:16 PM, Hendrik Leppkes h.lepp...@gmail.com wrote: On Sun, Nov 3, 2013 at 11:09 PM, Vittorio Giovara vittorio.giov...@gmail.com wrote: This will convert the gcc warning -Wno-maybe-unintialized into -Wno-sometimes-uninitialized for clang --- This was originally

Re: [libav-devel] [PATCH] aacps: Adjust some const qualifiers to suppress warnings

2013-11-04 Thread Alex Converse
On Mon, Nov 4, 2013 at 10:14 AM, Diego Biurrun di...@biurrun.de wrote: --- While clearly not a thing of supreme beauty, this silences all warnings on a previously very noisy file, both with hardcoded and generated tables. The only warning my compiler generates with this file is: warning:

[libav-devel] [PATCH] [v2] dsputil: Split off H.263 bits into their own H263DSPContext

2013-11-04 Thread Diego Biurrun
--- Adjusted the bits in configure so that mpegvideo now automatically selects h263dsp. Given that the h263dsp init is called unconditionally from mpegvideo.c, this is the correct solution. configure |3 +- libavcodec/Makefile |1 + libavcodec/dsputil.c

[libav-devel] [PATCH] Deprecate obsolete XvMC hardware decoding support

2013-11-04 Thread Diego Biurrun
XvMC has long ago been superseded by newer acceleration APIs, such as VDPAU, and few downstreams still support it. Furthermore XvMC is not implemented within the hwaccel framework, but requires its own specific code in the MPEG-1/2 decoder, which is a maintenance burden. --- For some funny reason

Re: [libav-devel] [PATCH] gitignore: Ignore .dirs

2013-11-04 Thread Anton Khirnov
On Sat, 02 Nov 2013 13:52:11 +0100, Luca Barbato lu_z...@gentoo.org wrote: On 02/11/13 10:13, Diego Biurrun wrote: On Sat, Nov 02, 2013 at 12:34:57AM +0100, Luca Barbato wrote: Quite useful to be able to git-clean while using .builddirs. --- .gitignore | 1 + 1 file changed, 1

Re: [libav-devel] [PATCH] Remove all Alpha architecture optimizations

2013-11-04 Thread Anton Khirnov
On Mon, 4 Nov 2013 15:58:43 +0100, Diego Biurrun di...@biurrun.de wrote: Alpha has been end-of-lifed and no more test machines are available. --- Now with the usual deprecation dance for the bits in avcodec.h and options_table.h... libavcodec/alpha/Makefile |9 -