[libav-devel] [PATCH] avpacket: make ff_interleave_add_packet return an error code

2014-07-07 Thread Nidhi Makhijani
--- is the commit tag and commit message correct? libavformat/audiointerleave.c | 8 +--- libavformat/internal.h| 2 +- libavformat/mux.c | 18 +- 3 files changed, 19 insertions(+), 9 deletions(-) diff --git a/libavformat/audiointerleave.c

[libav-devel] [PATCH] libfdk-aacdec: Decode the first AAC frame to reliably identify the bitstream

2014-07-07 Thread Omer Osman
For implicit signaling cases (as possible for Spectral Band Replication and Parametric Stereo Tools), the decoder must decode the first frame to correctly identify the stream configuration (as called from avformat_find_stream_info). The mechanism for this is built-in and only requires adding

[libav-devel] [PATCH v2] md5: check memory allocations

2014-07-07 Thread Nidhi Makhijani
--- added common fail for errors libavformat/md5proto.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/libavformat/md5proto.c b/libavformat/md5proto.c index 12ddde3..3dcfa12 100644 --- a/libavformat/md5proto.c +++ b/libavformat/md5proto.c @@ -59,7 +59,7 @@ static

Re: [libav-devel] [PATCH] libfdk-aacdec: Decode the first AAC frame to reliably identify the bitstream

2014-07-07 Thread Luca Barbato
On 07/07/14 10:28, Omer Osman wrote: For implicit signaling cases (as possible for Spectral Band Replication and Parametric Stereo Tools), the decoder must decode the first frame to correctly identify the stream configuration (as called from avformat_find_stream_info). The mechanism for this

Re: [libav-devel] [PATCH 1/2] mov: Do not group track if more than one is enabled per type

2014-07-07 Thread Luca Barbato
On 04/07/14 16:54, Luca Barbato wrote: The specification requires at most 1 track enable per alternate group. --- libavformat/movenc.c | 21 - libavformat/movenc.h | 2 ++ 2 files changed, 18 insertions(+), 5 deletions(-) diff --git a/libavformat/movenc.c

Re: [libav-devel] [PATCH] arm: Avoid functions using the deprecated 'setend' instruction on modern arches

2014-07-07 Thread Janne Grunau
On 2014-07-06 17:48:14 +0300, Martin Storsjö wrote: On Sun, 6 Jul 2014, Janne Grunau wrote: On 2014-07-04 18:27:54 +0300, Martin Storsjö wrote: [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2014-July/268504.html --- I don't think this is a good solution for the almost

Re: [libav-devel] [PATCH v4] vmd: split audio and video decoder

2014-07-07 Thread Diego Biurrun
On Mon, Jul 07, 2014 at 09:41:04AM +0530, Nidhi Makhijani wrote: --- removed unneccessary headers and corrected typos and copyrights libavcodec/Makefile| 4 +- libavcodec/vmdaudio.c | 233 + libavcodec/{vmdav.c =

Re: [libav-devel] [PATCH] libfdk-aacdec: Decode the first AAC frame to reliably identify the bitstream

2014-07-07 Thread Martin Storsjö
On Mon, 7 Jul 2014, Omer Osman wrote: For implicit signaling cases (as possible for Spectral Band Replication and Parametric Stereo Tools), the decoder must decode the first frame to correctly identify the stream configuration (as called from avformat_find_stream_info). The mechanism for this

Re: [libav-devel] [PATCH] dsputil: Split off FDCT bits into their own context

2014-07-07 Thread Diego Biurrun
On Thu, Jul 03, 2014 at 10:38:30AM -0700, Diego Biurrun wrote: --- Now w/o blackfin noise and with some PPC init fixes. configure | 13 +++--- libavcodec/Makefile| 3 +- libavcodec/asv.h

Re: [libav-devel] [PATCH 089/132] dsputil: Initialize static dsputil data from within dsputil itself

2014-07-07 Thread Diego Biurrun
On Tue, Jun 03, 2014 at 08:07:30PM +0200, Diego Biurrun wrote: On Wed, Mar 26, 2014 at 06:28:10PM +0100, Luca Barbato wrote: On 26/03/14 12:41, Diego Biurrun wrote: --- libavcodec/dsputil.c | 10 -- libavcodec/dsputil.h | 1 - libavcodec/utils.c | 13 - 3

Re: [libav-devel] [PATCH] arm: Avoid functions using the deprecated 'setend' instruction on modern arches

2014-07-07 Thread Martin Storsjö
On Mon, 7 Jul 2014, Janne Grunau wrote: On 2014-07-06 17:48:14 +0300, Martin Storsjö wrote: On Sun, 6 Jul 2014, Janne Grunau wrote: On 2014-07-04 18:27:54 +0300, Martin Storsjö wrote: [1] http://lists.infradead.org/pipermail/linux-arm-kernel/2014-July/268504.html --- I don't think this

Re: [libav-devel] [PATCH] dsputil: Split off FDCT bits into their own context

2014-07-07 Thread Luca Barbato
On 07/07/14 16:59, Diego Biurrun wrote: On Thu, Jul 03, 2014 at 10:38:30AM -0700, Diego Biurrun wrote: --- Now w/o blackfin noise and with some PPC init fixes. configure | 13 +++--- libavcodec/Makefile| 3 +-

Re: [libav-devel] [PATCH 089/132] dsputil: Initialize static dsputil data from within dsputil itself

2014-07-07 Thread Luca Barbato
On 07/07/14 17:07, Diego Biurrun wrote: On Tue, Jun 03, 2014 at 08:07:30PM +0200, Diego Biurrun wrote: On Wed, Mar 26, 2014 at 06:28:10PM +0100, Luca Barbato wrote: On 26/03/14 12:41, Diego Biurrun wrote: --- libavcodec/dsputil.c | 10 -- libavcodec/dsputil.h | 1 -

Re: [libav-devel] [libav-commits] nut: Support experimental NUT 4 features

2014-07-07 Thread Diego Biurrun
On 2014-05-28 23:30, Luca Barbato wrote: Module: libav Branch: master Commit: c94e2e85cb6af8a570d8542a830556243bd32873 Author:Luca Barbato lu_z...@gentoo.org Committer: Luca Barbato lu_z...@gentoo.org Date: Thu Mar 6 17:58:34 2014 +0100 nut: Support experimental NUT 4 features Add

Re: [libav-devel] [libav-commits] hevc: remove unused array min_cb_addr_zs

2014-07-07 Thread Diego Biurrun
On 2014-06-26 15:56, Gildas Cocherel wrote: Module: libav Branch: master Commit: a0e1c3517a656dd32293f054a339e0ac73328138 Author:Gildas Cocherel gildas.coche...@laposte.net Committer: Anton Khirnov an...@khirnov.net Date: Wed Jun 25 21:08:12 2014 +0200 hevc: remove unused array

Re: [libav-devel] [libav-commits] nut: Support experimental NUT 4 features

2014-07-07 Thread Luca Barbato
On 07/07/14 20:38, Diego Biurrun wrote: I guess you owe the crew a beer for ignoring this warning. Probably. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [libav-commits] mov: Export geotag metadata fields

2014-07-07 Thread Diego Biurrun
On 2014-06-06 09:49, Martin Storsjö wrote: Module: libav Branch: master Commit: e7d6d0bf3c5cc1bc048b0ddbc169a91862568e0c Author:Martin Storsjö mar...@martin.st Committer: Martin Storsjö mar...@martin.st Date: Tue Jun 3 14:48:19 2014 +0300 mov: Export geotag metadata fields The

[libav-devel] [PATCH] nut: Use nut-version in the version range check

2014-07-07 Thread Luca Barbato
It was wrongly left unchanged when the version field had been introduced. --- libavformat/nutdec.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/libavformat/nutdec.c b/libavformat/nutdec.c index 838c181..8ec67ae 100644 --- a/libavformat/nutdec.c +++

Re: [libav-devel] [PATCH] nut: Use nut-version in the version range check

2014-07-07 Thread Diego Biurrun
On Mon, Jul 07, 2014 at 08:49:45PM +0200, Luca Barbato wrote: It was wrongly left unchanged when the version field had been introduced. --- libavformat/nutdec.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) LGTM Diego ___

[libav-devel] [PATCH] mov: Mark a variable as unused

2014-07-07 Thread Martin Storsjö
This silences a warning with gcc. --- In my defense, clang didn't show this warning. --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 4a2d265..853c3e5 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@

Re: [libav-devel] [PATCH] mov: Mark a variable as unused

2014-07-07 Thread Diego Biurrun
On Mon, Jul 07, 2014 at 10:08:26PM +0300, Martin Storsjö wrote: This silences a warning with gcc. --- In my defense, clang didn't show this warning. :) --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) LGTM Diego ___

Re: [libav-devel] [PATCH] nut: Use nut-version in the version range check

2014-07-07 Thread Vittorio Giovara
On Mon, Jul 7, 2014 at 2:49 PM, Luca Barbato lu_z...@gentoo.org wrote: It was wrongly left unchanged when the version field had been introduced. Ok, you could mention the hash if you feel like it (c94e2e85cb6af8a570d8542a830556243bd32873). Vittorio

Re: [libav-devel] [PATCH] mov: Mark a variable as unused

2014-07-07 Thread James Almer
On 07/07/14 4:08 PM, Martin Storsjö wrote: This silences a warning with gcc. --- In my defense, clang didn't show this warning. --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 4a2d265..853c3e5 100644

Re: [libav-devel] [PATCH 7/8] yuv4mpegenc: do not access AVCodecContext.coded_frame

2014-07-07 Thread Vittorio Giovara
On Sat, Jul 5, 2014 at 5:20 AM, Anton Khirnov an...@khirnov.net wrote: Its contents are meaningful only if the stream codec context is the one actually used for encoding, which is often not the case (and is discouraged). Use AVCodecContext.field_order instead. --- libavformat/yuv4mpegenc.c

Re: [libav-devel] [PATCH] hevc: parse display orientation SEI message

2014-07-07 Thread Vittorio Giovara
On Wed, Jul 2, 2014 at 12:02 PM, Vittorio Giovara vittorio.giov...@gmail.com wrote: --- libavcodec/hevc.c | 15 +++ libavcodec/hevc.h | 5 + libavcodec/hevc_sei.c | 17 + 3 files changed, 37 insertions(+) ping -- Vittorio

Re: [libav-devel] [PATCH 1/4] fate: hevc: add missing version

2014-07-07 Thread Vittorio Giovara
On Wed, Jul 2, 2014 at 1:15 PM, Vittorio Giovara vittorio.giov...@gmail.com wrote: On Wed, Jul 2, 2014 at 2:48 AM, Anton Khirnov an...@khirnov.net wrote: On Mon, 30 Jun 2014 11:12:01 -0400, Vittorio Giovara vittorio.giov...@gmail.com wrote: On Mon, Jun 30, 2014 at 6:28 AM, Anton Khirnov

Re: [libav-devel] [PATCH] h264: parse display orientation SEI message

2014-07-07 Thread Vittorio Giovara
On Wed, Jul 2, 2014 at 11:58 AM, Vittorio Giovara vittorio.giov...@gmail.com wrote: --- libavcodec/h264.c | 15 +++ libavcodec/h264.h | 8 libavcodec/h264_sei.c | 22 ++ 3 files changed, 45 insertions(+) ping -- Vittorio

Re: [libav-devel] [PATCH] lavc: add pix_fmts_full to AVCodec

2014-07-07 Thread Vittorio Giovara
On Tue, Jul 1, 2014 at 1:23 AM, Vittorio Giovara vittorio.giov...@gmail.com wrote: Rather than using deprecated *J pixel formats, keep the full scale ones in a separate list, and check for avctx-color_range before opening the encoder. This slightly changes the format handshaking behaviour,

[libav-devel] [PATCH] libgsm: split decoder and encoder

2014-07-07 Thread Nidhi Makhijani
--- libavcodec/Makefile | 8 +-- libavcodec/{libgsm.c = libgsmdec.c} | 101 +- libavcodec/{libgsm.c = libgsmenc.c} | 115 +-- 3 files changed, 10 insertions(+), 214 deletions(-) copy libavcodec/{libgsm.c =